[webwired]

Hello everyone...

Could someone please help me with this looping problem that I am having...

I have a WHILE loop that iterates through the matching database records... and then inside of that loop I have a FOR loop... As you can probably already imagine, for every record there is in the database, it repeats the FOR loop that many times... I am baffled if I can think of a different way to code it though...

Here's what I have...

$query = "SELECT OrderTickets.*, Appointments.* FROM Appointments 
    LEFT JOIN OrderTickets ON Appointments.OrderTicketID = OrderTickets.OrderTicketID 
    WHERE Appointments.AppointmentStartDateTime BETWEEN '$BeginningDateTime' AND '$EndingDateTime'";
$result = mysql_query($query);
while ($row = mysql_fetch_assoc($result))
{
    for ($i=1;$i<25;$i+=0.25)
    {

I'd appreciate any advice anyone can give me...

[Sho Ke]

..What exactly are you trying to do with those loops? Are you getting an error? Is that the full code?

[webwired]

Sho Ke, on 07 September 2012 - 08:10 PM, said:

..What exactly are you trying to do with those loops?

Loop

Sho Ke, on 07 September 2012 - 08:10 PM, said:

Are you getting an error?

No, if I had, one would think that I would have mentioned that.

Sho Ke, on 07 September 2012 - 08:10 PM, said:

Is that the full code?

Let me answer your question with a question, do you typically find full code with open brackets?


You know, I am, and I'm sure that I can't be the only one... That is sick and tired of someone like you posting the first reply to a post with absolute garbage... For future reference, if you don't have something to contribute to a post, STAY THE F**K AWAY!

[Sho Ke]

Maybe if you would explain what you need help with, no one would have to ask?

You said you have two loops, then you ask for advice. Advice for what? "Looping"? If you're asking for help on how to "loop", then just slap on two ending braces and you're "looping". But I doubt that's what you want.

Then again, I don't know what you want to do because you didn't tell me.

[CTphpnwb]

webwired, on 07 September 2012 - 08:42 PM, said:

I have a WHILE loop that iterates through the matching database records... and then inside of that loop I have a FOR loop... As you can probably already imagine, for every record there is in the database, it repeats the FOR loop that many times... I am baffled if I can think of a different way to code it though...

Here's what I have...

$query = "SELECT OrderTickets.*, Appointments.* FROM Appointments 
    LEFT JOIN OrderTickets ON Appointments.OrderTicketID = OrderTickets.OrderTicketID 
    WHERE Appointments.AppointmentStartDateTime BETWEEN '$BeginningDateTime' AND '$EndingDateTime'";
$result = mysql_query($query);
while ($row = mysql_fetch_assoc($result))
{
    for ($i=1;$i<25;$i+=0.25)
    {

You haven't stated what the problem is so you should expect people to start from square one. Of course the for loop will be repeated for every record found in the table. That's what it means to put the for loop inside the while loop.

[webwired]

CTphpnwb, on 07 September 2012 - 08:35 PM, said:

You haven't stated what the problem is so you should expect people to start from square one. Of course the for loop will be repeated for every record found in the table. That's what it means to put the for loop inside the while loop.

If I didn't state what my problem was, then why did you just mention it in your reply?

BTW: Sho Ke, stay away from my posts.

This post has been edited by webwired: 07 September 2012 - 06:47 PM

[CTphpnwb]

So your problem is that the code is doing what you told it to do? The solution then is simple: don't tell it to do that! If it's something else, please tell us. Maybe you could start with why you've put a for loop inside a while loop.

[webwired]

CTphpnwb, on 07 September 2012 - 08:50 PM, said:

So your problem is that the code is doing what you told it to do? The solution then is simple: don't tell it to do that! If it's something else, please tell us. Maybe you could start with why you've put a for loop inside a while loop.

Well, with the FOR loop I'm creating a scheduling calendar... With the WHILE loop I was pulling from the database any appointments that fall in the calendar's timeline... When I wrote the code I only used one appointment to test it with... after I started adding other appointments on the same day I realized that I had put a loop inside of a loop... but now can't think of a way to write the code so that it's not a loop inside of a loop...

$BeginningDateTime = date('Y-m-d 00:00:00', strtotime($SelectedDay));
$EndingDateTime = date('Y-m-d 23:59:59', strtotime($SelectedDay));
$query = "SELECT OrderTickets.*, Appointments.* FROM Appointments LEFT JOIN OrderTickets ON Appointments.OrderTicketID = OrderTickets.OrderTicketID WHERE Appointments.AppointmentStartDateTime BETWEEN '$BeginningDateTime' AND '$EndingDateTime'";
$result = mysql_query($query);
while ($row = mysql_fetch_assoc($result))
{
    for ($i=1;$i<25;$i+=0.25)
    {
        $rowCounter += 1;
        echo '<tr class="'.OddOrEven2($rowCounter).'">';
        echo '<td class="time"> '; 
        if ($i > 12.5){$e = $i - 12;} else {$e = $i;} 
        $e = number_format($e, 2, '.', ' ');
        list($whole, $decimal) = explode('.', $e);
        if ($decimal == 25){$decimal = 15;}
        if ($decimal == 50){$decimal = 30;}
        if ($decimal == 75){$decimal = 45;}
        if ($whole == 0){$whole = 12;}
        if ($i < 12 || $i == 24){$f = 'am';}else{$f = 'pm';}
        if ($decimal == 00){echo $whole.':'.$decimal.' '.$f;}
        echo '</td>';

        $theBeginningTime = date('H:i:s', strtotime($row['AppointmentStartDateTime']));
        $theEndingTime = date('H:i:s', strtotime($row['AppointmentFinishDateTime']));
        $theTime = $whole.':'.$decimal.' '.$f;
        $LoopTime = date('H:i:s', strtotime($theTime));
        if ($LoopTime == $theBeginningTime || $LoopTime == $theEndingTime || ($LoopTime > $theBeginningTime && $LoopTime < $theEndingTime))
        {
            echo '<form id="CalendarCell" action="calendar.php?eventaction" method="post">';
            echo '<input type="hidden" id="AppointmentID" name="AppointmentID" value="'.$row['AppointmentID'].'" />';
            echo '<input type="hidden" id="SelectedTime" name="SelectedTime" value="'.$LoopTime.'" />';
            echo '<td class="appointment" style="background-color: #7C92AA;">';
            if ($LoopTime == $theBeginningTime)
            {
                echo date('g:i a', strtotime($row['AppointmentStartDateTime'])).' - '.date('g:i a', strtotime($row['AppointmentFinishDateTime'])).'<br />';
                if (!empty($row['OrderTicketID']))
                {
                    echo $row['OrderTicketFirstName'].' ' .$row['OrderTicketLastName'].' - '.$row['OrderTicketZipCode'].'<br />
                    '.FetchRequirements($row['OrderTicketID']);
                }
                else
                {
                    echo '<pre>'.$row['WhatsHappening'].'</pre>';
                }
            }
            else
            {
                echo '&nbsp;';
            }
            echo '</td>';
            echo '</form>';
        }
        else 
        {
            echo '<form id="CalendarCell" action="calendar.php?eventaction" method="post">';
            echo '<input type="hidden" id="SelectedTime" name="SelectedTime" value="'.$LoopTime.'" />';
            echo '<td class="appointment">&nbsp;</td>';
            echo '</form>';
        }
        echo '</tr>';
    }
}

This post has been edited by webwired: 07 September 2012 - 06:56 PM

[CTphpnwb]

It looks like you're trying to use $i to display time slots through the day, but you're doing that before you've got all the time slots. I would read everything into an associative array where the time is the key and then display the array. That way I could initialize the array to all blank values and used the database to fill in whatever it finds.

By the way, you really should be using prepared statements and not deprecated mysql* functions.

[modi123_1]

Everyone relax a bit. webwired - in the future if you are not getting errors, but instead getting odd behavior it's best to also explain what you would like the behavior to do in addition to what you put down (what it is currently doing). Once everyone has a picture then everyone can join arms and skip down the path of help and learning together! (humming Kumbaya is optional)

[webwired]

modi123_1, on 07 September 2012 - 09:17 PM, said:

Everyone relax a bit. webwired - in the future if you are not getting errors, but instead getting odd behavior it's best to also explain what you would like the behavior to do in addition to what you put down (what it is currently doing). Once everyone has a picture then everyone can join arms and skip down the path of help and learning together! (humming Kumbaya is optional)

You bet

CTphpnwb, on 07 September 2012 - 09:07 PM, said:

It looks like you're trying to use $i to display time slots through the day, but you're doing that before you've got all the time slots. I would read everything into an associative array where the time is the key and then display the array. That way I could initialize the array to all blank values and used the database to fill in whatever it finds.

By the way, you really should be using prepared statements and not deprecated mysql* functions.

I'm going to have to meditate on the Associative Array solution... Sounds like a plan, just takes me a while to put it together in my head first...

As far as the deprecated mysql, I know... It's all I know for now though, this application just has to last a little while before I have it professionally rewritten...

This post has been edited by webwired: 07 September 2012 - 07:22 PM

[CTphpnwb]

A little while may be all it takes to have your site hacked.

There are good PHP tutorials on this site, and then there are excellent ones like this one.

[JackOfAllTrades]

Doesn't this code render multiple forms with the id of CalendarCell? You can't have valid HTML with duplicate element IDs.

You want one form, and one form only.