Menu
Community /Pin to ProfileBookmark
@NogDogDec 17.2010 — #Couldn't have said it better myself. The main points I wanted to get across were to (a) never assume data is correct and within acceptable bounds, and (b) do some sort of error-reporting or logging to tell you what went wrong when it does. Or in just two words: defensive coding. ? (A third and less important point was the idea of using the array to get the month instead of 12 if conditions.)
[RESOLVED] Creating a "Header" redirect depending on the month
[i](Hmmm…that’s weird: how did my reply end up before the original post?)
I might do that part as:
[code=php]
$month = intval($month);
if($month < 1 or $month > 12) {
// display/log error or whatever you want to do in this case
}
else {
$monthName = array(1 => ‘january’, ‘february’, ‘march’, ‘april’, ‘may’,
‘june’, ‘july’, ‘august’, ‘september’, ‘october’, ‘novemeber’, ‘december’);
$location = sprintf(“http://example.com/%02d-%s.php”, $month, $monthName[$month]);
if(!headers_sent()) {
header(“Location: $location”);
exit;
}
else {
// display/log error or whatever you want to do in this case
}
}
Sign in
to post a comment6 Comments(s) ↴
0
@NogDogDec 17.2010 — #Couldn't have said it better myself. The main points I wanted to get across were to (a) never assume data is correct and within acceptable bounds, and (b) do some sort of error-reporting or logging to tell you what went wrong when it does. Or in just two words: defensive coding. ? (A third and less important point was the idea of using the array to get the month instead of 12 if conditions.)