会员到期判断代码逻辑问题：PHP代码未按预期触发过期提示

核心问题：判断逻辑完全反转 #

原代码里的条件strtotime($_SESSION['dateOfMembershipEnds']) >= date("Y-m-d")是判断会员有效期大于等于当前日期（即未过期），但你要触发的是过期提示，逻辑完全写反了。

其他潜在问题 #

• 日期比较时，一边是时间戳（strtotime转换结果），一边是字符串（date("Y-m-d")），类型不一致易出问题，应两边统一转成时间戳再比较。
• 代码未闭合：最后缺少}来结束else if代码块。
• 未确保Session已启动：使用$_SESSION前必须调用session_start()，否则无法读取Session变量。
• 需确认$_SESSION['dateOfMembershipEnds']的日期格式是strtotime可解析的（如YYYY-MM-DD），否则转换会失败。

修正后的代码 #

<?php
// 必须先启动Session
session_start();

// 从数据库获取会员到期日期
$_SESSION['dateOfMembershipEnds'] = $row['dateOfMembershipEnds'];

if(empty($email)){
    echo '<script>alert("Email is required!")</script>';
}else if(empty($password)){
    echo '<script>alert("Password is required!")</script>';
}else if(strtotime($_SESSION['dateOfMembershipEnds']) < strtotime(date("Y-m-d"))){
    // 当会员到期日期早于当前日期时，提示过期
    echo '<script>alert("Your Subscription Mr/Ms: '.$_SESSION['username'].' is already expired")</script>';
}
?>

额外优化建议 #

• 可以把当前日期的时间戳提前计算，避免重复调用strtotime：

  $today = strtotime(date("Y-m-d"));
  $membershipEnd = strtotime($_SESSION['dateOfMembershipEnds']);
  if($membershipEnd < $today) {
      // 提示过期
  }
  

• 若数据库中存储的日期包含时间，建议用date("Y-m-d H:i:s")来更精确判断。

内容的提问来源于stack exchange，提问作者chemuel castillo