如何生成唯一文件名避免覆盖PO+Invoice命名的上传附件？

首先你要排查之前用Guid.NewGuid()失效的原因：很大概率是你只修改了数据库中TPT对象的FileName字段，但保存文件的路径依然用了PO_Number.Text + Invoice.Text + ".pdf"，导致数据库存了Guid文件名，但实际文件还是用重复的业务标识命名，自然会被覆盖。

下面给你几个可靠的解决方案：

方案一：完全用Guid生成唯一文件名（最稳妥） #

生成Guid后，同时同步数据库记录和文件保存路径，确保两者一致：

修改你的SUbmit方法中的文件名相关逻辑：

// 生成唯一Guid文件名
string uniqueFileName = $"{Guid.NewGuid().ToString()}.pdf";

TPT tPT = new TPT();
tPT.Invoice_No = Invoice.Text;
tPT.PO_Number = PO_Number.Text;
tPT.Item_Code = Manufacture.Text;
tPT.Heat_Code = HeadCode.Text;
tPT.Description = Description.Text;
tPT.FileName = uniqueFileName; // 数据库存Guid文件名
dataBaseDataContext.TPTs.InsertOnSubmit(tPT); // 注意你原代码此处写的是TPT，应为变量tPT
dataBaseDataContext.SubmitChanges();

// 保存文件时用同一个Guid文件名
string path = Server.MapPath($"Attachments/{uniqueFileName}");
FileUploadtoServer.SaveAs(path);

方案二：结合业务标识+Guid（兼顾可识别性） #

如果需要从文件名快速关联业务信息，可以把PO、Invoice和Guid组合起来，既保证唯一，又方便后续排查：

string uniqueFileName = $"{PO_Number.Text}_{Invoice.Text}_{Guid.NewGuid().ToString()}.pdf";

后续的数据库赋值和文件保存路径都用这个变量即可。

方案三：增加文件存在性检查（双重保险） #

即使生成了唯一文件名，也可以在保存前检查目标路径是否已存在文件，避免极端情况下的冲突（虽然Guid重复概率极低，但可以做兜底）：

string uniqueFileName = $"{Guid.NewGuid().ToString()}.pdf";
string path = Server.MapPath($"Attachments/{uniqueFileName}");

// 检查文件是否存在，存在则重新生成Guid
while (System.IO.File.Exists(path))
{
    uniqueFileName = $"{Guid.NewGuid().ToString()}.pdf";
    path = Server.MapPath($"Attachments/{uniqueFileName}");
}

// 后续数据库赋值和保存操作同上

额外优化你的代码问题 #

1. 文件扩展名判断逻辑优化：原来的取3位字符的方式不够严谨，改用Path类的方法更可靠：

string fileExtension = System.IO.Path.GetExtension(FileUploadtoServer.FileName).ToLower();
if (fileExtension != ".pdf")
{
    Response.Write("<script>alert('Please select a pdf File!')</script>");
    return;
}

2. 字段为空时的逻辑修复：原来的代码在字段为空时没有return，会继续执行插入逻辑，导致空数据入库，需要加上：

if (string.IsNullOrEmpty(PO_Number.Text) || string.IsNullOrEmpty(HeadCode.Text) || string.IsNullOrEmpty(Manufacture.Text) || string.IsNullOrEmpty(Description.Text) || string.IsNullOrEmpty(Invoice.Text))
{
    Response.Write("<script>alert('Please Fill All Fields!')</script>");
    return; // 加上return终止后续逻辑
}

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