Скрипт для устаревшей скидки - PullRequest
Новая
       6

Скрипт для устаревшей скидки

0 голосов
/ 26 января 2012

Я немного новичок в PHP, поэтому работа над сценариями продолжается.У меня есть страница, на которой пользователи регистрируются на семинар, за который им приходится платить.Если они регистрируются за 5 дней до начала семинара, они получают скидку, если они регистрируются за 5 дней до начала семинара, они должны заплатить полную стоимость.Я написал PHP-скрипт, который, я думаю, сработает (в любом случае работает - тестирует), но у меня есть ощущение, что есть лучший способ его написания.Я просто ищу любые предложения, которые есть у кого-либо, если таковые имеются.Ваша помощь очень ценится.Мой сценарий выглядит следующим образом. 

date_default_timezone_set('MST');
$discount_check  = date("YmdHis", mktime(date("H"), date("i"), date("s"), date("m") , date("d")+5 , date("Y") ));

//Made an array because there are several seminars a year and because I need to make it easy enough for others in my office (who don't know PHP) to add/edit seminar dates
$array = array(
'Utah_seminar' => '20120130153804',
'Seattle_seminar' => '20120723000000',
'Florida_seminar' => '20121005000000'
);

while ($seminar_dates = current($array)) 
{
    if ($discount_check >= $seminar_dates) 
    {
        echo 'discount is not eligible';
        break;
    }
    else
    {
        echo 'discount received';
        break;
    }
next($array);
}

Опять же, любая помощь очень ценится.Даже если он находит какие-либо ошибки с кодом.

Ответы [ 3 ]

2 голосов
/ 26 января 2012 
<?
date_default_timezone_set('MST');

$now = new DateTime();

// Made an array because there are several seminars a year and because I need
// to make it easy enough for others in my office (who don't know PHP) to
// add/edit seminar dates
$array = array(
    'Utah_seminar'    => new DateTime('2012-01-30 15:38:04'),
    'Seattle_seminar' => new DateTime('2012-07-23 00:00:00'),
    'Florida_seminar' => new DateTime('2012-10-05 00:00:00')
);

foreach ($array as $seminar => $date) {
    $cutoff = clone $date;
    $cutoff->modify('-5 days');

    if ($now > $cutoff) {
        echo "$seminar - discount is not eligible\n";
    } else {
        echo "$seminar - discount received\n";
    }
}
?>
0 голосов
/ 26 января 2012

В зависимости от того, что вы собираетесь делать с массивом, я бы порекомендовал отсортировать и затем определить первый применимый индекс. 

$discount_check = mktime(date());
//Sort array in reverse order
arsort($array);
for($i = 0; $i < count($array);$i++){
    //If array value meets the discount criteria, 
    if($array[$i]  <=   $discount_check - 432000){
    //The index of the first array item to match the discount criteria, since array is sorted all subsquent are applicable
        $split = $i;
        $i = count($i);
    }   
}
0 голосов
/ 26 января 2012

Поскольку вы используете break, ваш цикл будет выполняться только один раз.Если я понимаю, что вы пытаетесь сделать, вы должны использовать: 

$eligable = false;
foreach ($array as $seminar_date) {
  if ($discount_check < $seminar_date) {
    $eligable = true;
    break;
  }
}

if ($eligable) {
  echo 'discount received';
} else {
  echo 'discount is not eligible';
}

Я изменил цикл while на foreach, так как я думаю, что они более понятны для понимания

...