Строка Конкатенация/Перезапись?

Question

Это программа, которая читает в CSV-файле, добавляет значения в класс словаря и затем анализирует строку в текстовом поле, чтобы узнать, соответствует ли какое-либо из слов словарю. Он заменит аббревиатуры (LOL, ROFL и т. Д.) На их настоящие слова. Он соответствует строкам, разбивая введенный текст на отдельные слова.

public void btnanalyze_Click(object sender, EventArgs e) 
    { 
     var abbrev = new Dictionary<string, string>(); 
     using (StreamReader reader = new StreamReader("C:/Users/Jordan Moffat/Desktop/coursework/textwords0.csv")) 
     { 
      string line; 
      string[] row; 

      while ((line = reader.ReadLine()) != null) 
      { 
       row = line.Split(','); 

       abbrev.Add(row[0], row[1]); 
       Console.WriteLine(abbrev); 
      } 
     } 


     string twitterinput; 
     twitterinput = ""; 
     // string output; 
     twitterinput = txtInput.Text; 
     { 
      char[] delimiterChars = { ' ', ',', '.', ':', '\t' }; 

      string text = twitterinput; 

      string[] words = twitterinput.Split(delimiterChars); 


     string merge; 
      foreach (string s in words) 
      { 
       if (abbrev.ContainsKey(s)) 
       { 
        string value = abbrev[s]; 


        merge = string.Join(" ", value); 
       } 

        if (!abbrev.ContainsKey(s)) 
        { 
         string not = s; 
         merge = string.Join(" ", not); 
        } 

         ; 
         MessageBox.Show(merge); 

       } 

Проблема до сих пор является то, что конечная строка выводится в текстовое поле, но печатает только последнее слово, как он переписывает. Это университетское задание, поэтому я ищу толчок в правильном направлении, а не фактический ответ. Большое спасибо!

Answer 1

Это проблема:

string not = s; 
merge = string.Join(" ", not); 

Вы просто присоединиться один элемент (последний) с космическим разделителем, таким образом переписывая то, что вы ранее введенным в merge.

Если вы хотите придерживаться строки, вам нужно использовать Concat, чтобы добавить новое слово на выход, хотя это будет медленным, поскольку вы каждый раз воссоздаете строку. Для создания результата будет более эффективным использование StringBuilder.

Если ваше назначение требует, чтобы вы использовали Join для создания выходных данных, тогда вам нужно будет заменить целевые слова в массиве words, когда вы будете перебирать их. Однако для этого вам понадобится использовать какой-либо другой механизм циклизации, чем foreach, так как это не позволяет вам изменять массив, который вы зацикливаете.

Answer 2

string.Join() берет набор строк, объединяет их и возвращает результат. Но в вашем случае коллекция содержит только один элемент: value, или not.

Чтобы сделать код работы, вы могли бы использовать что-то вроде:

merge = string.Join(" ", merge, value); 

Но из-за того, как струны работать, это будет довольно медленно, так что вы должны использовать вместо StringBuilder.

Answer 3

Лучше пользователя StringBuilder класса для этой цели

http://msdn.microsoft.com/en-us/library/system.text.stringbuilder.aspx