Switch funktioniert nicht ?!

ramon1611 · 3. Februar 2013

Hallo,
erstmal habe ich folgenden Code:

PHP

function validateUserData($type, $value) {
    $validData = TRUE;
    try {
        switch ($type) {
            case ('ip' || 'email' || 'username'):
                $answer = file_get_contents('http://www.stopforumspam.com/api?'.$type.'='.$value);
                if (preg_match('/\<response success="true"\>/i', $answer) === 1)
                    if (preg_match('/\<appears\>yes\<\/appears\>/i', $answer) === 1) $validData = FALSE;
                break;
            case ('ip,email' || 'ip,username' || 'email,ip' || 'email,username' || 'username,ip' || 'username,email'):
                $type = explode($type, ',');
                $value = explode($value, ',');




                $answer1 = file_get_contents('http://www.stopforumspam.com/api?'.$type[0].'='.$value[0]);
                $answer2 = file_get_contents('http://www.stopforumspam.com/api?'.$type[1].'='.$value[1]);




                if (preg_match('/\<response success="true"\>/i', $answer1) === 1 || preg_match('/\<response success="true"\>/i', $answer2) === 1)
                    if (preg_match('/\<appears\>yes\<\/appears\>/i', $answer1) === 1 || preg_match('/\<appears\>yes\<\/appears\>/i', $answer2) === 1) $validData = FALSE;
                break;
            case ('ip,email,username' || 'ip,username,email' || 'email,ip,username' || 'email,username,ip' || 'username,ip,email' || 'username,email,ip'):
                $type = explode($type, ',');
                $value = explode($value, ',');




                $answer1 = file_get_contents('http://www.stopforumspam.com/api?'.$type[0].'='.$value[0]);
                $answer2 = file_get_contents('http://www.stopforumspam.com/api?'.$type[1].'='.$value[1]);
                $answer3 = file_get_contents('http://www.stopforumspam.com/api?'.$type[2].'='.$value[2]);




                if (preg_match('/\<response success="true"\>/i', $answer1) === 1 || preg_match('/\<response success="true"\>/i', $answer2) === 1 || preg_match('/\<response success="true"\>/i', $answer3) === 1)
                    if (preg_match('/\<appears\>yes\<\/appears\>/i', $answer1) === 1 || preg_match('/\<appears\>yes\<\/appears\>/i', $answer2) === 1 || preg_match('/\<appears\>yes\<\/appears\>/i', $answer3) === 1) $validData = FALSE;
                break;
            default:
                $validData = 'ERROR('.$GLOBALS['errorCodes'][5].')';
                var_dump(1);
                break;
        }
    } catch (Exception $ex) { $validData = 'ERROR('.$GLOBALS['errorCodes'][6].')'; }
    return $validData;
}

Alles anzeigen

Wenn ich nun einen $type von z.B. "25" angebe, bleibt $validData bzw. die Funktion TRUE, sie sollte aber eigentlich 'ERROR('.$GLOBALS['errorCodes'][5].')' werden (siehe 'default:')
kann mir vllt. irgendjemand sagen, warum das so ist

Danke schonmal für jegliche Hilfe

EDIT: 'ERROR('.$GLOBALS['errorCodes'][5].')' soll folgendes ergeben: "ERROR(wrongArgumentOfFunction)"

cottton · 4. Februar 2013

Das einzige, was ich sehe ist, dass das break im default unnötig ist.
Vlt kommt daher der Fehler?
Hab allerdings noch keine Erfahrung mit switch.

EDIT:
@ramon1611

Die cases beim switch sind verdattelt.
Hab gestern noch was gelesen, dass man vorsichtig sein muss.
Was genau da falsch läuft, weiss ich momentan nicht. Aber guck dir mal das bsp an:

PHP

$type = 25;
$value = 'NA';
var_dump(validateUserData($type, $value));
function validateUserData($type, $value) {
    $validData = TRUE;
    try {
        switch ($type) {
            case ('test' || 'test2'):
                echo "1<br />";
                break;
            case 25:
                echo "2<br />";
                break;
            case ('ip' || 'email' || 'username'):
                echo "3<br />";
                break;
            case ('ip,email' || 'ip,username' || 'email,ip' || 'email,username' || 'username,ip' || 'username,email'):
                echo "4<br />";
                break;
            case ('ip,email,username' || 'ip,username,email' || 'email,ip,username' || 'email,username,ip' || 'username,ip,email' || 'username,email,ip'):
                echo "5<br />";
                break;
            default:
                echo "6<br />";
                break;
        }
    } catch (Exception $ex) { $validData = 'ERROR('.$GLOBALS['errorCodes'][6].')'; }
    return $validData;
}

Alles anzeigen

Es geht NICHT in den 2ten case. Die Bedingung funktioniert also nicht.

Wenn du aber nach true switchst:

PHP

$type = 23;
// $type = '....';
$value = 'NA';
var_dump(validateUserData($type, $value));
function validateUserData($type, $value) {
    $validData = TRUE;
    try{
        switch(true){
            case $type === 25:
                echo "1<br />";
                break;
            case ($type === 23 || $type === '....'):
                echo "2<br />";
                break;
            default:
            $validData = false;
        }
    }catch (Exception $ex) { $validData = 'ERROR('.$GLOBALS['errorCodes'][6].')'; }
    return $validData;
}

Alles anzeigen

So funktioniert es.
Ich persönlich würde einfach if/elseif nutzen

ramon1611 · 4. Februar 2013

ne, hab ich grade ausprobiert
daher kommts auf jeden fall nicht

EDIT:
okay
ich versuchs mal..

EDIT2:
Super, funktioniert wirklich
danke dir

cottton · 4. Februar 2013

Habs.
Im case kann man keine usw nutzen.
siehe:
http://www.php.net/manual/de/control-structures.switch.php

Zitat

working a bit around with it I found out that it is not possible to
compare the variable with two different values in one step like this
(system running a w2k server, apache2.0.43 & php430):

<?php
switch ($myvar) {
case ("foo" "bar"): //do something
break;
case ("other"): //do another thing
break;
default:
}
?>

rather use:

<?php
switch ($myvar) {
case ("foo"):
case ("bar"): //do something
break;
case ("other"): //do another thing
break;
default:
}
?>

Alles anzeigen

Es funktionert aber mit der switch(true) Methode.

Switch funktioniert nicht ?!

Jetzt mitmachen!

Teilen