Hirdetés

  2. Fórumok
  3. Szoftverfejlesztés
  4. PHP programozás
  5. (kiemelt téma)

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

  • #8913 Sk8erPeter nagyúr cAby #8912
    Új Válasz #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.

    [ Szerkesztve ]

    Sk8erPeter

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