[php] $_get geeft index fout

[sjongenelen]

Beste gotters

ik heb een mooi pakket geinstalleerd, USB Webserver - dit is een mooi pakket met
PHP 5, Apache 2 en MySQL..
op dit pakket heb ik een loginsysteem dat gegevens uit de database haalt in elkaar geflanst.

hij geeft alleen een error als ik de pagina laadt:

Notice: Undefined index: action2 in %location%\Root\aanmelden.php on line 9

%location% heb ik zelf verzonnen:P

ik kijk dus in die Line:

PHP:

if ($_GET['action2'] == 'signup') {

include("config.php");

De 'action2' wordt verwezen in:

PHP:

echo " <form action=action2=signup method=POST>
    <input type=hidden name=submit value=1>
    Username: <input type=text name=username maxlength=10>
    <p>
    Password: <input type=password name=password maxlength=15>
    <p>
    Email: <input type=text name=email>
    <br>
    Leeftijd: <input type=text name=leeftijd maxlength=2>
    <input type=submit value=Signup!>
    <input type=reset value=Clear>
    </form> ";[/td]

die staat onderaan die pagina. dus als de gebruiker niet ingelogd is Echo dan de Form om ze dat wel te laten doen...

- ik heb action2 al hernoemd (het was eerst action), omdat ik dacht dat dit misschien problemen kon geven met dubbele waarden ofzo..

- ik heb de register_globals aan gezet - misschien dat hij waarden niet kon laden oid.

- hij wérkt wel - hij geeft alleen die error

- ik heb hem niét op een andere server geprobeerd - ik heb liever een oplossing

als jullie de hele code willen zien dan post ik die later wel - misschien is het iets heel makkelijks wat ik fout heb gedaan

[ Voor 18% gewijzigd door sjongenelen op 13-04-2006 11:50 ]

[Janoz]

Dat stukje html is wel de meest brakke die ik ooit heb gezien. Probeer eens wat meer quotes om je attributen te gebruiken. Daarnaast kun je onderdelen die je niet door je gebruiker wilt laten invullen, maar wel met je formulier mee wilt zenden beter in een input met type="hidden" stoppen.

[NaN]

Sja, ik weet niet wat deze regel voorstelt:

<form action=action2=signup method=POST>

Komt mij vreemd over...

[OnTracK]

wat is dit?

action=action2=signup

Je "action" in de form-tag moet naar een URL verwijzen waar je form wordt verwerkt. Daarnaast wil je dan waarschijnlijk nog een hidden input hebben waarin staat dat je "actie" een signup is.

[Michali]

Gebruik daar de submit voor:

HTML:

<input type="submit" value="Signup!" name="action2">

[ Voor 47% gewijzigd door Michali op 13-04-2006 11:30 ]

[Snake]

de eerste keer dat je de pagina laad zijn er geen GET variablen, je url is gewoon index.php

hij checkt of action2 in de url voorkomt,maar dat is niet, je zou dus het volgende moeten doen:

code:

 if (isset($_GET['action2']) && $_GET['action2']=="signup")

[sjongenelen]

Janoz schreef op donderdag 13 april 2006 @ 11:28:
Dat stukje html is wel de meest brakke die ik ooit heb gezien. Probeer eens wat meer quotes om je attributen te gebruiken. Daarnaast kun je onderdelen die je niet door je gebruiker wilt laten invullen, maar wel met je formulier mee wilt zenden beter in een input met type="hidden" stoppen.

het is mn eerste script, ik vind het nog wel meevallen - ik heb nul ervaring met php of formulieren..
maar ik zoek alleen een oplossing voor het php gedeelte - de verstuurformulieren zijn bijzaak -
het gaat om het php gedeelte

[NaN]

OnTracK schreef op donderdag 13 april 2006 @ 11:29:
wat is dit?
[...]


Je "action" in de form-tag moet naar een URL verwijzen waar je form wordt verwerkt. Daarnaast wil je dan waarschijnlijk nog een hidden input hebben waarin staat dat je "actie" een signup is.

Juist, dat lijkt me logischer.

[Cyphax]

Sowieso staat er als method post in het form en vervolgens probeer je die variabelen uit get te halen. Die komen niet in $_GET terecht maar in $_POST.

[Sybr_E-N]

Om het weer even over die "error" te hebben. Het is geen error, wat dat staat er niet! Het is een notice, zeg maar waarschuwing. Waarom dan? Omdat je op regel 9, "if ($_GET['action2'] == 'signup') {", een index in de get-array probeert te benaderen niet van te voren niet is vastgesteld.

Met behulp van de functie isset() kun je nagaan of die variabele uberhaupt wel voorkomt in de get-array.

Maar waarom kun je hier zelf niet uitkomen? Het is namelijk een nogal voor de hand liggen "fout."

[ Voor 7% gewijzigd door Sybr_E-N op 13-04-2006 11:33 ]

[sjongenelen]

de verwerking gebeurd in dezelfde pagina....

met:

code:

 if (isset($_GET['action2']) && $_GET['action2']=="signup")

is de error wel weg maar word er niet weggeschreven in de database


dit is de hele code:

PHP:

<?php

echo "<font size=4>Signup</font>
    <br>
    <br>";

if ($_GET['action2'] == 'signup')
\\dit is de plek

 {

include("config.php");

if ($_POST['username'] == '')
{
echo "<font color=#ff0000>je hebt geen username ingevuld</font>";
}
else {
    if ($_POST['email'] == '')
        {
        echo "<font color=#ff0000>Je hebt geen e-mailadres ingevuld</font>";
        }
            else {
        if ($_POST['password'] == '')
            {
            echo "<font color=#ff0000>Je hebt geen password ingevuld</font>";
            }
                else {
                  if ($_POST['leeftijd'] == '')
                    {
                    echo "<font color=#ff0000>Je hebt geen leeftijd ingevuld! </font>";
                    }

$pw = md5(password);

mysql_query("INSERT INTO members (username,email,password,leeftijd) VALUES ('$_POST[username]','$_POST[email]','$pw','$_POST[leeftijd]')")or die(mysql_error());


echo "je bent aangemeld";

                    }
                }
    }
}

else {

\\en hier is de plek
echo " <form action=action2=signup method=POST>
    <input type=hidden name=submit value=1>
    Username: <input type=text name=username maxlength=10>
    <p>
    Password: <input type=password name=password maxlength=15>
    <p>
    Email: <input type=text name=email>
    <br>
    Leeftijd: <input type=text name=leeftijd maxlength=2>
    <input type=submit value=Signup!>
    <input type=reset value=Clear>
    </form> ";
}
?>

in config.php staan het wachtwoord en pass voor welke database..

hoe maak ik een mooie tabel waarin de opmaak bewaard blijft?

[ Voor 51% gewijzigd door sjongenelen op 13-04-2006 11:49 ]

[NaN]

$pw = md5(password);

Moet naar mijn idee $pw = md5($password);

zijn

[Spider.007]

Ondanks dat dit erg basic is past het beter in Programming; ik verplaats het daar dus ook naartoe; het is even ter beoordeling van de moderators aldaar of je inzet als voldoende wordt ingeschat; ook voor je eerste script kun je je namelijk gewoon inlezen

SA > P

[sjongenelen]

Spider.007 schreef op donderdag 13 april 2006 @ 11:42:
Ondanks dat dit erg basic is past het beter in Programming; ik verplaats het daar dus ook naartoe; het is even ter beoordeling van de moderators aldaar of je inzet als voldoende wordt ingeschat; ook voor je eerste script kun je je namelijk gewoon inlezen

SA > P

ik heb ook op tweakers op die index fout gezocht, maar daar was de oplossing een <? en ?> tussen de codes te zetten. Dit werkt bij mij niet hoor


enneh: het password gedeelte werkt ook - encrypted ^^
alleen nog één (simpel ) dingetje

[ Voor 16% gewijzigd door sjongenelen op 13-04-2006 11:49 ]

[Snake]

PHP:

//hier stond schandalige code

onder voorbehoud natuurlijk

offtopic:
en nu pizza maken

[ Voor 234% gewijzigd door Snake op 13-04-2006 12:09 . Reden: doh ]

[sjongenelen]

eh, geen foutmelding - dat klopt! maar ook niets te zien in de database...

offtopic:
even wachten met de pizza

[Janoz]

TheNymf schreef op donderdag 13 april 2006 @ 11:30:
[...]


het is mn eerste script, ik vind het nog wel meevallen

Het maakt weinig uit wat jij er van vindt, je browser/computer houd absoluut geen rekening met het feit of een ontwikkelaar net begonnen is of een gevorderde is.

- ik heb nul ervaring met php of formulieren..
maar ik zoek alleen een oplossing voor het php gedeelte - de verstuurformulieren zijn bijzaak -
het gaat om het php gedeelte

Maar de fout zit grotendeels in je html code.

[Snake]

TheNymf schreef op donderdag 13 april 2006 @ 11:55:
eh, geen foutmelding - dat klopt! maar ook niets te zien in de database...

offtopic:
even wachten met de pizza

offtopic:
Ie zit in de oven

Check script nu eens, ik had GET en POST omgedraaid

PHP:

<?php
echo "<font size=4>Signup</font><br><br>";

if ($_SERVER['REQUEST_METHOD']=="POST") {

    include("config.php");
    
    if ($_POST['username'] == '') {
        echo "<font color=#ff0000>je hebt geen username ingevuld</font>";
    } elseif ($_POST['email'] == '') {
        echo "<font color=#ff0000>Je hebt geen e-mailadres ingevuld</font>";
    } elseif ($_POST['password'] == '') {
        echo "<font color=#ff0000>Je hebt geen password ingevuld</font>";
    } elseif ($_POST['leeftijd'] == '') {
        echo "<font color=#ff0000>Je hebt geen leeftijd ingevuld! </font>";
    }
    
    $pw = md5($_POST['password']);
    
    mysql_query("INSERT INTO members (username,email,password,leeftijd) VALUES ('".$_POST['username']."','".$_POST['email']."','".$pw."','".$_POST['leeftijd']."')") or die(mysql_error());


    echo "je bent aangemeld";
} else {
    ?>
    <form method="post" action="<?php echo $_SERVER['PHP_SELF']; ?>">
        Username: <input type="text" name="username" maxlength="10"><br>
        Password: <input type="password" name="password" maxlength="15"><br>
        Email: <input type="text" name="email"><br>
        Leeftijd: <input type="text" name="leeftijd" maxlength="2"><br>
        <input type="submit" value="Signup!"><br>
        <input type="reset" value="Clear"><br>
    </form>
<?php
}
?>

getest, en het werkt hier

offtopic:
yay pizza klaar

[ Voor 47% gewijzigd door Snake op 13-04-2006 12:10 ]

[sjongenelen]

het werkt idd -

ik weet alleen niet wat het doet...

<?php echo $_SERVER['PHP_SELF']; ?>

maar dat ga ik wel even lezen...

bedankt!

[Snake]

TheNymf schreef op donderdag 13 april 2006 @ 12:40:
het werkt idd -

ik weet alleen niet wat het doet...


[...]


maar dat ga ik wel even lezen...

bedankt!

als je script index.php heet, dan wordt $_SERVER['PHP_SELF'] index.php
heet het aanmelden.php dan wordt het aanmelden.php

enz...

http://php.net

[NMe]

Goed, dit topic is samen te vatten in één zinnetje: "Koop een boek, lees wat tutorials door en spendeer veel meer tijd op PHP.net." Alle vragen die de revue passeren in dit topic zouden met een beetje moeite van jouw kant ook wel opgelost kunnen worden. Beginner zijn is geen "excuus" om geen boeken te hoeven lezen.