Tell me more ×
Stack Overflow is a question and answer site for professional and enthusiast programmers. It's 100% free, no registration required.
up vote 0 down vote favorite

Is it possible to sort a complex for loop? Would I store the values in an array and then sort by that?

This is the for loop I'm working with, I would like to sort by the date & time of the events I'm pulling through

foreach ($recurring_events as $recurring_event):
    if ($recurring_event->period == 'week') { 
        $StartDate = strtotime($event->RequestDateStart);
        $EndDate = strtotime($event->RequestDateEnd);
        $TotalDays = round(($EndDate-$StartDate)/(60*60*24*7));
        for($i=0; $i<($TotalDays-1); $i++) {
            $StartDate += (60*60*24*7);
            if (date('WMY', $StartDate) == date('WMY')) {
                echo '<div class="col-12">';

                echo '<strong>Event Name:</strong> ' . $event->EventName . '<br /><strong>Date:</strong> ' . date('l, F j o',$StartDate) . '<br /><strong>Start Time:</strong> ' . date('g:i a',strtotime($event->RequestTimeStart));

                echo '</div>';
            }
        }
    }
endforeach;
share|improve this question
Is $recurring_event supposed to be $event? – Philip Whitehouse Feb 4 at 23:34
@PhilipWhitehouse no, this is just a snippet of a lot bigger code :) – ftntravis Feb 4 at 23:36
I don't entirely understand what you're sorting by then - $StartDate is going to be the same every time you go round the loop. – Philip Whitehouse Feb 4 at 23:40
@PhilipWhitehouse $StartDate is being increased by a week every time in the loop so it is not the same. – ftntravis Feb 4 at 23:58
what you want sort by aces or desc? – rohitarora Feb 6 at 5:03
show 4 more comments

4 Answers

up vote 0 down vote accepted 
foreach ($recurring_events as $recurring_event):
    if ($recurring_event->period == 'week') { 
        $StartDate = strtotime($event->RequestDateStart);
        $EndDate = strtotime($event->RequestDateEnd);
        $TotalDays = round(($EndDate-$StartDate)/(60*60*24*7));
        for($i=0; $i<($TotalDays-1); $i++) {
            $EndDate -= (60*60*24*7);
            if (date('WMY', $StartDate) == date('WMY')) {
                echo '<div class="col-12">';

                echo '<strong>Event Name:</strong> ' . $event->EventName . '<br /><strong>Date:</strong> ' . date('l, F j o',$StartDate) . '<br /><strong>Start Time:</strong> ' . date('g:i a',strtotime($event->RequestTimeStart));

                echo '</div>';
            }
        }
    }
endforeach;

try this start from the end date and minus it.if you want it ti be in decsnding sorry if i dint understood you question

share|improve this answer
I tried it out with no luck :( it is producing multiples of each date. When trying to put the results in an array, I am ending up with empty values. If there is a way to sort the array in the format I'm getting it, I think that would work. Check out how it looks here: beta.revival.tv/test – ftntravis Feb 9 at 19:47
i think there is something with the code itself can you please write the complete code so i can decend you whole data.. – rohitarora Feb 10 at 3:48
do you have an email address I can send it to you at? Just to keep this clean and not overwhelming with too much info. – ftntravis Feb 11 at 3:10
[email protected] you can mail me. – rohitarora Feb 11 at 5:08
up vote 2 down vote

put it in an array, then refer to this post for how to sort by date

How to sort a date array in PHP

share|improve this answer
up vote 1 down vote

You need to put your results into an array, then sort it, then output it.

$output = array();
foreach ($recurring_events as $recurring_event):
    if ($recurring_event->period == 'week') { 
        $StartDate = strtotime($event->RequestDateStart);
        $EndDate = strtotime($event->RequestDateEnd);
        $TotalDays = round(($EndDate-$StartDate)/(60*60*24*7));
        for($i=0; $i<($TotalDays-1); $i++) {
            $StartDate += (60*60*24*7);
            if (date('WMY', $StartDate) == date('WMY')) { //$StartDate >= strtotime('now') &&
                $output[$StartDate] .= '<div class="col-12">';

                $output[$StartDate] .='<strong>Event Name:</strong> ' . $event->EventName . '<br /><strong>Date:</strong> ' . date('l, F j o',$StartDate) . '<br /><strong>Start Time:</strong> ' . date('g:i a',strtotime($event->RequestTimeStart));

                $output[$StartDate] .='</div>';
            }
        }
    }
endforeach;

And then use sort

sort($output);

Assuming that your StartDate is the date you need (then it should be an unix timestampe?)

http://php.net/manual/en/function.sort.php

share|improve this answer
I will also need to sort by time. So just specify a time variable as well in the array? I can pull the time variable through as well so it will be populated. So something like Something like $output[$StartDate, $StartTime]? – ftntravis Feb 4 at 23:35
This does not work, it is storing it in multiple arrays, not a single array. – ftntravis Feb 4 at 23:58
So the second parameter of the date function is a timestamp, an integer, so the value of $StartDate will include the time as well as the date. When sorted it will sort date and time. To clarify, the timestamp is the number of seconds passed since Jan 1st 1970, for example: 1360031642 – SSH This Feb 5 at 1:31
up vote 1 down vote

It's not possible to sort the data while looping round it to output it. Instead, sort the data, then loop round the sorted data to output.

A useful function is uasort which takes an array and uses a function you define to sort it:

function sortEvents($a, $b) {
   if (strtotime($a->RequestDateStart) == strtotime($b->RequestDateStart)) {
      return 0;
   }
   return ($a->RequestDateStart < $b->RequestDateStart) ? -1 : 1;
}
uasort($recurring_events,'sortEvents');
share|improve this answer
I've been looking more into this function, but will that work with what I am trying to do? I am using a for loop to build my results so I can't sort before hand. It would have to be after somehow. – ftntravis Feb 9 at 19:49
Build, Sort, Output. Don't mix them. Looping 3 times is not expensive. – Philip Whitehouse Feb 10 at 16:25

Your Answer

 
discard

By posting your answer, you agree to the privacy policy and terms of service.

Not the answer you're looking for? Browse other questions tagged or ask your own question.