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

• #8913 Sk8erPeter nagyúr cAby #8912

  Új Válasz 2012-03-28 12:13:27 #8913

  Ú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 cAby #8912 üzenetére

  "Index.php-ben is nyilván van ilyen ciklus"
  Szerintem ez a kódod alapján nem annyira nyilvánvaló.

  "Így működik ahogy akarom, felveszi a listába és onnan el lehet küldeni e-mail-ben."
  Honnan fogod tudni, MELYIKET kell felvenni a listába?
  Hadd "idézzek" a kódodból:

  if ( $_SESSION['fav_' . $row['id']] == 'false' )
  {
  echo "<form action='index.php' method='post'>
  <input type='submit' name='add' value='add' />
  </form>";
  }
  elseif ( $_SESSION['fav_' . $row['id']] == 'true' )
  {
  echo "<form action='index.php' method='post'>
  <input type='submit' name='del' value='del' />
  </form>";
  }

  Itt kiraksz egy formot, de magában a formban egy darab azonosítót nem rejtesz el (mondjuk egy hidden inputtal, vagy másképp), így nem tudom, honnan szeded, hogy mondjuk épp az 5-ös azonosítójút szeretném eltárolni a kedvencek közé.
  Egyébként itt az if-elseif vizsgálat nem túl ésszerű - inkább azt kéne vizsgálnod, hogy mondjuk true értéke van-e, ha igen, kirakod a törlőformot, KÜLÖNBEN (simán else) a hozzáadó formot.

  Amúgy ha mutatsz egy hasonló kódot, akkor nem ártana a teljeset, hogy el tudjuk dönteni, valamilyen kód indokolt-e egyáltalán.
  Az például nem túl szép megoldás, hogy az ismeretlen eredetű $sum_items mennyiségig végigmész az elemeken, és egyesével, cikluslépésenként kéred le adatbázisból a különböző azonosítójú elemeket.

  További tipp: ha azt szeretnéd, hogy egy form ugyanoda legyen elküldve, ahol ki is van írva, akkor érdemes az action attribútumot inkább üresen hagyni a formnál (action=""), ez egy valid megoldás, és könnyebben "költöztethető" - tehát ha az index.php helyett megváltoztatod a fájl nevét asdasdasd.php-re, akkor is működni fog.

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