mit if() eine variable prüfen und ausgabe machen

[|Coding]

hi,

ich hab mir folgende funktion geschrieben:

PHP-Code:

<?
function out()
{
    global $var;
    if($a == "hallo")
    {
        $var = "hallo";
    }
    if($a == "tag")
    {
        $var = "tag";
    }
    if($a == "hi")
    {
        $var = "hi";
    }
}
?>

...und möchte dann $var, wenn es mittels formular gesendet wurde, mit echo ausgeben lassen.

PHP-Code:

<? 
include("func.php"); 
if(isset($sub)) 
{ 
    out(); 
    echo $_GET['var'];
} 
else 
{ 
    echo "<form action="".$PHP_SELF."" method="post">

        <select name="a">

            <option value="hallo">Hallo</option>

            <option value="tag">Tag</option>

            <option value="hi">Hi</option>

        </select>

        <input type="submit" name="sub" value="los...">

    </form>"; 
} 
?>

...ich habe es auch schon mit $_GET['var'] versucht und in der function habe ich die variable auch auf global gestezt, aber nichts geht, die variable bleibt immer leer. weiß wer rat?

thx schon mal.

[Progman]

Wenn du die Daten mit method=Post sendest dann sind die Daten in $_POST und nicht in $_GET ;)

[|Coding]

hmm,

upsss, ist wohl eine anfangerfalle :-)
aber jetzt weiß ich es ja.

thx Progman

[|Coding]

hmm,

ich könnte mich in den hintern beißen, das geht immer noch nicht...

ich habe das formular mit method="post" versendet und wollte die variable mit $_POST['var'] auslesen, geht nicht :-(

[Nev]

Hi @ll

Erst mal zu dem Fehler in dem Script:

in der Funktion ist die Variable a nicht globaldefiniert.
Deswegen ist sie leer.

Alle Variablen die in einer Funktion von aussen kommen , müssen als global-definiert werden.

nun noch was dazu.

Das Script funkt zwar, aber geschrieben ist es schlecht.
Nachteile:
es werden 2 globale Variablen definiert, die nicht nötig sind.
die If-abfragen sind unnötig, dauert zulange, wenn noch einige Werte abgefragt werden.

So wäre es schöner:

PHP-Code:

<?
function out($out = "")
{
    switch ($out)
    {
        case "hallo":
        return "hallo";
        break;
        case "tag":
        return "tag";
        break;
        case "hi":
        return "hi";
        break;
    default:
            return;
            break;
    }
}

//include("func.php"); 
if($sub) 
{ 
    echo out($a); 
} 
else 
{
    echo "<form action='".$PHP_SELF."' method='post'>
";
        echo "<select name='a'>
";
            echo "<option value='hallo'>Hallo</option>
";
            echo "<option value='tag'>Tag</option>
";
            echo "<option value='hi'>Hi</option>
";
        echo "</select>
";
        echo "<input type='submit' name='sub' value='los...'>
";
    echo "</form>";
} 
?>

Noch was:
if ($sub) bingt genau das selbe wie if (isset($sub))

Warum schreibt ihr immer " ??
Jeder Browser erkennt auch ', ist schneller ;-)

[|Coding]

@ Nev,

ich schreibe meinen quellcode so, weil ich meine seiten auf XHTML umstelle/umgestellt habe,da ja, html mit version 4.1 in rente geschickt wurde.

das heißt es wird kein html 5.0 mehr geben :-(

in XHTML müssen alle werte in anführungszeichen (") stehen.

[Progman]

if($var) ist nicht das selbe wie if(isset($var)). if($var) ist gültig wenn $var nicht 0 oder ein String länger 0 ist.
if(isset($var)) ist wahr, wenn die Variable $var definiert ist. Dies ist wichtig wenn man mit error_reporting(E_ALL); arbeitet

ps.: ist echo $a; nicht einfacher als echo out($a); ?

[Nev]

@Progman

in diesem Beispiel wäre es egal ob du isset verwendest.

Aber ansonsten gebe ich dir recht.

zu dem echo:

es ist besser mit echo out($a); zu machen;

So sparst du dir eine [/b]globale-Variable[/b]

wenn du den Wert weiter verwenden willst kannst du es natürlcih auch so machen:

PHP-Code:

  $a = out($a);
  echo $a; 


Aber generell sollte man so wenig globale Variable wie möglich verwenden.

ersten wird es unsicher
zweitens kann es zu ungewollten Manipulationen der Variablen inerhalb des Scripts kommen

[|Coding]

@ Progman,

einfacher? in wie fern einfacher?

[Progman]

Die funktionen Liefern den übergebenen Parameter zurück, was soll das denn bringen? Da brauche ich doch keine Funktion, da gebe ich die Var einfach aus.