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

• #6542 Speeedfire félisten Sk8erPeter #6506

  Új Válasz 2011-04-07 17:48:27 #6542

  Ú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

  

  Speeedfire

  félisten

  válasz Sk8erPeter #6506 üzenetére

  Ezt a bejegyzést meg észre sem vettem.

  Miért nem használod fel a $_GET['phpoldal'] változót? Úgy értem, akkor minek adod át ennek a query stringnek a címet?

  Így láttam célszerűnek anno, minden eshetőségre fel akartam készülni. Lehet, hogy pont nem a legjobb megoldás rá. Majd egyszer lehet nekiesek megint, jobban átgondolva az egészet.

  http://localhost/!!!szapar.hu/
  Ehelyett meg létrehozhatnál egy bejegyzést a hosts fájlban, meg apache-beállításokban egy VirtualHost-ot, és lehetne a címe http://szapar.local Így azé' szebb, meg nem kell annyit pötyögni a cím beírásához.
  Tudom, csak nincs kedvem mindegyiket beírni. Nem mintha sokáig tartana, csak localhoston annyira nem zavar a dolog. Ha meg felkerül a serverre akkor meg már édes mindegy.

  $valogatas = "select * from szapar_alias where eng = '".$uri."' ";
  Itt azért az $uri változót nem ártana escape-elni, az SQL Injection elkerülése érdekében!
  $i= 0;
  foreach ($valogat as $ertek) {
  if ($i != 0) {
  parse_str($ertek);
  }
  $i++;
  }
  Ezt nem is értem, minek csinálod, ha utána egyáltalán sehol nem használod fel az $ertek változót?
  Vagy felhasználod, csak valami include-olt fájlban? Vagy csak benne maradt?
  Erre eddig nem is gondoltam, tényleg egy biztonsági rés. Ki is javítottam már.
  Már hogyne használnám fel az $ertekeket. Azok az oldalnak a "$get paraméterei", a script végén hívom meg a php fájlokat amikben felhasználom.

  if (!mysql_query($valogatas,$con)) {
  die('Hiba: ' . mysql_error());
  }
  Itt a die() helyett érdemesebb lenne inkább valami felhasználóbarátabb hibaüzenetet, hogy nem elérhető az adatbázis, látogasson vissza később. Ráadásul a felhasználónak semmi köze a konkrét hibaüzenethez. Nem célszerű kiírni! Főleg, hogy nem is túl szép.
  Az ilyen jellegű hibákat amúgy nagyon faszán le lehet kezelni kivételkezeléssel, ha valami kritikus jellegű hibád van, azonnal dobsz egy kivételt, hogy ne is futkorásszon tovább a kód, nem is kell bonyolult és ronda if-else blokkokat csinálni, egyszerűen valahol elkapod a kivételt, megfelelő módon kezeled, és kész.
  Hogy érted azt, hogy nem célszerű kiírni? Beszélgettünk már a kivétel kezelésről, de amikor a kód készült azt sem tudtam mi az a try() meg catch() blokk.

  if (!empty($valogat['url']) and isset($valogat['url']))
  Ennek a feltételvizsgálatnak így nem sok értelme van, itt elég lenne a !empty() részt vizsgálni, nyilván ha nem üres a változó (nem is NULL, nem is üres string, stb.), akkor be van állítva, tehát a második feltétel már felesleges. Sőt, itt előbb célszerű lenne inkább megvizsgálni, hogy van-e kapott eredményhalmaz, vagy sem, ha már úgyis lekérdezed a kapott sorok számát a mysql_num_rows() függvénnyel.
  Valami oka volt, hogy így írtam. De már nem tudom, hogy mi volt az.

  Na, a többi részéhez most nem volt türelmem.
  Kössz!

  "Amelyik táblában az url-ek vannak cachelve van."
  Hogyan bírod rá így külön az adatbázist, hogy cache-elje? Tudtommal default cache-eli, indexeléssel lehet esetleg segíteni a lekérdezés gyorsaságát.
  Az indexelésre gondoltam, csak a szakzsargon nem mindig jut eszembe.

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