Új hozzászólás Aktív témák

• #15492 Sk8erPeter nagyúr umek7 #15491

  Új Válasz 2014-04-12 02:35:11 #15492

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  Sk8erPeter

  nagyúr

  válasz umek7 #15491 üzenetére

  Azt a qrva, de jó ronda. Hogy sikerült ezt így összehozni? Attól kifejezetten rossz minőségű lesz a kód, ha be van hányva egy sorba egy csomó minden - lásd azt a csodálatos while+regexp-tesztelés+változónak értékadás+sprintf+rand sort... Az ilyen hányadék kódhoz már szinte művészi tehetség kell.
  Ha ezt meg így kaptad, hát az szívás, de itt az ideje, hogy átalakítsd valami áttekinthető formátumúra.

  $i=($QUERY_STRING)?($QUERY_STRING):"10";
  helyette
  $i=( isset($QUERY_STRING) ? $QUERY_STRING : 10 );
  vagy
  $i=( !empty($QUERY_STRING) ? $QUERY_STRING : 10);

  Amúgy minek ide a $QUERY_STRING változóhoz a nagybetű? És ez valami globális változós undormány? Miért nem kapta meg ezt paraméterként a függvény, és lett egy default értéke a paraméternek, ami jelen esetben 10 (amennyiben nem lenne az megadva a fvhíváskor; ja, és nem mindegy, hogy nem "10" stringként, hanem 10 intként)?

  A $pwd változót meg deklaráld előre még az első while-ciklus előtt:
  $pwd = '';

  A regexp betűiből szándékosan maradt ki az i és l (mint ló), I (mint Ilona) és O betű, meg a 0-s és 1-es szám, mert valaki azt gondolta, ettől biztonságosabb lesz a dolog?

Új hozzászólás Aktív témák