Sætte en variabel inde i en funktion?

fordi du gør variablen error_billedid global i funktionen - dvs. den peger på den variabel der findes uden for funktionen - hvis den altså findes!

som du er inde på, skal de gøres globale!

derfor, prøv at sæt global $error_billedeid til sidst (man kan jo ikke lave en ikke eksisterende variabel global)

Morten

Jeg forstår ikke jeres (tipsen, dea_pg's) indlæg.

Så vidt jeg kan se burde følgende:

check_billed();
echo $error_billedid;

Da udskrive hans fejlstreng (forudsat at if betingelserne i funktionen er opfyldt).

http://925.dk/test.phtml

Hmm, jeg fatter minus her

Jeg tror måske fejlen er et andet sted :(

Jeg tager delvist fejl - ovenstående kode burde virke finde fra php v. 4.2.x, mens der tidligere har været en fejl som bevirkede at variablen skulle initialiseres globalt inden lokal brug i funktioner! Dvs. hvis du bruger en relativt gammel version af PHP er det muligt at du er nødt til at initialisere variablen som du selv har fundet ud af. Se evt: http://dk.php.net/manual/en/language.variables.scope.php hvor der er nogle gode bruger-kommentarer.

Det tror jeg også.

Rettelse: "...burde virke fint fra..."

Doooooh, jeg havde lige glemt global i den funktion der skulle udskrive den. Og når jeg testede som det eksempel mercur8 lavede, så kaldte jeg ikke funktionen først :(

Men tak for hjælpen anyway :) Smidder i et svar?

Jeps.

Iøvrigt hvis du har lyst til at prøve at undgå de globale variable, kan du prøve en konstruktion ala:

<?
function check_billed(&$error_string) {
    # Checker om der skal være billed, og om det er valgt et
    if (isset($_POST['billed']) and $_POST['billed'] == "j" and (!isset($_POST['billed_id']) or !is_numeric($_POST['billed_id']))) {
        # Hvis ikke, sætter fejlindikator og fejlmeddelelse
        $error_string = "<tr><td colspan=\"3\"><div class=\"menu-error\">Husk at vælge et billed</div></td></tr>";
        return FALSE;
    }
    return TRUE;
}

$error_string = NULL;
echo !check_billed($error_string) ? $error_string : "alt ok";
?>

Overvej endvidere om du har brug for både en fejl streng og et fejl flag (som du gør nu - sætter 2 variable i tilfælde af en fejl). Det er muligt du har.

Læs iøvrigt: http://dk.php.net/manual/en/language.references.pass.php

Det er som oftest en god idé at undgå globale variable, idet de blandt andet "forurener" dit variablescope. Altså der kan bliver overlap mellem variabel navne osv.

Derudover er det noget rod med alt den html-kode du har blandet ind i funktionen - det gør ikke funktionen særlig fleksibel.

mercur8 > Det ser umiddelbart smart ud, det vil jeg lige kigge nærmere på.

tipsen > Jeg ved det. Jeg burde have et kursus i programmrings disciplin :(
Jeg er ikke stjernen til at sætte op, samle i filer osv.

lukker