如果IF满足要求，则在LOOP时停止

Question

I have try to find the solution in stackoverflow to fix my problem but no one of them match with my problem.

my problem is;

I have if statement inside my javascript which inside if statement have a popup. I want my javascript display the popup for 1 time only if if statement meet the condition. I try to use break inside if statement, but it still show a popup for a several time.

code:

<?php
if(isset($_POST['submit'])){ 

$reserve_stime = htmlentities($_POST['starttime']);
$reserve_etime = htmlentities($_POST['endtime']);

$reservedstime = "SELECT startmeetingtime, endmeetingtime 
                  FROM meeting_room_reservation 
                  WHERE room_reserve_session = '$varsession' 
                  AND room_reserve_date = '$convardate'
                  AND room_id = $idroom";
$result7=mysql_query($reservedstime) or die("invalid query:".mysql_error()); 
while($row7=mysql_fetch_assoc(@$result7)){
     $reservedstime2 = "SELECT timereserve_start_id
                        FROM timereserve_start 
                        WHERE timereserve_start_id BETWEEN " . $row7['startmeetingtime'] . " AND " . $row7['endmeetingtime'] . "";
     $result8=mysql_query($reservedstime2) or die("invalid query:".mysql_error()); 

     while ($row8=mysql_fetch_assoc(@$result8)) {
           $timealreadyreserved = $row8['timereserve_start_id'];

           echo "<script type='text/javascript'>

                    for (i=$reserve_stime; i <= $reserve_etime ; i++){

                      if (i == $timealreadyreserved){
                        alert('Time is not Available');
                        break;
                      }
                      else{
                        //do nothing
                      }
                    }

                 </script>";

     }

  }
}
?>

my friends said, the problem is before for loop have do while loop, so that is why the popup still display for a several time even already put the break inside if statement

any idea? thank you, faizal

Answer 1

You can use a boolean variable(like $flag) to check whether the condition has been met or not, and subsequently break out of the loop, like this:

// your code

while($row7=mysql_fetch_assoc(@$result7)){
    $reservedstime2 = "SELECT timereserve_start_id
                        FROM timereserve_start 
                        WHERE timereserve_start_id BETWEEN " . $row7['startmeetingtime'] . " AND " . $row7['endmeetingtime'] . "";
    $result8=mysql_query($reservedstime2) or die("invalid query:".mysql_error()); 

    $flag = false;
    while($row8=mysql_fetch_assoc(@$result8)) {
        $timealreadyreserved = $row8['timereserve_start_id'];

        for ($i=$reserve_stime; $i <= $reserve_etime ; $i++){
            if($i == $timealreadyreserved){
                $flag = true;
                ?>
                <script>alert("Time is not Available");</script>
                <?php
                break;
            }
        }
        if($flag){
            break;
        }
    }
    if($flag){
        break;
    }

}

// your code 

Caution: Do not mix javascript and php in your code, like you did in inner for loop.

Sidenote: Don't use mysql_ database extensions, they were deprecated in PHP 5.5.0 and were removed in PHP 7.0.0. Use mysqli or PDO extensions instead. And this is why you shouldn't use mysql_ functions.