
在 php 开发中,我们经常会遇到需要执行相似操作,但仅少数变量值不同的情况。原始代码中展示了一个典型的例子:当处理文件下载时,需要根据 $file['type'] 发送文件,但存在多个文件 id 字段,例如 $file['file_id'] 和 $file['file_id2'],并且可能需要扩展到 $file['file_id3'] 乃至 $file['file_id24']。这种通过简单复制粘贴代码来适应不同文件 id 的做法,会导致代码冗余、难以维护且容易出错。
核心需求是:
为了避免代码重复,我们可以利用 for 循环结合字符串拼接来动态构造变量名。PHP 允许通过数组键的字符串形式访问关联数组的元素,这使得我们可以在循环中动态生成 file_id1、file_id2 等键名。
以下是优化后的代码片段,它将原始代码中重复发送文件的逻辑封装在一个 for 循环中,以处理从 file_id1 到 file_id24 的情况:
if (preg_match('/^\/start (.*)/', $text, $match) or preg_match('/^\/get_(.*)/', $text, $match)) {
$id = $match[1];
if (isJoin($from_id)) {
$fileData = mysqli_query($db, "SELECT * FROM `file` WHERE `id` = '{$id}'");
$file = mysqli_fetch_assoc($fileData);
if (mysqli_num_rows($fileData)) {
if ($file['password']) {
sendMessage($from_id, "please send pass :", "markdown", $btn_back, $message_id);
mysqli_query($db, "UPDATE `user` SET `step` = 'password', `getFile` = '$id' WHERE `from_id` = '$from_id'");
} else {
$downloads = number_format($file['downloads']);
$downloads++;
$caption = urldecode($file['caption']);
// 循环发送文件,处理 file_id1 到 file_id24
// 假设 file_id 字段从 1 开始计数,并且我们希望处理到 24 个文件
$max_file_index = 24; // 固定循环次数
for ($i = 1; $i <= $max_file_index; $i++) {
$file_id_key = "file_id" . $i; // 动态构造键名,例如 "file_id1", "file_id2"
// 检查对应的文件ID是否存在且不为空
if (isset($file[$file_id_key]) && !empty($file[$file_id_key])) {
Ilyad("send{$file['type']}", [
'chat_id' => $from_id,
$file['type'] => $file[$file_id_key], // 使用动态键名访问文件ID
'caption' => "? count : {$downloads}\n{$caption}\n Thanks",
'parse_mode' => "html",
]);
}
}
mysqli_query($db, "UPDATE `file` SET `downloads` = `downloads`+1 WHERE `id` = '$id'");
mysqli_query($db, "UPDATE `user` SET `step` = 'none', `downloads` = `downloads`+1 WHERE `from_id` = '$from_id'");
}
} else {
sendMessage($from_id, "hi welcome to bot", 'markdown', $btn_home, $message_id);
}
} else {
joinSend($from_id);
mysqli_query($db, "UPDATE `user` SET `getFile` = '$id' WHERE `from_id` = '$from_id'");
}
}代码解析:
立即学习“PHP免费学习笔记(深入)”;
在实际应用中,文件数量可能不是固定的 24 个,而是根据具体情况而定。为了使代码更具灵活性,我们可以从数据库中获取循环的上限值。假设在 file 表中有一个字段,例如 num_attachments,存储了当前文件记录所关联的附件数量。
if (preg_match('/^\/start (.*)/', $text, $match) or preg_match('/^\/get_(.*)/', $text, $match)) {
$id = $match[1];
if (isJoin($from_id)) {
$fileData = mysqli_query($db, "SELECT * FROM `file` WHERE `id` = '{$id}'");
$file = mysqli_fetch_assoc($fileData);
if (mysqli_num_rows($fileData)) {
if ($file['password']) {
sendMessage($from_id, "please send pass :", "markdown", $btn_back, $message_id);
mysqli_query($db, "UPDATE `user` SET `step` = 'password', `getFile` = '$id' WHERE `from_id` = '$from_id'");
} else {
$downloads = number_format($file['downloads']);
$downloads++;
$caption = urldecode($file['caption']);
// 从数据库中获取动态循环上限
// 假设 $file 数组中包含一个名为 'num_attachments' 的字段
$max_file_index = isset($file['num_attachments']) ? (int)$file['num_attachments'] : 1; // 默认至少发送一个文件
// 确保上限至少为1,且不超过某个合理的最大值(例如24或更多,视系统设计而定)
// 避免数据库数据错误导致无限循环或资源耗尽
$max_file_index = max(1, $max_file_index);
// 也可以设置一个硬性上限,防止意外情况
// $max_file_index = min($max_file_index, 50);
for ($i = 1; $i <= $max_file_index; $i++) {
$file_id_key = "file_id" . $i;
if (isset($file[$file_id_key]) && !empty($file[$file_id_key])) {
Ilyad("send{$file['type']}", [
'chat_id' => $from_id,
$file['type'] => $file[$file_id_key],
'caption' => "? count : {$downloads}\n{$caption}\n Thanks",
'parse_mode' => "html",
]);
}
}
mysqli_query($db, "UPDATE `file` SET `downloads` = `downloads`+1 WHERE `id` = '$id'");
mysqli_query($db, "UPDATE `user` SET `step` = 'none', `downloads` = `downloads`+1 WHERE `from_id` = '$from_id'");
}
} else {
sendMessage($from_id, "hi welcome to bot", 'markdown', $btn_home, $message_id);
}
} else {
joinSend($from_id);
mysqli_query($db, "UPDATE `user` SET `getFile` = '$id' WHERE `from_id` = '$from_id'");
}
}代码解析:
立即学习“PHP免费学习笔记(深入)”;
数据库设计优化: 将 file_id1, file_id2, ..., file_idX 作为单独的字段存储在数据库中并非最佳实践。更好的做法是创建一个独立的关联表,例如 file_attachments,其中包含 file_id (主文件ID), attachment_id (附件ID), attachment_type 等字段。这样可以更灵活地处理任意数量的附件,而无需修改表结构。
例如: file 表: id, caption, type, downloads, password, num_attachmentsfile_attachments 表: id, parent_file_id, attachment_file_id, order_index
查询时通过 JOIN 操作获取所有附件: SELECT fa.attachment_file_id FROM file_attachments fa WHERE fa.parent_file_id = '{$id}' ORDER BY fa.order_index ASC
然后遍历查询结果即可。
SQL 注入防护: 原始代码中使用了 mysqli_query 并直接拼接了变量,例如 WHERE id = '{$id}'。这存在 SQL 注入的风险。在生产环境中,应始终使用参数化查询(如 mysqli_prepare 和 mysqli_stmt_bind_param)来防止此类安全漏洞。
错误处理:
性能考虑: 如果 num_attachments 的值非常大,在一个请求中发送大量文件可能会导致性能问题或超时。考虑分页、异步处理或限制单次请求发送的文件数量。
代码可读性: 尽管循环减少了重复代码,但如果逻辑变得过于复杂,可以考虑将循环体内的逻辑封装成一个独立的函数,提高代码的可读性和复用性。
通过 for 循环和动态键名构造,我们可以高效地处理 PHP 中重复的代码逻辑,尤其是在处理序列化变量(如 file_id1, file_id2 等)时。结合从数据库动态获取循环上限的方法,能够极大地提升代码的灵活性、可维护性和扩展性。同时,为了构建健壮和安全的应用程序,务必注意数据库设计、SQL 注入防护、错误处理以及性能优化等方面的最佳实践。
以上就是PHP 代码重构:利用循环优化重复逻辑与动态数据访问的详细内容,更多请关注php中文网其它相关文章!
PHP怎么学习?PHP怎么入门?PHP在哪学?PHP怎么学才快?不用担心,这里为大家提供了PHP速学教程(入门到精通),有需要的小伙伴保存下载就能学习啦!
Copyright 2014-2025 https://www.php.cn/ All Rights Reserved | php.cn | 湘ICP备2023035733号