C#字符串分割报错求助：自定义分割逻辑运行异常排查

首先，你的代码能读取内容但报错，主要有几个核心问题：

1. 直接触发报错的根源：索引越界+类型不匹配 #

你代码里的这行是明显的错误：

if (string1[i + 2] == (string1.Length - 3))

• 左边是char类型（字符串的单个字符），右边是int类型（字符串长度减3），两者类型不兼容，编译器其实应该已经给你警告了；
• 当循环到字符串末尾附近时，i + 2会超出字符串的最大索引范围（字符串索引从0到string1.Length-1），比如当i = string1.Length - 2时，i+2 = string1.Length，直接触发IndexOutOfRangeException。

2. 部分变量（part）的更新逻辑完全错误 #

你只在part == 2时遇到冒号才让part +=1，这意味着：

• 第一个冒号出现时，part还是1，不会切换到2；
• 第二个冒号出现时，part变成3；
• 第三个冒号出现时，part已经是3，不会再递增到4，导致word4永远不会被赋值。

3. 字符串拼接效率低下（非报错但需优化） #

直接用word1 = word1 + string1[i]这种方式拼接字符串，每次都会创建新的字符串对象，当字符串较长时性能很差，建议用StringBuilder来处理。

修复方案 #

方案一：用内置Split方法（推荐，简洁不易出错） #

因为你的字符串格式是固定的word1:word2:word3:word4，直接用Split方法分割是最省心的：

string string1 = "你的目标字符串";
string[] parts = string1.Split(':');

// 先判断分割后的部分数量是否符合预期，避免索引越界
if (parts.Length == 4)
{
    string word1 = parts[0];
    string word2 = parts[1];
    string word3 = parts[2];
    string word4 = parts[3];
    // 这里可以使用提取到的变量
}
else
{
    // 处理字符串格式不符合要求的情况，比如提示错误
}

方案二：手动修复循环逻辑（如果一定要自己实现分割） #

如果你想手动实现循环分割的逻辑，修正后的代码如下：

string string1 = "你的目标字符串";
string word1 = "";
string word2 = "";
string word3 = "";
string word4 = "";
int part = 1;
StringBuilder sb = new StringBuilder(); // 用StringBuilder提升拼接效率

for (int i = 0; i < string1.Length; i++)
{
    char currentChar = string1[i];
    // 遇到冒号且还没到最后一部分时，切换part并保存当前内容
    if (currentChar == ':' && part < 4)
    {
        switch (part)
        {
            case 1:
                word1 = sb.ToString();
                break;
            case 2:
                word2 = sb.ToString();
                break;
            case 3:
                word3 = sb.ToString();
                break;
        }
        sb.Clear();
        part++;
    }
    else
    {
        sb.Append(currentChar);
    }
}

// 处理最后一部分word4
word4 = sb.ToString();

这个版本修复了part的更新逻辑，用StringBuilder优化了拼接，同时避免了索引越界的问题。

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