在将文件上传到我的服务器后，我正在使用它来将文件附加到邮件中：
for ($i = 0; $i <= 2; $i++)
{
$mail->AddAttachment($locatie.$_FILES['uploaded'.$i]['name'], $_FILES['uploaded'.$i]['name']);
}
添加附件是可选的，但是当没有上传文件时会报错：无法访问文件
如何防止出现这个错误？
【问题讨论】：
$locatie 代表什么？
标签： php attachment phpmailer
先看这里如何处理文件上传：
您需要参考临时文件名。这是获取实际文件名所必需的，而不仅仅是文件名。
$_FILES['userfile']['tmp_name']
所以访问$_FILES['userfile']['name']获取附件的书面文件名和$_FILES['userfile']['tmp_name']指定上传的实际文件（数据）。
大致放入您的代码中，incl. a validation 您也应该验证它实际上是文件上传：
for ($i = 0; $i <= 2; $i++)
{
# ignore file that have not been uploaded
if (empty($_FILES['uploaded'.$i])) continue;
# get the data of the file
$fileName = $_FILES['uploaded'.$i]['name'];
$filePath = $_FILES['uploaded'.$i]['tmpname'];
# add only if the file is an upload
is_uploaded_file($filePath)
&& $mail->AddAttachment($filePath, $fileName)
;
}
您的代码将两个工作混合在一起。这让您难以调试和改进 - 以及处理文件/系统问题和安全性等问题。
我建议您采用一种不同的方法：一步一步地做。在您的情况下，即 1.) 处理文件上传并收集您需要的数据，2.) 添加这些附件。
您可以通过查看 PHP 手册来改进第一部分。如果你想支持多个文件的上传，我建议你定位于页面Uploading multiple files给出的建议。然后处理文件上传并形成一个数组，其中包含客户端计算机上的文件名和服务器系统上每个条目的路径。
// see PHP Manual for multi file uploads, this is based on it
$validAttachments = array();
foreach($_FILES['userfile']['name'] as $index => $fileName)
{
$filePath = $_FILES['userfile']['tmp_name'][$index];
if(is_uploaded_file($filePath))
{
$attachment = new stdClass;
$attachment->fileName = $fileName;
$attachment->filePath = $filePath;
$validAttachments[] = $attachment;
}
}
如果该部分已经有错误，您知道它与文件上传过程有关。这是未经测试的代码，所以只是说明一个方向。
在第二步中，您可以遍历这样的数组并添加附件：
foreach($validAttachments as $attachment)
{
$mail->AddAttachment($attachment->filePath, $attachment->fileName);
}
然后，您可以更好地检查不同部分中的错误，而不会将一个问题与另一个问题混为一谈。
【讨论】：
在添加附件之前，您应该检查上传的文件是否存在，例如与fopen。
【讨论】：
for ($i = 0; $i <= 2; $i++)
{
if (file_exists($locatie.$_FILES['uploaded'.$i]['tmp_name'])) {
$mail->AddAttachment($locatie.$_FILES['uploaded'.$i]['tmp_name'], $_FILES['uploaded'.$i]['name']);
}
}
您使用$locatie.$_FILES['uploaded'.$i]['name'] 而不是$locatie.$_FILES['uploaded'.$i]['tmp_name'] 作为要加载的文件。上传文件时，它会使用临时名称重命名并放置在临时文件夹中。这就是你会得到它的地方，这就是为什么你需要用$locatie.$_FILES['uploaded'.$i]['tmp_name']引用它
【讨论】：
我已经能够使用计数器解决这个问题：
$locatie = 'uploads/';
$upload_count = -1;
for ($i = 0; $i <= 2; $i++)
{
if($_FILES['uploaded'.$i]['type'] != 'application/octet-stream') // Geen php files
{
$folder = $locatie.basename($_FILES['uploaded'.$i]['name']) ;
if(move_uploaded_file($_FILES['uploaded'.$i]['tmp_name'], $folder))
{
$upload_count ++;
}
}
循环查找附件：
for ($i = 0; $i <= $upload_count; $i++)
{
$mail->AddAttachment($locatie.$_FILES['uploaded'.$i]['name'], $_FILES['uploaded'.$i]['name']);
}
【讨论】：
同意@patapizza，但在检查文件系统之前（无论如何你都应该这样做），你也可以只检查 $_FILES 之类的
for ($i = 0; $i < count($_FILES); $i++){
// only iterate over the number of files you actually have
}
【讨论】：
在添加附件之前，您需要检查该文件是否实际上传/存在。类似的东西
for ($i = 0; $i <= 2; $i++)
{
if (file_exists($locatie.$_FILES['uploaded'.$i]['tmp_name'])) {
$mail->AddAttachment($locatie.$_FILES['uploaded'.$i]['tmp_name'], $_FILES['uploaded'.$i]['name']);
}
}
【讨论】：
$locatie.$_FILES['uploaded'.$i]['tmp_name']。我已经更新了答案——请检查。
不确定这是否完全符合您的要求，但这就是我附加多个上传文件并保留网络表单中的原始文件名的方式。用户的错误处理/消息传递不会为 UPLOAD_ERR_NO_FILE 提供错误，因为文件上传是可选字段，但仍会处理 foreach 循环中的每个输入字段。
foreach ($_FILES["userfile"]["error"] as $key => $error)
{
if ($error == UPLOAD_ERR_OK) {
$tmp_name = $_FILES["userfile"]["tmp_name"][$key];
$name = $_FILES["userfile"]["name"][$key];
$mailer->AddAttachment($tmp_name, $name);
}
$name = $_FILES["userfile"]["name"][$key];
switch($error){
case UPLOAD_ERR_INI_SIZE: echo $errmsg1.$name.$errmsg2;
break;
case UPLOAD_ERR_FORM_SIZE: echo $errmsg1.$name.$errmsg2;
break;
case UPLOAD_ERR_PARTIAL: echo $errmsg1.$name.$errmsg2;
break;
case UPLOAD_ERR_NO_FILE:
break;
case UPLOAD_ERR_NO_TMP_DIR: echo $errmsg1.$name.$errmsg2;
break;
case UPLOAD_ERR_CANT_WRITE: echo $errmsg1.$name.$errmsg2;
break;
case UPLOAD_ERR_EXTENSION: echo $errmsg1.$name.$errmsg2;
break;
}
}
【讨论】：