0

私は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);
}

繰り返しになりますが、どんな助けでも大歓迎です。コードに問題がある場合でも。

4

3 に答える 3

2 
<?
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";
    }
}
?>
于 2012-01-26T14:09:12.437 に答える
0

breakループを使用しているため、実行は1回だけです。あなたがやろうとしていることを私が理解しているなら、あなたは以下を使うべきです：

$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理解するのが簡単だと思うからです。

于 2012-01-26T14:08:29.783 に答える
0

配列で何をしようとしているのかに応じて、最初に適用可能なインデックスを並べ替えてから決定することをお勧めします。

$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);
    }   
}
于 2012-01-26T14:18:27.817 に答える