php5: throw exception

[php_rookie]

hi!

ist es eigentlich normal, dass zusätzlich zum werfen einer exception ein E_WARNING geworfen wird?

schematisches beispiel:

if function($param) {
// statement
} else {
throw SomeException
}

function kann beispielsweise durch copy() u.ä. ersetzt werden.

[penizillin]

was sagt denn das warning?

[php_rookie]

was sagt denn das warning?

na, dass die datei nicht gefunden wurde, wie im testfall beabsichtigt. die warning wird geworfen obwohl eine eigene exception geworfen wird. ein beispiel:

<?php
try {
    if (!copy("foo.bar", "/")) {
        throw new Exception("description");
    }
} catch (Exception $e) {
    echo "exception caught!";
}
?>

liefert:

Warning: copy(foo.bar) [function.copy]: failed to open stream: No such file or directory in [...] on line 3
exception caught!

[penizillin]

warum kein file_exists davor?

[php_rookie]

weil es eine fehlermeldung auslösen SOLL. das nämlich ist mein problem. warum wird eine E_WARNING ausgelöst wo die meldung doch von einer exception abgefangen werden sollte?

[penizillin]

hab zwar wenig erfahrung mit php5, aber ich meine der block try {} wird zunächst ganz normal ausgeführt. und wenn darin etwas ne warning schmeißt, dann wird's so sein. deine exception ändert nichts an der "normalen" ausführung des ersten blocks, afaik.

wenn du so ein warning nicht anders behandeln willst, kannst du ein user_warning (oder wie das heißt) anlegen.

copy ist nun mal ein befehl, der nur eine bestehende datei kopiert. file_exists fänd ich eine schönere lösung, als das warning mit einem @-operator zu schlucken.

ändere den block doch so:

try {
  if(file_exists("bla")){
    copy(/*bla*/);
  } else
     throw exception //etc...
}
catch ... //etc... 

[ghostgambler]

Original geschrieben von penizillin
hab zwar wenig erfahrung mit php5, aber ich meine der block try {} wird zunächst ganz normal ausgeführt. und wenn darin etwas ne warning schmeißt, dann wird's so sein. deine exception ändert nichts an der "normalen" ausführung des ersten blocks, afaik.

Das ist so richtig!

Du solltest wenn zuerst ein file-exists machen und bei dessen Fehlschlag die Fehlermeldung ausgeben. Try/Catch ist nicht dazu da um unsaubere Programmierung auszubessern, sondern um die Fehlerbehandlung zu verbessern!