Question

I have made a small script, which should redirect to one of 8 pages by random. Before picking the page I ask the database, to narrow the choices. (So that every page is picked on the same seed) . Unfortunatly I got stuck in the do-while loop. Here is the code:

 <?php
$surveys = array(
    1 => "711275", 
    2 => "488985", 
    3 => "515385",
    4 => "467411",
    5 => "755429",
    6 => "335888",
    7 => "673921",
    8 => "532261");
$surveysCount = array(
    1 => 0, 
    2 => 0, 
    3 => 0,
    4 => 0,
    5 => 0,
    6 => 0,
    7 => 0,
    8 => 0);
$path = 'http://www.unet.univie.ac.at/~a0106191/index.php/';
$pathPostfix = '/lang-de';  

$user = 'xxxx';
$pass = 'xxxx';
$host = 'xxxx'; 

// establish database connection
$conn = mysql_connect($host, $user, $pass) or die('Cannot connect to Database - Please try later!');

mysql_select_db('a0106191', $conn) or die('Cannot connect to Database - Please try later!');

foreach ($surveys as $nr=>$survey) {
    $stmt = 'SELECT submitdate FROM lime_survey_'.$survey.' WHERE submitdate IS NOT NULL';
    //echo $stmt.'<br/>';
    if($res = mysql_query($stmt)) {
        if (mysql_num_rows($res) > 0) {
            $surveysCount[$nr] = mysql_num_rows($res);
        }
    }

    //echo $surveysCount[$nr].'<br/>';
}
$selection = array();
$counter = 0;
do {
    $offset = 0;
    for ($i = 1; $i < 9; $i++) {
        if ($surveysCount[$i] === $offset){
            $sel = $surveys[$i];
            $selection[] = $sel;
            $counter++;             
        }
    }
    $offset++;
    //echo $selection;
} while ($counter < 1);
// clean up connections
mysql_close($conn);
$whichSurveyNr = mt_rand(1, $counter);
$which = $selection[$whichSurveyNr-1];
$which = $path.$which.$pathPostfix;
  echo "wSrNr:";
    echo $whichSurveyNr;
echo "selection: ";
echo count($selection);
echo " which: ";
echo $which;
    foreach($selection as $k => $v)
        echo " sel: k: ".$k." v: ".$v;*/
header("Location: $which"); 
?>
Was it helpful?

Solution

You are currently always resetting offset within the loop. I don't know this is what you intend.

Try

$offset = 0;
do {

    for ($i = 1; $i < 9; $i++) {
        if ($surveysCount[$i] === $offset){
            $sel = $surveys[$i];
            $selection[] = $sel;
            $counter++;             
        }
    }
    $offset++;
    //echo $selection;
} while ($counter < 1);

OTHER TIPS

You need to use == not with ===,it will returns you always false

do {
     $offset = 0;
    for ($i = 1; $i < 9; $i++) {
        if ($surveysCount[$i] == $offset){
            $sel = $surveys[$i];
            $selection[] = $sel;
            $counter++;             
        }
    }
    $offset++;
    //echo $selection;
} while ($counter < 1);             

You need to make sure that $counter is being incremented.

Make sure $surveysCount[$i] === $offset always happens.

Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top