0

Есть RichTextBox, в который введен произвольный текст. При нажатии комбинации клавиш alt+Right хочу поочередно выделять каждое слово в тексте (последовательность символов от пробела до пробела). Добавил для этого KeyEventHandler, в котором прописал:

    private void hotKeys(object sender, KeyEventArgs e) {
            if (e.Alt && e.KeyCode == Keys.Right) {
                if (richTextBox.Focused)
                {
                    int pos = Regex.Match(richTextBox.Text, @"\s+").Index;
                    richTextBox.SelectionLength = pos;
                }
            }
        }
    

Но комбинация клавиш работает некорректно: запоминает в pos длину первого слова в тесте и далее подставляет его везде. Как я понял данное регулярное выражение ищет в тексте любое количество пробелов начиная с текущей позиции курсора. (Для начала текста нужно указать ^), а значит переменная pos просто не обновляется.

  1. Почему pos не обновляла значение?
  2. Нет ли в коде утечки памяти, ведь я создаю pos каждый раз при нажатии комбинации клавиш?
8
  • Никакой разницы между этими вариантами нет. Утечки памяти там тоже нет.
    – user176262
    6 окт 2020 в 13:20
  • @Igor Да, вы правы, еще раз проверил... Почему не обновляется значение переменной? Курсор же находится в новой позиции при повторном вызове метода 6 окт 2020 в 13:28
  • 2
    Что значит "не обновляется". Вы определяете ее значение совершенно независимо от позиции курсора. Если текст в richTextBox не меняется, то pos всегда будет присваиваться одно и то же значение.
    – user176262
    6 окт 2020 в 13:30
  • 1
    И да, в C# стандартные типы данных (managed) корректно удаляются из памяти при выходе из блока {...}, утечка памяти может быть только на unmanaged ресурсах.
    – CrazyElf
    6 окт 2020 в 13:32
  • 1
    @Konstantin_SH Вы путаете понятия. Указания где в тексте что-то искать для регулярок (например, с начала текста) не имеют никакого отношения к позиции курсора в элементе GUI. Вы же берёте просто весь текст из контрола: richTextBox.Text
    – CrazyElf
    6 окт 2020 в 14:17

1 ответ 1

0

Попробуйте так

   int pos = 0;
   private void hotKeys(object sender, KeyEventArgs e) {
        if (e.Alt && e.KeyCode == Keys.Right) {
            if (richTextBox.Focused)
            {
                pos += Regex.Match(richTextBox.Text.Substring(pos), @"\s+").Index;
                richTextBox.SelectionLength = pos;
            } else pos = 0;
        } 
    }

Правильно будет устанавливать позицию когда текс бокс получит фокус

6
  • Не скомпилируется.
    – user176262
    6 окт 2020 в 15:31
  • И Какая же ошибка? 6 окт 2020 в 15:34
  • Например - "использование неинициализированной локальной переменной pos".
    – user176262
    6 окт 2020 в 15:59
  • Ну переменную я тут не объявляю. Я просто показал как изменить код 6 окт 2020 в 16:08
  • Код не скомпилируется.
    – user176262
    6 окт 2020 в 16:11

Ваш ответ

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

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