Forum PHP.pl > [PHP] Struktura aplikacji

Pomoc - Szukaj - Użytkownicy - Kalendarz

MadMan77

4.03.2018, 18:08:47

Witam,

Na wstępie oświadczam iż jestem początkujący dlatego też prosiłbym o wyrozumiałość i w miarę prosty język w odpowiedziach.
W internetach nie znalazłem jednoznaczniej odpowiedzi na moje pytanie dlatego zwracam się do Was.
Wszystko co do tej pory zrobiłem działa dlatego też nie dodaję kodu. Pytanie jest bardziej natury metody wędrowania danych między stronami i etyką tworzenia właśnie takich akcji.

Próbuję stworzyć pewną aplikację internetową która będzie zapisywała do bazy dane, które później będzie można odczytywać.
Stworzyłem już formularz logowania. Zrobiłem także stronę główną na której działa dodawanie danych do bazy.
Ze strony głównej formularzem metodą POST wysyłam dane do innej strony, która to dodaje dane do bazy i wraca do strony głównej. Teraz właśnie rodzi się moje pytanie : Czy powinienem tworzyć nową stronę i wysyłać tam dane czy wykonywać tą operacje na stronie głównej? Strona główna będzie takim centrum dowodzenia i będzie znajdowało się tam sporo rubryk w których będę dodawać lub odczytywać dane. Dlatego też będzie to potężny plik jeżeli wszystko będę robić w jego trzewiach.

Staram się dzielić aplikację na bloki dlatego też jest już kilka stron np. connect.php (aby nie wpisywać tego w kodzie wszędzie) logout.php, zaloguj.php który loguje i sprawdza użytkowników czy ktoś jest zalogowany itp. Czy powinienem tak samo robić z dodawaniem danych do bazy. Tak aby to inny plik zajmował się wysyłaniem do bazy a strona główna będzie tylko to wyświetlać. Jeżeli tak to rodzi się kolejne pytanie odnośnie wyświetlenia komunikatu, że rekord dodał się do bazy. Aktualnie wyświetla mi się to na tej dodatkowej stronie. Ostatnio zrobiłem przekierowanie, żeby po dodaniu danych wracało do strony głównej jednak wtedy nie wiem jak dodać komunikat o poprawności dodania danych. Jeżeli jednak wykonywać to wszystko na stronie głównej wtedy będę umiał dodać taki komunikat. Tylko nie umiem wtedy zrobić aby wyświetlał się na chwilę a nie ciągle tam wisiał ale to już problem na inny temat.

Dziękuję z góry za przeczytanie mojego posta i ewentualne odpowiedzi.
Pozdrawiam MadMan

leonpro778

4.03.2018, 18:42:13

Co do SAMEJ struktury plików to raczej nie ładuj wszystkiego w jeden plik (owszem, da się ale potem się w nim odnaleźć to tragedia). Chcesz porządku w kodzie? Zainteresuj się MVC (dokładnie tam będzie opis jak dane wędrują po stronie). Lepiej chyba abyś pokazał jednak ten działający kod aby było się do czego odnieść.

MadMan77

4.03.2018, 19:10:51

To jest mój główny plik. Head wykasowałem bo nic nie wnosi do danej sprawy, nie chciałem zaśmiecać.

[PHP] pobierz, plaintext 
<?php 
 session_start();
 if(!isset($_SESSION['zalogowany']))		
 {											
	 header('Location:index.php');			
	 exit();
 }
?>
<!DOCTYPE HTML> 
<html lang="pl">
<body>
<?php
	echo "<p>Witaj ".$_SESSION['login'].'! [<a href="logout.php">Wyloguj</a>]</p>';	
?>
 
	<form action="nazwa.php" method="POST">
	inwestycja<br/>
    <input type="text" name="nazwa"><br/>
	<input type="submit" value="zapisz">
	</form>
 
[PHP] pobierz, plaintext

Tutaj (nazwa.php) ładuje to co dostałem w formularzu i dodaje do bazy

[PHP] pobierz, plaintext 
<?php 
 
 if(isset($_SESSION['zalogowany']))
 {
	 header('Location:index.php');
	 exit();
 }
require_once "connect.php"; 
 
$polaczenie = new mysqli($host, $db_user, $db_password, $db_name);
 
$nazwa=$_POST['nazwa'];
 
if($nazwa) 
{
	$sql=mysqli_query($polaczenie, "INSERT INTO inwestycje (idinw, nazwa) values ('','$nazwa')");
                if($sql) 
		{
 			header('location: glowna.php');
			$polaczenie->close(); 
		}
}
?>
[PHP] pobierz, plaintext

Jest jeszcze strona zaloguj.php Jednak jej chyba tutaj dodawać nie muszę bo robi zupełnie co innego niż dotyczy ten problem. Chyba że się mylę.
Jest to sam początek dlatego też plik "glowna" rozrośnie się. Dlatego tak jak pisałeś, żeby się połapać dodałem plik nazwa.php który ma robić konkretną robotę.

viking

4.03.2018, 19:18:38

Taki kod to jedna wielka sieczka z którą praktycznie nic nie można potem zrobić. Zobacz jakiś prosty framework np slim, poczytaj może o systemach szablonów, poczytaj o prepared statements i bezpieczeństwie aplikacji.

MadMan77

4.03.2018, 19:34:34

Rozumiem.
Aktualnie bezpieczeństwo aplikacji nie jest na pierwszym miejscu ponieważ będzie stała ona na prywatnym serwerze w prywatnej sieci.
Chciałbym żebyś rozwinął myśl sieczki. Dlaczego z takim kodem później nie będę mógł nic zrobić?
Jestem początkujący. Uczę się php około tygodnia i staram się używać każdej nowej zdobytej wiedzy.
Na razie jest to bardziej nauka niż kodowanie. Jednak chciałbym napisać użyteczny program dlatego też każde rady bardziej doświadczonych wezmę do siebie. Nauka przychodzi powoli ponieważ muszę przyswoić sporo nowej wiedzy. Chciałbym najpierw opanować php na rozsądnym poziomie a dopiero później zabierać się za frameworki. Chyba ,że powinienem równolegle uczyć się np slim o którym wspomniałeś.

leonpro778

4.03.2018, 23:21:35

Cytat(MadMan77 @ 4.03.2018, 19:34:34 )

Rozumiem.
Aktualnie bezpieczeństwo aplikacji nie jest na pierwszym miejscu ponieważ będzie stała ona na prywatnym serwerze w prywatnej sieci.

Błąd. Skoro zaczynasz z PHP od razu sobie "wbijaj" pewne standardy (mi to zajęło spoooro czasu). Później będziesz miał łatwiej.

Cytat(MadMan77 @ 4.03.2018, 19:34:34 )

Chciałbym żebyś rozwinął myśl sieczki. Dlaczego z takim kodem później nie będę mógł nic zrobić?

To co tutaj pokazałeś to taki bardzo prosty "prototyp" dodawania rekordu do bazy (jak słusznie zauważył viking - bez żadnych zabezpieczeń)

Za bardzo nie wiadomo co ta Twoja stronka ma robić bo jak na razie to widzimy, że dodaje rekord do bazy. Rozpatrz sobie przykład. Użytkownik wchodzi na Twoją stronę i dostaje okno logowania. Loguje się i następnie co... widzi tylko 'Witaj XXX' i pole input do wpisania jakiejś nazwy. Naciska on ZAPISZ i... co dalej widzi? Znowu IDENTYCZNĄ stronę jak po zalogowaniu.

Nie ma nawet szansy przejrzeć co dodał i ile tych rzeczy jest. Teraz jak Ty to robisz? Budujesz kolejną stronę przegladaj.php żeby można było sobie przejrzeć dotychczasowe wpisy. Efektem jest to, że masz już DWIE strony glowna.php oraz przegladaj.php. Na KAŻDEJ z nich musisz sprawdzić czy użytkownik jest zalogowany. Jeżeli jest to wyświetlasz mu ten komunikat:

[PHP] pobierz, plaintext 
<?php
	echo "<p>Witaj ".$_SESSION['login'].'! [<a href="logout.php">Wyloguj</a>]</p>';	
?>
[PHP] pobierz, plaintext

I teraz pomyśl nad jedną rzeczą. Klient wymyślił aby napis witaj był czcionką czerwoną z dopiskiem 'Sz. P.'. Co Ty robisz? Otwierasz mozolnie KAŻDY plik zawierający ten zapis i zmieniasz. W przypadku DWÓCH plików to trudne nie jest ale jak masz tych plików 20? To taki najprostszy przykład czemu nie wolno trzymać wszystkiego w jednym pliku.

Jeżeli nie chcesz przesiadać się od razu na frameworka pisz przynajmniej w taki sposób aby ktoś, kto po tobie weźmie ten kod umiał się w nim odnaleźć

P.s. W pliku nazwa.php

[PHP] pobierz, plaintext 
if(isset($_SESSION['zalogowany']))
 {
	 header('Location:index.php');
	 exit();
 }
[PHP] pobierz, plaintext

Powinno być chyba:

[PHP] pobierz, plaintext 
if(!isset($_SESSION['zalogowany']))
 {
	 header('Location:index.php');
	 exit();
 }
[PHP] pobierz, plaintext

nospor

5.03.2018, 11:14:36

Ja tylko tak na marginesie odnosnie tej wypowiedzi

Cytat

Co do SAMEJ struktury plików to raczej nie ładuj wszystkiego w jeden plik (owszem, da się ale potem się w nim odnaleźć to tragedia

Nedzne odnalezienie sie w pojedynczym pliku to najmniejszy z problemow. Glownym jest to, ze jak ten jeden plik bedzie sobie tak rosl i rosl to w pewnym momencie strona zacznie sie ladowac kilka sekund.

Kiedys zrobilem test i machnalem jeden duuuuzy plik php po czym odpalilem stronke.... To bylo bolesne

alberthus

5.03.2018, 11:40:54

Takie pytanie do kolegów wyżej?

Dlaczego wciskacie autorowi frameworki, prepared statements, bezpieczeństwo i pewnie zaraz PDO skoro on podstaw php dobrych nie ma?

viking

5.03.2018, 11:50:01

A na czym ma pracować? A starym Mysql? Pisząc aplikację dowolną powinno się od razu robić wszystko zgodnie z dobrymi praktykami. Prepared statements to az jedna linia więcej. Mikroframework pozwoli mu zrozumieć jak można pisać inaczej oraz jak uporządkować swój kod.

MadMan77

5.03.2018, 18:53:31

Miło mi, że już kilka osób włączyło się w temat. Pomoże mi to poszerzyć swoją wiedzę na temat php więc dzięki

Teraz odniosę się do kilku spraw i zadam kila wynikłych z tego pytań :

Cytat(leonpro778 @ 4.03.2018, 23:21:35 )

Błąd. Skoro zaczynasz z PHP od razu sobie "wbijaj" pewne standardy (mi to zajęło spoooro czasu). Później będziesz miał łatwiej.

Zabezpieczenia, które zastosowałem odnoszą się do wstrzykiwania sql przez formularz. Jednak są one w pliku zaloguj.php, którego tutaj nie pokazałem. Fakt w tych kodach nie uwzględniłem tego. Co można dodać do zabezpieczeń oprócz walidacji danych od użytkownika? Proszę tylko o same hasła resztę będę starał się sam doszukać.

Cytat(leonpro778 @ 4.03.2018, 23:21:35 )

To co tutaj pokazałeś to taki bardzo prosty "prototyp" dodawania rekordu do bazy (jak słusznie zauważył viking - bez żadnych zabezpieczeń) smile.gif Za bardzo nie wiadomo co ta Twoja stronka ma robić bo jak na razie to widzimy, że dodaje rekord do bazy. Rozpatrz sobie przykład. Użytkownik wchodzi na Twoją stronę i dostaje okno logowania. Loguje się i następnie co... widzi tylko 'Witaj XXX' i pole input do wpisania jakiejś nazwy. Naciska on ZAPISZ i... co dalej widzi? Znowu IDENTYCZNĄ stronę jak po zalogowaniu.

Tak. Jest to prosty sposób dodawania danych do bazy. Jednak dla mnie był skomplikowany zanim tego nie rozgryzłem. Na razie tylko tak potrafię. Użytkownik po zalogowaniu będzie widział wiele pól w których będzie mógł dodawać dane. Idea tej aplikacji jest taka, że właśnie użytkownik nie może widzieć dodanych danych. Z aplikacji będzie korzystało więcej osób. Będą dodawały tam dane dotyczące czasu pracy itd dlatego nie mogą tego widzieć. Taką możliwość będą miały np tylko dwie osoby.

Cytat(leonpro778 @ 4.03.2018, 23:21:35 )

Na KAŻDEJ z nich musisz sprawdzić czy użytkownik jest zalogowany. Jeżeli jest to wyświetlasz mu ten komunikat:

[PHP] pobierz, plaintext 
  <?php
    	echo "<p>Witaj ".$_SESSION['login'].'! [<a href="logout.php">Wyloguj</a>]</p>';	
    ?>
[PHP] pobierz, plaintext

I teraz pomyśl nad jedną rzeczą. Klient wymyślił aby napis witaj był czcionką czerwoną z dopiskiem 'Sz. P.'. Co Ty robisz? Otwierasz mozolnie KAŻDY plik zawierający ten zapis i zmieniasz.

Muszę sprawdzać na każdej stronie czy jest zalogowany żeby nikt postronny nie mógł wejść na tą stronę wpisując np adres z reki w przeglądarkę ( to odnośnie zabezpieczeń). W takim razie w jaki sposób mogę sobie ułatwić taką zmianę jaką Ty podałeś. Z góry dziękuję za podpowiedz.

leonpro778 Dzięki za wychwycenie błędu w pliku nazwa.php

Ps. Nie wiem jak dodać, że pomogłeś.

nospor brałem właśnie to pod uwagę dlatego nie chce tego doświadczać

Cytat(viking @ 5.03.2018, 11:50:01 )

Staram się pisać nowszymi komendami. Dlatego np zamiast mysql użyłem w kodzie mysqli. Niestety aktualnie pisze zgodnie z moim stanem wiedzy. Jestem świadom, że tą aplikację poprawiał będę parę razy. Jednak dzięki temu może uda mi się nauczyć php na żywym przykładzie nie przepisując jedynie wzorów z internetu, które podane tylko same nie maja większego zastosowania. Podczas pisania tej aplikacji staram się na bieżąco pokonywać napotkane problemy i tym samym uczyć się.
Chciałbym pisać kod zgodnie z dobrymi praktykami jednak do końca ich nie znam. Manual na razie mnie jakoś odstrasza
dlatego wspomagam się tutorialami.
Może ktoś z Was mógłby polecić mi jakąś książkę ? lub inne źródło wiedzy. Czy jednak zmusić się do studiowania manuala?

leonpro778

5.03.2018, 20:40:25