PHP密钥兑换页面问题:文件循环读取与密钥校验失效排查
我帮你梳理下代码里导致功能失效的几个核心问题,一个个来看:
文件打开模式错误
你用fopen("./generatedkeys.txt", "a")打开文件,"a"是追加模式,这种模式下文件指针一开始就停在文件末尾,feof($file)直接返回true,所以下面的while循环根本不会执行,$generatedk数组始终是空的,自然检测不到任何密钥。要读取文件的话,应该用只读模式:$file = fopen("./generatedkeys.txt", "r"); // 只读模式,指针在文件开头file_get_contents传入了错误参数
代码里$contents = file_get_contents($file);这一行,file_get_contents需要的是文件路径字符串(比如"./generatedkeys.txt"),但你传的是fopen返回的文件句柄资源,这会直接报错,导致后续的替换和写入都无法执行。fgets读取的内容包含换行符,导致匹配失败
fgets($file)会把每行的换行符(\n或者\r\n)也一起读进数组里,而你对比的是hash("sha256", $gkey)的纯哈希字符串,两者因为换行符的存在永远不相等,所以密钥检测逻辑根本触发不了。解决方法是读取时去掉换行符:$generatedk[] = trim(fgets($file)); // trim去掉首尾的空白、换行符文件句柄的关闭时机错误
你在foreach循环里匹配到密钥后就调用fclose($file),但此时你还在尝试操作这个文件,而且如果文件句柄没关闭就直接用file_put_contents,可能会因为文件被锁定导致写入失败。正确的做法是先关闭读取用的文件句柄,再进行文件内容的修改。未定义的
$hwid变量
代码里用到了hash("sha256", $hwid),但$hwid变量没有被赋值(比如从$_GET["hwid"]获取),这会产生PHP警告,而且写入到accs.txt的内容会是无效的哈希值(基于空字符串的哈希)。
另外给你提个优化建议:直接用file()函数读取文件会更简洁,它会把文件每行内容存入数组,还可以通过参数自动去掉换行符和空行:
$generatedk = file("./generatedkeys.txt", FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES);
修正后的核心逻辑大概是这样的:
$gkey = $_GET["key"]; $targetHash = hash("sha256", $gkey); $keyFile = "./generatedkeys.txt"; $accFile = "./accs.txt"; // 读取密钥文件,自动去掉换行符和空行 $generatedk = file($keyFile, FILE_IGNORE_NEW_LINES | FILE_SKIP_EMPTY_LINES); foreach ($generatedk as $index => $key) { if ($key === $targetHash) { // 移除匹配的密钥 unset($generatedk[$index]); // 重新写入文件 file_put_contents($keyFile, implode("\n", $generatedk) . "\n"); // 处理hwid(记得先获取$hwid的值) $hwid = $_GET["hwid"] ?? ''; // 假设从GET获取,可根据实际调整 if (!empty($hwid)) { file_put_contents($accFile, hash("sha256", $key) . "," . hash("sha256", $hwid) . "\n", FILE_APPEND); } break; } }
内容的提问来源于stack exchange,提问作者Cottient

