如何在 PHP 中正确实现表单提交与函数调用的分离设计-php教程-PHP中文网

如何在 PHP 中正确实现表单提交与函数调用的分离设计

本文详解为何将 post() 函数直接用于

本文详解为何将 post() 函数直接用于

以下是重构后的推荐结构：

✅ 步骤一：分离表单与处理逻辑

post.php（仅渲染表单）

btpanel phpsite 宝塔面板PHP网站

宝塔面板 PHP 网站管理：站点创建、删除、启停、PHP 版本切换、域名管理、SSL证书管理、伪静态管理、数据库管理

下载

<?php require 'includes/dbh.inc.php';
session_start();
date_default_timezone_set('America/Chicago');
?><title>New Post</title>

✅ 步骤二：创建独立处理器（includes/post_handler.php）

<?php require_once 'dbh.inc.php'; // 确保数据库连接可用
require_once 'post.inc.php'; // 加载函数定义

// ✅ 关键：仅当收到 POST 请求时才调用函数
if ($_SERVER['REQUEST_METHOD'] === 'POST' && isset($_POST['postSubmit'])) {
    post($conn); // 此处调用才真正有意义
} else {
    header('Location: ../home.php?error=invalid_request');
    exit();
}

✅ 步骤三：优化 post.inc.php（修复逻辑缺陷）

原函数存在严重问题：

重复调用 session_start()（已在 post.php 启动）；
while ($PostIDDuplicate = true) 是赋值而非比较，造成无限循环；
缺少对关键变量（如 $_SESSION['UserID']）的存在性校验。

修正后的 post() 函数示例：

function post($conn) {
    // ✅ 避免重复 session_start()
    if (session_status() === PHP_SESSION_NONE) {
        session_start();
    }

    // ✅ 严格校验会话与输入
    if (!isset($_SESSION['UserID']) || empty($_POST['Text'])) {
        header('Location: ../home.php?error=access_denied');
        exit();
    }

    $UserID = $_SESSION['UserID'];
    $Date = $_POST['Date'] ?? date('Y-m-d H:i:s');
    $Text = trim($_POST['Text']);
    $SectionID = 1;

    // ✅ 修复死循环：使用 !== false 和 proper condition
    $PostIDLength = 11;
    $chars = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
    $PostID = null;
    $attempts = 0;
    do {
        $PostID = substr(str_shuffle($chars), 0, $PostIDLength);
        $stmt = mysqli_prepare($conn, "SELECT 1 FROM PostTable WHERE PostID = ?");
        mysqli_stmt_bind_param($stmt, "s", $PostID);
        mysqli_stmt_execute($stmt);
        $result = mysqli_stmt_get_result($stmt);
    } while (mysqli_num_rows($result) > 0 && ++$attempts = 100) {
        header('Location: ../home.php?error=id_generation_failed');
        exit();
    }

    $stmt = mysqli_prepare($conn, "INSERT INTO PostTable (PostID, UserID, SectionID, Date, Text) VALUES (?, ?, ?, ?, ?)");
    mysqli_stmt_bind_param($stmt, "ssiss", $PostID, $UserID, $SectionID, $Date, $Text);

    if (mysqli_stmt_execute($stmt)) {
        header('Location: ../home.php?success=post_created');
    } else {
        header('Location: ../home.php?error=db_error');
    }
    exit();
}