Forum PHP.pl

[PHP] pobierz, plaintext 
if(false == $zmienna)
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
for ($i = 0; $i < 10; ++$i) {
  echo $i;
}
echo $i;
[PHP] pobierz, plaintext 

• niczym nie zabezpieczona rekurencja (tym gorsza, że uzależniona od parametrów z zewnątrz)
• echowanie czego popadnie prosto z requesta lub tablicy $_SERVER, konstrukcje typu ... where id = $_GET['id'] i tak dalej
• pobieranie wszystkiego z bazy jak leci albo wrzucanie do sesji jak leci - no bo wygodnie mieć wszystko w sesji, najlepiej całą bazę danych od razu wrzucić sobie do sesji, "by mieć taki kesz"
• "dziś miejsce na dysku nie jest już problemem" - słyszałem to już x razy po czym po chwili dysk pada bo logujemy do plików pierdów bez liku

• jednolita konfiguracja dla każdego środowiska, przez co regularnie commituje ustawienia lokalne albo po update wściekam się, że coś nie działa
• brak jakichkolwiek komentarzy czy dokumentacji, najczęściej im paskudniejszy kod tym komentarzy jak na złość mniej
• brak sprawdzania, czy coś się wykonało, założenie, że jak funkcja coś zwraca to z pewnością jest to spam, potem coś po prostu nie działa i sobie możesz szukać kilka dni no bo przecież "funkcja powinna wygenerować warninga chociaż jak jej kiepawo idzie"
• używanie gdzie popadnie różnych fantastycznych wynalazków cudownego dziecka skryptu typu eval, goto, dynamiczne generowanie zmiennych - ekstrakcja to mały pikuś przy tym
• obiektowość rozumiana tak, że np. co każde 100 linijek walniemy klasę a co 20 metodę, tworzenie jakiś dziwnych fabryk czy raczej młockarni obiektów które tworzą takie plewy, że człowiek tęskni za dynamicznym generowaniem zmiennych

Cytat(thek @ 14.01.2013, 22:13:48 )

[PHP] pobierz, plaintext 
for ($i = 0; $i < 10; ++$i) {
  echo $i;
}
echo $i;
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
eval('$i = ' . $j . ';');
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$records = $db->getRecords("SELECT * FROM table1");
 
foreach($records as $record)
{
    $item = $db->getRecords("SELECT * FROM table2 WHERE category_id = ?", $record['id']);
    echo $item['name'];    // jakakolwiek operacja
}
 
/**  a tutaj jak to powinno wyglądać  **/
$records = $db->getRecords("SELECT * FROM table1 t1 JOIN table2 t2 ON t2.category_id = t1.id");
 
foreach($records as $record)
{
    echo $item['name'];    // jakakolwiek operacja
}
 
/**  lub:  **/
$records = $db->getRecords("SELECT * FROM table1");
$ids = array();
 
foreach($records as $record)
{
    $ids[] = $record['id'];
}
 
$records = $db->getRecords("SELECT * FROM table2 WHERE category_id IN (?)", implode(', ', $ids));
 
foreach($records as $record)
{
    echo $item['name'];    // jakakolwiek operacja
}
[PHP] pobierz, plaintext 

Cytat(thek @ 15.01.2013, 09:16:55 )

[HTML] pobierz, plaintext 
<img src="/admin/post.php?act=deleteAll&confirmed=1" />
[HTML] pobierz, plaintext 

Cytat(Sephirus @ 15.01.2013, 09:02:05 )

Przypomniał mi się tutaj jeden fajny bajer jaki widziałem w kodzie młodego kolegi... i żeby nie było to autentyk