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

• #5539 Sk8erPeter nagyúr D@ni88 #5538

  Új Válasz 2010-10-14 01:26:02 #5539

  Ú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 D@ni88 #5538 üzenetére

  Nem is nagyon értem, mit akarsz helyenként a kódodban... Pl. mit szeretnél azzal a mysql_error(); sorral? Az nem fog neked semmit kiírni... Akkor már az előző sor után tegyél egy or echo mysql_error(); részt, vagy így:
  if (mysql_errno()) {
  echo 'Hiba a lekérésnél: '.mysql_error();
  }

  Bár ezt inkább logolni kéne, nem a felhasználónak mutatni a konkrét hibaüzenetet.

  Az ilyeneket felejtsd el:
  $kiir .=
  "\n\t\t\t<td class=\"nev\">".$sor["fnev"]."</td>".
  "<td class=\"Fnev\">".$sor["Vnev"]." ".$sor["knev"]." </td>".
  "<td class=\"tel\">".$sor["telefonszam"]."</td>".
  "<td class=\"email\">".$sor["email"]."</td>".
  "<td class=\"cim\">".$sor["cim"]."</td>".
  "<td class=\"tel\">".$sor["ellenorzott"]."</td>";

  valami kegyetlenül átláthatatlan, helyette akkor már:
  $kiir .= '
  <td class="nev">'.$sor['fnev'].'</td>
  <td class="Fnev">'.$sor['Vnev'].' '.$sor['knev'].'</td>
  <td class="tel">'.$sor['telefonszam'].'</td>
  <td class="email">'.$sor['email'].'</td>
  <td class="cim">'.$sor['cim'].'</td>
  <td class="tel">'.$sor['ellenorzott'].'</td>';

  Ez már egy pár fokkal jobb.

  Mellesleg tök feleslegesen gyűjtöd a $kiir stringbe ezeket a sorokat, ha utána egyből ki is íratod.
  Legyen első az adatbázis-lekérdezés, ha az nem ad vissza hibát, akkor mehet egyből az echo-zás.

  Az adatok kiírásánál nagyon helytelen a táblázatod, a <form> nyitótag előtt lezárod a korábbi sort, és nem is nyitsz újat, még be kéne raknod egy <tr> nyitótagot...
  Ja, meg ezek szerint minden egyes felhasználónál akarsz egy külön submit gombot, hogy mindegyiknél el tudd küldeni, ellenőrizte-e már a júzer, akkor a <form> nyitótag tök rossz helyen van, a while cikluson belül kellene lennie, hiszen így minden egyes felhasználóhoz tartozik egy-egy form.
  Tehát töröld ki azt a <form> sort a while ciklus elől, és legyen a while cikluson belül (!) egy <tr>, majd a </tr> a while végén, és a sorokon belül oldd meg, hogy legyen a többi adat a submit gombbal együtt... Igazából szabályosan jelen esetben sztem táblázatba ágyazott táblázattal lehetne (persze egyszerűbben is meg lehet oldani, de most arról beszélek, ahogy a Te kódod kinéz).

  Mindenesetre a lényeg, hogy minden egyes ellenőrizendő felhasználóhoz külön form tartozzon.
  Kemény a kódod, belezöldülök, mire átlátom...

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