#47576 [Opn->Fbk]: "continue i;" working wrong on foreach
ID: 47576 Updated by: j...@php.net Reported By: fischikowski at web dot de -Status: Open +Status: Feedback Bug Type: Scripting Engine problem Operating System: Windows Vista 32bit PHP Version: 5.2.9 New Comment: Works fine on windows, linux 32/64bit for me. Previous Comments: [2009-03-10 14:43:21] fischikowski at web dot de I don't think to use any. I am using PHP on an apache-webserver, if that makes a difference. Could also not reproduce this on linux with Version 5.2.6, had no higher versions on a linux system to test it. Will try that later. [2009-03-10 13:51:43] scott...@php.net I can't reproduce this on Linux or OS X Do you have any Zend Extensions enabled such as eAccelerator, APC, Zend Optimizer or anything like that? [2009-03-10 10:42:39] fischikowski at web dot de Except from the "" I already posted the shorter script. [2009-03-05 17:34:29] fischikowski at web dot de Description: The echo ""; affects the "return true;" below. As long as the echo is there everything works as expected, if you put "//" in front of the echo, the return will be called even if it should not. Reproduce code: --- function matchHostList($host, $list_file) { $host = explode(".", $host); //$list_contents = explode("\n", file_get_contents($list_file)); $list_contents = array("www.google.de"); foreach($list_contents as $list_host) { $list_host = explode(".", rtrim($list_host)); //if the list-host is more specific than the tested host we can't match if(count($host) < count($list_host)) continue; for($i = 1;$i<=count($list_host) && $i <= count($host);$i++) { if($list_host[count($list_host)-$i] != $host[count($host)-$i]) continue 2; } echo ""; //this is necessary to avoid wrong return return true; } return false; } echo matchHostList("false.google.de", "")?"true":"false"; Expected result: When running as shown above this returns false (because "continue 2;" continues the loop above the "return true;"), this is OK. If you remove the echo ""; above the "return true;" You would expect that nothing changes (because we only removed some null-output), but in fact the return value changes to true. Actual result: -- see Expected result -- Edit this bug report at http://bugs.php.net/?id=47576&edit=1
#47576 [Opn->Fbk]: "continue i;" working wrong on foreach
ID: 47576 Updated by: scott...@php.net Reported By: fischikowski at web dot de -Status: Open +Status: Feedback Bug Type: Scripting Engine problem Operating System: Windows Vista 32bit PHP Version: 5.2.9 New Comment: I can't reproduce this on Linux or OS X Do you have any Zend Extensions enabled such as eAccelerator, APC, Zend Optimizer or anything like that? Previous Comments: [2009-03-10 10:42:39] fischikowski at web dot de Except from the "" I already posted the shorter script. [2009-03-10 10:34:43] j...@php.net Thank you for this bug report. To properly diagnose the problem, we need a short but complete example script to be able to reproduce this bug ourselves. A proper reproducing script starts with , is max. 10-20 lines long and does not require any external resources such as databases, etc. If the script requires a database to demonstrate the issue, please make sure it creates all necessary tables, stored procedures etc. Please avoid embedding huge scripts into the report. [2009-03-06 08:30:09] fischikowski at web dot de I could shorten the test-code (the previous was my original working code): function test() { $array = array("test"); foreach($array as $element) { for($j=0;$j<5;$j++) { continue 2; } return true; } return false; } echo test()?"true":"false"; I expect, that this returns false, because "continue 2" should step to the next foreach-loop (and "return true" should not be executed). But it happens, that true is returned instead. If you insertecho "";in front of the "return true" everything works fine again (it then returns false). I also noticed, that the outer loop has to be a foreach-loop, if it was a for-loop the result would be also correct. [2009-03-05 17:34:29] fischikowski at web dot de Description: The echo ""; affects the "return true;" below. As long as the echo is there everything works as expected, if you put "//" in front of the echo, the return will be called even if it should not. Reproduce code: --- function matchHostList($host, $list_file) { $host = explode(".", $host); //$list_contents = explode("\n", file_get_contents($list_file)); $list_contents = array("www.google.de"); foreach($list_contents as $list_host) { $list_host = explode(".", rtrim($list_host)); //if the list-host is more specific than the tested host we can't match if(count($host) < count($list_host)) continue; for($i = 1;$i<=count($list_host) && $i <= count($host);$i++) { if($list_host[count($list_host)-$i] != $host[count($host)-$i]) continue 2; } echo ""; //this is necessary to avoid wrong return return true; } return false; } echo matchHostList("false.google.de", "")?"true":"false"; Expected result: When running as shown above this returns false (because "continue 2;" continues the loop above the "return true;"), this is OK. If you remove the echo ""; above the "return true;" You would expect that nothing changes (because we only removed some null-output), but in fact the return value changes to true. Actual result: -- see Expected result -- Edit this bug report at http://bugs.php.net/?id=47576&edit=1
#47576 [Opn->Fbk]: "continue i;" working wrong on foreach
ID: 47576 Updated by: j...@php.net Reported By: fischikowski at web dot de -Status: Open +Status: Feedback Bug Type: Scripting Engine problem Operating System: Windows Vista 32bit PHP Version: 5.2.9 New Comment: Thank you for this bug report. To properly diagnose the problem, we need a short but complete example script to be able to reproduce this bug ourselves. A proper reproducing script starts with , is max. 10-20 lines long and does not require any external resources such as databases, etc. If the script requires a database to demonstrate the issue, please make sure it creates all necessary tables, stored procedures etc. Please avoid embedding huge scripts into the report. Previous Comments: [2009-03-06 08:30:09] fischikowski at web dot de I could shorten the test-code (the previous was my original working code): function test() { $array = array("test"); foreach($array as $element) { for($j=0;$j<5;$j++) { continue 2; } return true; } return false; } echo test()?"true":"false"; I expect, that this returns false, because "continue 2" should step to the next foreach-loop (and "return true" should not be executed). But it happens, that true is returned instead. If you insertecho "";in front of the "return true" everything works fine again (it then returns false). I also noticed, that the outer loop has to be a foreach-loop, if it was a for-loop the result would be also correct. [2009-03-05 17:34:29] fischikowski at web dot de Description: The echo ""; affects the "return true;" below. As long as the echo is there everything works as expected, if you put "//" in front of the echo, the return will be called even if it should not. Reproduce code: --- function matchHostList($host, $list_file) { $host = explode(".", $host); //$list_contents = explode("\n", file_get_contents($list_file)); $list_contents = array("www.google.de"); foreach($list_contents as $list_host) { $list_host = explode(".", rtrim($list_host)); //if the list-host is more specific than the tested host we can't match if(count($host) < count($list_host)) continue; for($i = 1;$i<=count($list_host) && $i <= count($host);$i++) { if($list_host[count($list_host)-$i] != $host[count($host)-$i]) continue 2; } echo ""; //this is necessary to avoid wrong return return true; } return false; } echo matchHostList("false.google.de", "")?"true":"false"; Expected result: When running as shown above this returns false (because "continue 2;" continues the loop above the "return true;"), this is OK. If you remove the echo ""; above the "return true;" You would expect that nothing changes (because we only removed some null-output), but in fact the return value changes to true. Actual result: -- see Expected result -- Edit this bug report at http://bugs.php.net/?id=47576&edit=1