If-sætning

Prøv med:
if ($felt[3] !="ingen" || $felt[2]!="hidden"){


!? :0)

//Mathias

Nej, jeg mener:
if ($felt[3] !="ingen" || $felt[2] !="hidden"){

:0)

Nej nej nej :0(
Jeg er ikke rigtig vågen he he :0)
if ($felt[3] != "ingen" || $felt[2] != "hidden"){

det burde hjælpe :0)

virker stadig ikke - se selv på www.fodboldenslegestue.dk/anbefalside.php og kig kildekoden. Her har jeg skrevet felt[3] og felt[2] ud, og som du kan se kommer felterne også frem, hvor de er hhv. ingen og hidden, og det skulle if-sætningen jo gerne afhjælpe. Nogen forklaring?

Altså selve html-kildekoden - og ikke kildekoden nederst i dokumentet! :)

Du har vel chekket at dit array er korrekt. Prøv at teste det med at printe fx felt[2] eller felt[3] ud, og se hvad de egentlig indeholder.

Ja det har jeg. Det handler min besked 12:16:10 om.

Nu skal du vel bruge AND - ellers må et af felterne godt være "intet" eller "hidden"

if(($felt[3] !="ingen")||($felt[2] !="hidden")){

nej undskyld, det skal være OR ;-)

Kunne du ikke sige hvor du får dit array fra? Umiddelbart kan jeg ikke se det ud fra den url du har givet.

Mit array er det der står nederst på siden på det opgivne link!

I får lige hele funktionen!

function skriv_formular($formular) {

    // udskriv validering
    print "<script language=\"JavaScript\" type=\"text/javascript\">\n";
    print "<!--\n";
    print "function Checkform(thisform) {\n";
    print "      // Startværdier\n";
    print "  strError = 'Der opstod følgende fejl:\\n\\n';\n";
    print "  intError = 0;\n";

    foreach($formular AS $felt) {

        if ($felt[3] != "ingen" OR $felt[2] != "hidden"){
            print "  // felt[3] ".$felt[3]."\n";
            print "  // felt[2] ".$felt[2]."\n";       
       
            print "  // Tjekker ".$felt[0]."\n";
            print "  if (thisform.".$felt[1].".value == \"\") {\n";
            print "      strError = strError + '-".$felt[0]." mangler\\n';\n";
            print "      intError = 1;\n";
            print "      thisform.".$felt[1].".focus();\n";
            print "  }\n";
        }
   
       
        if ($felt[3] == "email") {
            print "  // Check for @ i email\n";
            print "  if (thisform.".$felt[1].".value.indexOf(\"@\") == -1 && thisform.".$felt[1].".value != \"\") {\n";
            print "      strError = strError + '-Mangler @ i emailadresse\\n';\n";
            print "      intError = 1;\n";
            print "      thisform.".$felt[1].".focus();\n";
            print "  }\n";   
        }

    }

    print "  if (intError == 1) {\n";
    print "      alert(strError);\n";
    print "      return false;\n";
    print "  }\n";
    print "}\n";
    print "//-->\n";
    print "</script>\n";

// udskriv formularen
   
    print '<form method="post" action="?" name="thisform" id="thisform" onSubmit="return Checkform(this);">';

   
    print "<input type=\"hidden\" name=\"action\" value=\"submit\" />";
   
    foreach($formular AS $felt) {
        if ($felt[2]!="hidden") {
            if ($felt[2] == "textarea"){
                print $felt[0].'<textarea name="'.$felt[1].'" onFocus="'.$felt[1].'"></textarea><br />';
            }
            else {
                  print $felt[0].' <input type="'.$felt[2].'" name="'.$felt[1].'" onFocus="'.$felt[1].'" /><br />';
            }
        }
        elseif ($felt[2] !="hidden") {
                print '<input type="'.$felt[2].'" name="'.$felt[1].'" value="'.$felt[0].'" />';
        }
    }

    print '<input type="submit" value="Send" />';   
    print '</form>';
}

Skal måske lige sige, at jeg har haft lavet det om til || med det samme resultat som nu!

Det fordi du bruger foreach forkert. Når du siger:
foreach ($formular as $felt)
$felt er så den pågældende value for $formular, som den kører igennem. Det vil samtidig sige at $felt[0] ikke er giver dig det første element i $formular.

Prøv fx at køre dette meget enkle script, så forstår du nok hvad jeg mener:
<?php
$arr = array("Hej","dette","er","et","array");
foreach ($arr as $felt)
{
    echo $felt[0];
        //Output: Hdeea
}
?>

Du kunne i stedet gemme hver input-value i en variabel sådan at mail blev gemt i $mail, og afsender blev gemt i $afsender osv. Sådan at hvis du har et text-input kaldet "navn" henter du den ved at sige
$navn = $_POST['navn'];

Men du må sige til hvis du er uenig med mig.

Medmindre selvfølgelig du bruger et array med indbyggede arrays:
$arr = array(array("Værdi","Værdi"),array("Værdi2","Værdi3"));
så bliver $felt[0] ganske rigtigt til Værdi. Hvis jeg husker rigtig var det da vist også sådan du gjorde :-)

Så virker det. Ved ikke helt, hvad jeg har lavet om, men I har nok alle hjulpet. Skovenborg og mathiash smid lige et svar.

Okay :-)

Hvad skrev du i stedet for?

skovenborg > Er faktisk ikke helt klar over, hvordan det kom til at virke. Der er gået cirka en uge siden jeg arbejdede med det sidst, og pludselig var den der bare?

Mærkeligt - men dit script så jo også meget korrekt ud.

Forresten er det ligemeget om du bruge OR eller || - PHP kan finde ud af begge. Til gengæld kan javascript kun bruge || (og &&) hvilket godt kan virke forvirrende hvis man bruger dem begge.

Ja, PHP er slet ikke så dum endda :) Gad vide, hvorfor de har implementeret begge løsninger?

Okay, der er forskel på dem, da de har forskellige forrang (precedence).
http://www.php.net/manual/en/language.operators.php#language.operators.precedence
hvor and har lavere rang end &&.
Hvad det helt præcis betyder ved jeg ikke, og i dit tilfælde er det egentlig også lige meget, men her er et eksempel på hvor de to giver forskelligt resultat:
$b = "Hej";
$c = "med dig";
hvor
$a = $b && $c; //resultat: 1
$a = $b and $c; //resultat: Hej

... men så vidt jeg har forstået er det ligesom i matematik hvor gange har højere rang en fx plus.

ok, herligt at blive klogere hele tiden *-)

... og (nu bliver jeg ved) i mit ovenstående tilfælde har = højere rang end and, derfor returner den kun Hej, hvorimod && har højere rang end =, og returnerer 1 (fordi de begge er true).

Tak for alle svarene. Nu kan jeg få det hele til at virke :)