Can I optimise this loop?

[Perad]

Is there anyway I can make this loop stop searching for something once it has found its value?

 

for($i=0; $i < count($validpages); $i++) {
if ($_GET['page'] == $validpages[$i]) {
	$page = $validpages[$i];
}
}

[The Little Guy]

<?php
for($i=0; $i < count($validpages); $i++) {
if ($_GET['page'] == $validpages[$i]) {
	$page = $validpages[$i];
	break;
}
}
?>

[wildteen88]

if $validpages contains an array of pages and you all you want to do is see if the value of $_GET['page'] is set within that array then use in_array function instead rather than looping through the array manually.

 

If you used in_array the above code can be written as:

if(in_array($_GET['page'], $validpages)
{
    $page = $_GET['page'];
}