c# - 如何在c#Sharp中将char数组中的字符向左和向右移动

Question

伙计们！ 我有这种情况：给定一个字符串和另外两个字符 X 和 Y。将所有 X 字符移动到字符串的开头，将所有 Y 字符移动到字符串的末尾。 字符串中其他字符的顺序保持不变。 我写了两个函数MoveCharsLeft和MoveCharsRight来将 X 向左移动，Y 向右移动，但是有一个错误System.IndexOutOfRangeException: 'Index was outside the bounds of the array。 ' 在这行代码char toReplace = splitText[charToCheck]; 而我不知道如何处理它，以解决这个练习。 你们能帮我解决这个问题吗，功能应该如何？

static void Main()
    {
        string text = Console.ReadLine();
        char[] splitText = text.ToCharArray();
        string firstLetter = Console.ReadLine();
        char[] firstChar = firstLetter.ToCharArray();
        string secondLetter = Console.ReadLine();
        char[] secondChar = secondLetter.ToCharArray();
        char one = firstChar[0];
        char two = secondChar[0];
        Console.WriteLine(CheckChars(splitText, one, two));
        Console.ReadLine();
    }

    static char[] CheckChars(char[] splitText, char one, char two)
    {
        for (char letter = 'a'; letter <= 'z'; letter++)
        {
            if (Array.IndexOf(splitText, one) > -1)
            {
                MoveCharsLeft(splitText, one);
            }

            if (Array.IndexOf(splitText, two) > -1)
            {
                MoveCharsRight(splitText, two);
            }
        }

        return splitText;
    }

    static void MoveCharsLeft(char[] splitText, char charToCheck)
    {
        char toReplace = splitText[charToCheck];
        char currentLetter = splitText[0];
        for (int i = 0; i <= charToCheck; i++)
        {
            char temporary = splitText[i];
            splitText[i] = currentLetter;
            currentLetter = temporary;
        }

        splitText[0] = toReplace;
    }

    static void MoveCharsRight(char[] splitText, char charToCheck)
    {
        char toReplace = splitText[charToCheck];
        char currentLetter = splitText[-1];
        for (int i = 0; i <= charToCheck; i++)
        {
            char temporary = splitText[i];
            splitText[i] = currentLetter;
            currentLetter = temporary;
        }

        splitText[-1] = toReplace;
    }

Answer 1

你没有检查你的界限。 我会说CheckChars方法是多余的，在MoveCharsLeft和MoveCharsRight您不会检查您是否仍在splitText 。

这是我将如何实现MoveCharsLeft ，我希望它会引导您朝着正确的方向前进。

char[] MoveCharactersLeft(char[] text, char character) {
    var lowerCharacter = char.ToLower(character);
    var left = new List<char>();
    var right = new List<char>();
    
    foreach (var letter in text) {
        if (char.ToLower(letter) == lowerCharacter) {
            left.Add(letter);
            continue;
        }
        
        right.Add(letter);
    }
    
    left.AddRange(right);
    return left.ToArray();
}

这是MoveCharactersRight ：

char[] MoveCharactersRight(char[] text, char character) {
    var lowerCharacter = char.ToLower(character);
    var left = new List<char>();
    var right = new List<char>();
    
    foreach (var letter in text) {
        if (char.ToLower(letter) == lowerCharacter) {
            right.Add(letter);
            continue;
        }
        
        left.Add(letter);
    }
    
    left.AddRange(right);
    return left.ToArray();
}

现在你有了这两个，你只需在MoveCharactersRight的结果上调用MoveCharsLeft和MoveCharsLeft 。

Answer 2

您可以就地交换它们，尽管它需要大量复制。 这是一个将字符从数组中的一个位置移动到另一个位置的函数。 你可以在网上和SO上找到几个这样的例子。

    public static void Move(char[] buffer, int from, int too)
    {
        if (from == too)
            return;

        char temp = buffer[from];

        if (too < from)
        {
            Array.Copy(buffer, too, buffer, too + 1, from - too);
        }
        else
        {
            Array.Copy(buffer, from + 1, buffer, from, too - from);
        }

        buffer[too] = temp;
    }

然后你只需要遍历数组测试字符，测试它们并将它们移动到正确的位置。 这是一个函数来做到这一点：

    public static void ArrangeChars(char[] buffer, char left, char right)
    {
        int leftIndex = 0;
        int rightIndex = buffer.Length - 1;

        int i = 0;

        while(i <= rightIndex)
        {
            char ch = buffer[i];

            if (ch == left && i > leftIndex)
                Move(buffer, i, leftIndex++);
            else if (ch == right && i < rightIndex)
                Move(buffer, i, rightIndex--);
            else
                ++i;
        }
    }

像这样称呼它：

    static void Main(string[] _)
    {
        char[] data = "test me and move".ToCharArray();

        Console.WriteLine(new string(data));

        ArrangeChars(data, 'e', 't');

        Console.WriteLine("Move 'e' to start, move 't' to end");

        Console.WriteLine(new string(data));
    }

通常的警告适用 - 这还没有经过很好的测试，如果你使用它，你应该检查所有的边缘情况。