C#文件名增量逻辑异常：123456.9被错误增量为123456.110等

需求说明 #

• 输入123456，若对应文件已存在，应自动增量为123456.1.pdf
• 输入123456.1，若对应文件已存在，应自动增量为123456.2.pdf
• 输入123456.11，若对应文件已存在，应自动增量为123456.12.pdf

问题描述 #

当已存在123456.9.pdf时，程序错误生成123456.110或123456.1111这类不符合预期的文件名，需排查代码问题。

原始代码 #

MyWO = WO.Text; 
string plusOne = MyWO.Substring(MyWO.Length - 1);
string FnlName;
int cnt;
if (MyWO.Length == 8 ) { cnt = Convert.ToInt32(plusOne) + 1; }
else { cnt = 1; }

while (File.Exists(savePath + MyWO + ".pdf"))
{
    if (MyWO.Length == 6)
    {
        FnlName = (MyWO + "." + cnt.ToString());
    }
    else
    {
        string Fnl = MyWO.Remove(MyWO.Length - 1, 1);
        FnlName = (Fnl + cnt.ToString());
    }
    cnt++;
}

错误原因分析 #

1. 增量数字提取逻辑错误：
   你用MyWO.Substring(MyWO.Length - 1)只取文件名的最后一个字符作为增量数，这在增量是多位数时完全失效。比如123456.11会被提取最后一个1，而非完整的11；对于123456.9，提取逻辑看似没问题，但后续修改逻辑会出错。

2. 长度判断逻辑过于僵化：
   用MyWO.Length == 8判断是否为带增量的文件名，仅适配123456.1这种8位格式，一旦增量是两位数（比如123456.11是9位），这个判断就会失效，导致cnt被错误初始化为1。

3. 循环逻辑错误：

   • 循环判断的是原始输入文件名对应的文件是否存在，而非新生成的FnlName是否存在，导致只要原始文件存在，循环就会无限执行，cnt持续递增。
   • 修改文件名时，用MyWO.Remove(MyWO.Length - 1, 1)只删除最后一个字符，比如123456.9会被改成123456.，再加上不断递增的cnt，就会生成123456.10、123456.11...直到123456.110这类错误文件名。

修正方案 #

核心思路是正确拆分文件名的前缀和增量部分，基于完整的增量数字进行递增，同时循环判断新生成的文件名是否存在：

string inputWO = WO.Text;
string prefix;
int currentIncrement;

// 拆分前缀和增量数字
string[] parts = inputWO.Split('.');
if (parts.Length == 1)
{
    // 无增量，前缀是输入内容，初始增量为1
    prefix = parts[0];
    currentIncrement = 1;
}
else
{
    // 有增量，前缀是第一部分，增量转成整数后加1
    prefix = parts[0];
    int.TryParse(parts[1], out currentIncrement);
    currentIncrement += 1;
}

string finalName = inputWO;
// 循环判断新文件名是否存在，直到找到可用的
while (File.Exists(Path.Combine(savePath, finalName + ".pdf")))
{
    finalName = $"{prefix}.{currentIncrement}";
    currentIncrement++;
}

// 最终可用的文件名是finalName + ".pdf"

修正后逻辑说明 #

1. 通过Split('.')将输入文件名拆分为前缀和增量部分，不管增量是几位都能正确提取。
2. 循环判断的是当前生成的新文件名是否存在，避免无效循环。
3. 每次循环基于前缀和递增后的数字生成新文件名，完全符合需求中的增量规则。

内容的提问来源于stack exchange，提问作者s.d.c321