Siema
Od jakiegoś czasu uczę się php.. Gdy załapałem podstawy postanowiłem zrobić blog. Nie jest on skończony.. właściwie jest tylko logowanie ,rejestracja, wyświetlanie wpisów i komentarze. Nie zamierzam go kończyć . Mam w planach naukę mvc. Chciałbym abyście ocenili sam kod.. powiedzieli jakie błędy robię, może zaczynam mieć jakieś niepoprawne nawyki.
Wiem że kod nie jest zbyt czytelny.. z niektórych błędów zdaje sobie sprawę ale stosuje je po prostu z lenistwa ;p Oczekuje jakiś wskazówek.. być może przyzwyczajam się do jakiś "złych praktyk".

kod:
http://www.wrzuc.to/oUYOI0Tt.wt

[PHP] pobierz, plaintext 
<?php
//skopiowane z jezus cms
 
 /**
 ** JEZUS CMS 
 ** @autor JEZUS DEVELOPER TEAM
 ** licencja gnu/gpl 
 ** Nie biore odpowiedzialności za nic.
 **  klasa obsługi bazy mysql 
 ** @version 0.1
 **/
 defined('LOL') or die('Kaaaaboom, access denied');
[PHP] pobierz, plaintext 

Jak się chcesz już na czymś wzorować, to wzoruj się na czymś, co jest porządnie napisane...


Z takim podejściem, to za dużo nie zrobisz.

A czemu ona jest aż taka zła ?



Więc chyba będę musiał zmienić podejście.

- pakuj paczki jako ZIP, zapomnij że RAR istnieje
- jak już coś pakujesz, to pakuj cały katalog, a nie tylko pliki jakie się w nim znajdują.
- deklaruj w plikach z klasami jakiego rodzaju są metody public, private czy protected.
- gdy piszesz kod, zamień w swoim edytorze, aby tabulacja była zamieniana na spacje 3 lub 4.
- opisuj kod poprawnie, czyli jakiś wstęp by się przydał, a nie tylko wykaz parametrów.
- __autoload() jest stary, zapoznaj się z SPL, lub specjalnymi ładowarkami które są klasą.
- pisz nazwy klas, zmiennych itd. po angielsku.


To nie jest poprawny kod dokumentacji.

Tabulacje są po to by robić wcięcia, spacje by robić odstępy.
Nie rozumiem, dlaczego zamienianie tabulacji na spacje to dobra praktyka - raczej zła. Dla jednych wcięcie z dwóch spacji może być za małe, z 4 za duże, z 3 też złe - w przypadku tabulacji wystarczy sobie ustawić w edytorze ich długość jak chcesz i osoby z dużą rozdzielczością ustawiają sobie na 4, z małą na 2 lub 1. Do tego zajmuje to 1 bajt, nie 3-4.

Nie proszę nie rozśmieszaj mnie! ;] Każdy edytor z jakim miałem coś wspólnego zapisuje [tab] jako spacje, a ich ilość zależy od ustawień w programie, więc z jakiej paki wtrącasz tutaj jakieś porównania jak jest to to samo? Ja też preferuję [tab] == 4x[space], tak się przyjęło i większość kodu który jest w sieci tak właśnie jest formatowana.

Dobrze że to zauważyłeś, więc rozumiesz na czym polegają wcięcia w kodzie? Jeśli używasz do formatowania kodu, samej tabulacji, to współczuję każdemu kto będzie chciał go kiedykolwiek edytować u siebie, po prostu Twój kod się rozjedzie... Zresztą nie trzeba długo szukać, wklej kod z edytora tu na forum... wersja z tabulacją się rozjedzie już na początku, a ze spacjami nie.

zal.pl to co wy tu wypisujecie, ja wcale nie daje spacji i tabow bo tak sie przyzwyczailem i tak mi pasuje, i co? mam z tego powodu zly kod? kazdy ma swoje przyzwyczajenia i tyle, tak samo z jezykiem, jestem polakiem wiec pisze nazwy klas i zmiennych po polsku a wy zawsze wszystkim narzucacie ze trzeba pisac po angielsku. NIE TRZEBA! Kazdy pisze jak chce i to moim zdaniem nie podlega ocenie. to jest totalny bezsens, zaraz powiecie, ze zamiast nazwy zmiennej $zobacz powinien miec $sprawdz, madre to?

Często zgłaszasz swój sprzeciw wobec świata na poziomie gimnazjalnym?
Jeśli piszesz dla siebie i tylko DLA SIEBIE, to nic nikomu do tego. Nie wiem czy zauważyłeś, ale większość koderów na tym forum pracuje czy rozwija skrypty na szerszą skalę niż PL. A jak Ci się zdaje, czy koder z FR będzie wiedział co to jest "$zobacz"? Poza tym język angielski w programowaniu to standard, czy Ci się to podoba czy nie.


Piszesz ciurkiem? Toć się nawet parser PHP wyłoży.

daje entery, to wystarcza jesli kod jest dobrze opisany a wy oczywiscie wielcy programisci piszecie skrypty po angielsku i je sprzedajecie za wielka kase obcokrajowcom.... gratuluje:) pisanie po angielsku to zaden standard tylko narzucanie i oczywiscie sie sprzeciwiam glupim narzucaniem kto jak ma pisac i czego sie uczyc i bede to robil nadal a, ze Ty w tym widzisz gimnazjalny sprzeciw to nie moj problem, mam swoje zdanie na kazdy temat i tyle.

Cieszy mnie fakt że masz zdanie na każdy temat, ale powinieneś też posiadać tajemną wiedzę wysławiania się. Pisanie po angielsku to nie standard? Ciekawe, a mógłbyś to bardziej rozwinąć? Czyli że Twórcy ZendFramework (lub innego) pisząc go z angielskim nazewnictwem mieli taki kaprys? A w jakim języku napisane są nazwy np. funkcji w PHP? Japońskim? Widzę w tym poziom gimnazjum, jeśli nie niżej... "zal.pl"?

Zasada jest taka, że niekoniecznie tylko Ty w przyszłości będziesz rozwijał swój kod. Jak chcesz komuś utrudniać życie, to oczywiście- możesz.

Wklejam kod z edytora tutaj:

[PHP] pobierz, plaintext 
<?php
// :(
	// ONCE..
		// TWICE..
[PHP] pobierz, plaintext 

Działa
Jeśli mój kod ktoś będzie chciał oglądać, wystarczy, że zmieni sobie długość taba na odpowiadającą. Jeśli ty chcesz mieć kod z wcięciami jako 3 spacje, a ja wole, kiedy są dwie (jako tab), musze przejechać cały projekt wyrażeniem regularnym i zamienić. Nieprawdaż, że prościej zmienić długość taba?

Nie rozśmieszajcie mnie. Każde IDE, a nawet gEdit, posiada opcję opcję "zapisuj taby jako spację", którą można wyłączyć/włączyć oraz ustawianie długości taba. Widocznie nie psojrzeliście nawet na konfiguracje edytora.

Co do 'ciekawskiii' - to nie jest poziom gimnazjalny, jako prawie-absolwent stwierdzam, że trochę niższy.
Angielski jest językiem informatyki - znajomość jest niezbędna. Masz swoje zdanie na każdy temat? Ja mam swoje zdanie na tematy, o których coś wiem.

Na koniec polecam książkę "Clean Code. A handbook of Agile software craftmanship.".

Zend i funkce php sa w jezyku angielskim moze dlatego, ze byly odrazu przeznaczone dla ludzi na calym swiecie? oczywiscie ten blog tez jest przeznaczony dla wszystkich na calym swiecie a w najgorszym wypadku sprzeda ten blog obcokrajowcowi, nie rosmieszaj mnie.

jezeli bede chcial cos napisac i sprzedac obcokrajowcowi to bede pisal po angielsku, oczywiscie np. interia.pl i onet.pl czy inne nie sa pisane w standardzie bo sa w jezyku polskim...

tu zgadzam sie z prowseed ale nie do konca, to nie zadna zasada tylko tak jak pisalem, jezeli chcesz sprzedac obcokrajowcowi to piszesz po angielsku a jesli pisze dla siebie to nikt mi nie bedzie wmawial ze angielski to standard tylko pisze jak JA CHCE

absolwent? haha to teraz czuj sie kims bo jestes wyksztalcony i co z tego?

Właśnie pokazałeś że nie działa ile tam masz tej tabulacji z 10 spacji?



Tylko po co mam zmieniać cokolwiek u siebie w edytorze? Spacje nie są inwazyjne, bo nie ma znaczenia czy jest ich 3 czy 4, tabulacja w każdym edytorze jest inaczej interpretowana i może rozwalić cały układ.


Przeczytaj Waść dokładnie, bo bzdury snujesz.

ciekawskiii - jesteś genialny.

Czemu w takim razie ja mam zmieniać sobie twoje cztery spacje na dwie, jesli lubie male wciecia?
Spacje nie są inwazyjne? Mnie denerwuje kiedy muszę robić 4 razy backspace by usunąć jedno wcięcie, wolę wcisnąć to tylko raz.
Co do ostatniego cytowanego fragmentu, odpowiedź do:

Otóż każdy edytor z którym ja pracowałem domyślnie zapisuje tab jako tab i tak powinno być.

Co do forum: czy ktoś tutaj zamieszcza jako źródło tak dużo projektów, że jest to wada? W edytorze można sobie ustawić długość taba, jeżeli twoim zdaniem w projekcie jest ich zbyt wiele, stawiasz na 2, w przeciwnym razie możesz dać 4. A co, gdy ktoś formatuje kod dwoma spacjami, a one tobie nie odpowiadają i chcesz mieć większe wcięcia?

To zdecyduj się w końcu. Dla mnie nie ma różnicy w tym gdzie prezentowany/edytowany jest kod, ważne żeby miał ręce i nogi. Co mnie obchodzi Twoje pojęcie estetyki że wolisz mieć 2 spacje zamiast 4. To kompletnie nie ma znaczenia, bo spacja nie rozwali mi wyglądu, stylu zakodowanego dokumentu, czego nie można powiedzieć o tabulacji. Używając spacji, nikt do nikogo nie musi się dostosować, dostajesz kod taki jaki jest, ewentualnie masz mniej 2,3 spacje, ale to nie jest coś strasznego... Używając tabu u Ciebie może być ok, ale u mnie tab może mieć tak jak tu na forum 10 spacji... widzisz już różnicę?

Świetnym rozwiązaniem jest http://nickgravgaard.com/elastictabstops/ szkoda że tak mało jest implementacji.

O matko, jak jeszcze mogę dyskutować na temat języka, toż kod bez wcięć to samobójstwo, chyba że Twój rekord to max 100 lini!

My tutaj pokazujemy jak pisać się powinno by w przyszłości nie trzeba było zmieniać na siłę swoich nawyków, a już od samego początku stosować się do norm, pamiętam kod gdzie była tabela w htmlu i jej każdy wewnętrzny znacznik był zapisywany w nowej linii w taki sposób:
<?php echo '<tr>'; ?>
<?php echo '<td>'; ?>
Itd...

Czy to nie zadziała? Działa i to tak jak powinno ale edycja takiego pliku to już koszmar, albo jeszcze ciekawsze praktyki, przypisanie $_POST do $superUkrytyPost i przypisanie do $_POST pustego array. Jak widać wszystko się da, ale nie jest to poprawne i komuś kto po Tobie będzie to poprawiać lub dodawać coś nowego przyjdzie się zastrzelić.



Powiem Ci że ide same rozpoznają "4 spacje" (w zależności od configu) jako coś co można usunąć jednym backspace, nie spotkałem się jeszcze z ide który w standardzie zapisywał by tab jako tab, ale faktem jest że taka opcja widnieje nie zmieni to jednak mojego (i jak widać nie tylko mojego) zdania.

A jakieś inne błędy oprócz tych nieszczęsnych tabulatorów znaleźliście ?

Poza wymienionymi wyżej przyczepiłbym się do:

[PHP] pobierz, plaintext 
$password = $_POST['password']
[PHP] pobierz, plaintext 

Nie przypisuj superglobalnych do zmiennych, szkoda zachodu.

[PHP] pobierz, plaintext 
if(WERYFIKACJA == '1') 
[PHP] pobierz, plaintext 

Samo umieszczenie WERYFIKACJA w warunku wystarczy (jeśli ma być true/false).

[PHP] pobierz, plaintext 
@$autor = mysql_real_escape_string(core::filtruj_text($_POST['autor']));
[PHP] pobierz, plaintext 

Co Ty zoo otwierasz? Wywal @ w ogóle zapomnij że coś takiego istnieje.

[PHP] pobierz, plaintext 
if(isset($_GET['id_wpisu'])) 
    $id_wpisu = (int)$_GET['id_wpisu'];
else
    $id_wpisu = 0;
$tpl->id_wpisu = $id_wpisu;
[PHP] pobierz, plaintext 

Używaj klamer jeśli po else jest więcej niż 1 instrukcja zmiennej, poprawia to czytelność kodu.

[PHP] pobierz, plaintext 
for($i=0 ; $wiersz = $baza->fetch('array') ; $i++)
{
    $wpisy[$i]['id']= $wiersz[0];
    $wpisy[$i]['autor']= user::give_nick($wiersz[1]);
    $wpisy[$i]['tytul']= $wiersz[2];
    $wpisy[$i]['tresc']= $wiersz[3];
}
[PHP] pobierz, plaintext 

Pętla fireach by w zupełności starczyła. poza tym... $wiersz = $baza->fetch('array') ?

Jest jeszcze:

[PHP] pobierz, plaintext 
    <?php
        Helpers::komunikat($message_tresc, $message_typ);
    ?>
 
    <?php
        echo user::give_nick($post['id_autor']).'<br />';
        echo $post['tytul'].'<br />';
        echo $post['tresc'].'<br />';
    ?>
[PHP] pobierz, plaintext 

Tak jest lepiej:

[PHP] pobierz, plaintext 
    <?php
        Helpers::komunikat($message_tresc, $message_typ);
 
        echo user::give_nick($post['id_autor']).'<br />';
        echo $post['tytul'].'<br />';
        echo $post['tresc'].'<br />';
    ?>
[PHP] pobierz, plaintext 

A jeszcze lepiej jakbyś zrobił to za pomocą 1 echo. I jeszcze właśnie co do echo...

Nie

[PHP] pobierz, plaintext 
echo "<div class=\"komentarz\">";
[PHP] pobierz, plaintext 

tylko:

[PHP] pobierz, plaintext 
echo '<div class="komentarz">';
[PHP] pobierz, plaintext 

Używaj apostrofów. Działa to wydajniej, bo PHP traktuje wtedy to jako tekst bez dodatkowego parsowania.

Należy sobie to dokładnie wziąć do siebie, apostrofy, apostrofy i jeszce raz apostrofy.
Mam taką ciekawostkę, to zjawisko o którym wspomina kolega wyżej, że PHP szuka sobie w cudzysłowach dodatkowych rzeczy (np zmiennych) nosi nazwę interpolacji

Nie jesteś konsekwentny:

[PHP] pobierz, plaintext 
    <?php
        Helpers::komunikat($message_tresc, $message_typ);
    ?>
 
    <?php
        echo user::give_nick($post['id_autor']).'<br />';
        echo $post['tytul'].'<br />';
        echo $post['tresc'].'<br />';
    ?>
[PHP] pobierz, plaintext 

komunikat, give_nick - zdecyduj się czy piszesz po polsku czy po angielsku...

Jesteś Polakiem, to pisz po polsku, a nie jakieś tam message_tresc

No właśnie nie..
$wpisy[id wpisu][id] - chciałem aby wszystkie wpisy byly w jednej tablicy.. kolejne wpisy pod kolejnym "id wpisu"
Nie bardzo ogarniam jak by to można zrobić w foreach
No chyba że cię źle zrozumiałem

No i za wszystkie uwagi dziękuje na pewno mi się przydadzą

Przecież pobierasz i tak wszystkie wpisy w tablicy Grupujesz te o konkretnym id, lub wyświetlasz rekurencyjnie.

[PHP] pobierz, plaintext 
$i=0;
foreach ($baza->fetch('array') as $key => $value) 
{
    $i++;
    $wpisy[$i]= $wiersz;
}
[PHP] pobierz, plaintext 

o coś takiego ci chodzi ?

Nie mogę patrzeć na ten kod pisany po polsku, tragedia.

To jakby Lays pisać Lejs, bo przecież jesteśmy polakami.

@bladeer

Nie ma znaczenia czy piszesz kod dla Polaka, Anglika, Niemca, Rosjanina czy kogokolwiek - Przyjęło się że kod pisany jest po angielsku, a nie po nagasakarukitukizimbaubmbecweulne, bo jeśli ktoś będzie chciał do tego projektu coś dołożyć to nie będzie się w stanie połapać w tym bałaganie.

Co do samego kodu:
1. Przede wszystkim niczego nie tłum jak już wcześniej wypatrzył !*!
2. Skoro piszesz obiektowo to zastosuj się do zasady SRP.
3. Klasa Db - Po co Ci ona? Od czego masz PDO stosowane już praktycznie w każdym projekcie? Jeśli chcesz dodać jakieś metody czy narzędzia, to zrób wyznaczone do tego klasy korzystające z niej lub stwórz własny adapter dla PDO.
4. Nie korzystaj z global - Wiele tematów przewałkowało się nt. jego wad i braku zalet, skoro już korzystasz z klas i obiektów to wykorzystaj jakiś wzorzec - Może być to Dependency Injection lub nawet dla tak małej aplikacji Registry.
5. Nie korzystaj wewnątrz klas z takich danych jak np $_SESSION - Po prostu przekaż to jako argument metody.
6. Nie używaj nigdy echo w klasach, funkcjach czy czymkolwiek podobnym, zwracaj dany wynik, a dopiero na zewnątrz go w jakiś sposób wyświetlaj.
7. Skorzystaj z jakiegoś sensownego autoloadera klas jak np OPL.

Ok.. pobawiłem się troche w mvc. wyklekociłem coś takiego
http://www.wrzuc.to/gJQ5gG.wt
Wiem. dalej jest __autoload() zamiast SPL i pare nazw klas po polsku ale kod pisałem zanim napisałem ten temat i już tak zostało.
Pewnie to mvc wygląda tragicznie .. ale chciałem spytać czy dobrze zrozumiałem założenie mvc

Wrzuć to gdzie indziej, bo nie idzie ściągnąć pliku.



Nie ma czegoś takiego jak "dobre MVC" szczególnie w PHP. Interpretacja dowolna. Twoje MVC jest na tyle dobre na ile sam je zrozumiesz.

@bladeer
MVC Wikipedia
MVC Forum PHP

Z tego drugiego linku przytoczę fragment:





Wrzucenie jakiegoś autoloadera to kwestia minuty. Wykorzystaj chociażby ten PHP 5.3 Namespace Autoloader.