1

У меня есть цикл:

string[] lines = File.ReadAllLines(@"F:\VS Projects\Со звёздочкой\diceware.txt");
for ( int i = 0; i <= lines.Length; i++)
{
                char[] textByCharacter = lines[i].ToCharArray();
                int number = object1.CharacterCheck(textByCharacter[0]);

                if (number == 1)
                {
                    textBox1.AppendText(lines[i] + "\t\t");

                }

                if (number == 2)
                {
                    textBox2.AppendText(lines[i] + "\t\t");
                }

                if (number == 3)
                {
                    textBox3.AppendText(lines[i] + "\t\t");
                }

                if (number == 4)
                {
                    textBox4.AppendText(lines[i] + "\t\t");
                }

                if (number == 5)
                {
                    textBox5.AppendText(lines[i] + "\t\t");
                }

                if (number == 6)
                {
                    textBox6.AppendText(lines[i] + "\t\t");
                }
            }

В этом цикле у меня по очереди заполняется каждый из шести textBox. Как реализовать, чтоб они заполнялись одновременно? Метод определения переменной number:

public int CharacterCheck(char character)
        {
            string symbol = System.Convert.ToString(character);

            if (symbol == "1")
            {
                transmitsTheNumber = 1;
            }

            if (symbol == "2")
            {
                transmitsTheNumber = 2;
            }

            if (symbol == "3")
            {
                transmitsTheNumber = 3;
            }

            if (symbol == "4")
            {
                transmitsTheNumber = 4;
            }

            if (symbol == "5")
            {
                transmitsTheNumber = 5;
            }

            if (symbol == "6")
            {
                transmitsTheNumber = 6;
            }

            return transmitsTheNumber;
        }
3
  • Что значит одновременно? Типа сначала вычислить все слова, а потом готовый результат записать в текстбоксы, чтобы было не видно, как они наполняются, так чтоли?
    – aepot
    4 ноя 2020 в 11:06
  • Можно за один раз определить все слова, а потом одновременно чтоб они записывались в textBox'ы 4 ноя 2020 в 11:27
  • Кстати, для чего вам \t\t? Если для перевода на новую строку за счет небольшой ширины текстбокса, то попробуйте \r\n, это и есть сам переход на новую строку, или есть переменная окружения, содержащая то же самое Environment.NewLine.
    – aepot
    4 ноя 2020 в 11:42

1 ответ 1

1

Во-первых, кучу однотипных if-else можно оформить через switch-case. Во-вторых, можно сделать лучше.

public int CharacterCheck(char character)
{
    if (int.TryParse(character.ToString(), out int digit) && digit > 0 && digit < 7)
    {
        return digit;
    }
    return 0;
}

К символам в строке можно обращаться напрямую без преобразования в массив символов.

string[] results = Enumerable.Repeat(string.Empty, 6).ToArray(); // массив пустых строк
for (int i = 0; i < lines.Length; i++)
{
    int number = object1.CharacterCheck(lines[i][0]);
    if (number != 0)
        results[number - 1] += lines[i] + "\t\t";
}
textBox1.Text = results[0];
textBox2.Text = results[1];
textBox3.Text = results[2];
textBox4.Text = results[3];
textBox5.Text = results[4];
textBox6.Text = results[5];

Многпоточные вычисления здесь имеют очень мало смысла, потому что, как вы сказали, 6000 строк - это очень мало, и выполнится очень быстро. Во-вторых здесь можно еще ускорить код. В-третьих у вас было 6000 обращений к текстбоксам, стало в 1000 раз меньше. И в-четвертых отрисовка формы может быть ускорена с помощью двойной буферизации.

10
  • Это все круто, но оно все равно будет заполнять сначала один текст бокс, потом второй и так далее. У меня в текстовом документе сначала записаны слова с индексом 1, потом 2, потом 3 и т.д. Слов очень много, порядка 6000 4 ноя 2020 в 11:34
  • @ЯрославОвчар вы попробовали?
    – aepot
    4 ноя 2020 в 11:44
  • Суть стоит ни во времени, ни в практичности. Задача стоит просто сделать так, чтоб они заполнялись одновременно. Но Ваш код я тоже попробую, спасибо. 4 ноя 2020 в 13:26
  • @ЯрославОвчар в UI приложениях для работы с элементами управления у вас только один поток, технически невозможно делать с интерфейсом 100 дел одновременно. А чтобы получить визуально результат одновременной отрисовки, вы можете либо оптимизировать код, как я показал выше, либо останавливать отрисовку на время наполнения контролов данными плюс использовать буферизацию отрисовки. Параллельность здесь не играет большой роли, я и так в приведенном выше примере свел взаимодействие с интерфейсом к минимуму. С большой вероятностью вы получите именно то что хотите. Если нет, пишите, будем разбираться.
    – aepot
    4 ноя 2020 в 14:01
  • @ЯрославОвчар в любом случае, вы сначала попробуйте предложенное решение, а потом ставьте диагноз, но не наоборот.
    – aepot
    4 ноя 2020 в 14:05

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.