String.Replace не работает, как ожидалось

Question

У меня есть программа

public void TestMethod2() 
    { 
     string[] keywords = 
     { 
      "SELECT", "FROM", "WHERE", "GROUP", "HAVING", "ORDER", "LEFT", "RIGHT", "JOIN", "INNER", "OUTER", "ASC", 
      "DESC", "AND", "OR","IN", "BETWEEN", "BY", "NOT", "ON", "AS", "CASE", "WHEN", "ELSE", "UPDATE", "SET" 
     }; 

     var actualString = "SELECT * FROm A Join B On C in D case e join t left outer join inner join right join"; 

     foreach (var text in actualString.Split(' ')) 
     { 
      var isExists = keywords.Any(x => x.Equals(text, StringComparison.OrdinalIgnoreCase)); 

      if (!isExists) 
      { 
       continue; 
      } 

      actualString = actualString.Replace(text, text.ToUpper()); 
     } 


     var expectedString = "SELECT * FROM A JOIN B ON C IN D CASE e JOIN t LEFT OUTER JOIN INNER JOIN RIGHT JOIN"; 

    } 

Я новичок в C#. Я не понимаю, почему метод Replace() не работает должным образом. Он показывает выход SELECT * FROM A JOIN B ON C IN D СЛУЧАЙ е РЕГИСТРИРУЙТЕСЬ т LEFT OUTER JOIN INNER JOIN RIGHT JOIN

Может кто-то пожалуйста просветить меня почему Replace() ведет себя, как это? Спасибо заранее.

Answer 1

Он ведет себя так, как ожидалось. Вот вывод построчно:

 
[SELECT] -> SELECT * FROm A Join B On C in D case e join t left outer join inner join right join 
[FROm] -> SELECT * FROM A Join B On C in D case e join t left outer join inner join right join 
[Join] -> SELECT * FROM A JOIN B On C in D case e join t left outer join inner join right join 
[On]  -> SELECT * FROM A JOIN B ON C in D case e join t left outer join inner join right join 
[in]  -> SELECT * FROM A JOIN B ON C IN D case e joIN t left outer joIN INner joIN right joIN 
[case] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t left outer joIN INner joIN right joIN 
[join] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t left outer joIN INner joIN right joIN 
[left] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT outer joIN INner joIN right joIN 
[outer] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN right joIN 
[join] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN right joIN 
[inner] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN right joIN 
[join] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN right joIN 
[right] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN RIGHT joIN 
[join] -> SELECT * FROM A JOIN B ON C IN D CASE e joIN t LEFT OUTER joIN INner joIN RIGHT joIN 

Вы можете видеть, что, когда в заменяется на В, в строке 5; затем присоединиться к заменяется на РЕГИСТРИРУЙТЕСЬ

Но тогда, что случилось, что ваш призыв к Keywords.Any работает правильно, и соответствует присоединиться и РЕГИСТРИРУЙТЕСЬ, однако, заменить вызов все еще чувствительный к регистру, и замена не состоится.

Replace is C# чувствителен к регистру и, к сожалению, .NET не предлагает версию, не учитывающую регистр; но есть решения, плавающие вокруг.

Проверить this link для замены нечувствительного к регистру.

Answer 2

Если вы отлаживаете его, вы заметите, что ваше ключевое слово «в» заменяет второе соединение, поэтому вы получаете joIN. Позже ваша текстовая переменная попытается заменить «join» на «JOIN», но не найдет «join», потому что часть ее была в верхнем регистре.

Answer 3

Проблема в том, что вы уже замените IN в соединении. Поэтому сравнение не выполняется, если вы хотите заменить соединение с помощью JOIN. (joIN! = join)

Answer 4

Есть четкое представление из ответов here.Finally решить, что, изменяя Еогеасп с для цикла. Спасибо вам всем.

public void TestMethod2() 
{ 
    string[] keywords = 
    { 
     "SELECT", "FROM", "WHERE", "GROUP", "HAVING", "ORDER", "LEFT", "RIGHT", "JOIN", "INNER", "OUTER", "ASC", 
     "DESC", "AND", "OR","IN", "BETWEEN", "BY", "NOT", "ON", "AS", "CASE", "WHEN", "ELSE", "UPDATE", "SET" 
    }; 

    var actualString = "SELECT * FROm A Join B On C in D case e join t left outer join inner join right join"; 

    var cnt = sqlTest.Split(' ').Count(); 
    for (int i = 0; i < cnt; i++) 
    { 
     var text = sqlTest.Split(' ')[i]; 
     var isExists = keywords.Any(x => x.Equals(text, StringComparison.OrdinalIgnoreCase)); 

     if (!isExists) 
     { 
      continue; 
     } 

     actualString = actualString.Replace(text, text.ToUpper()); 
    } 


    var expectedString = "SELECT * FROM A JOIN B ON C IN D CASE e JOIN t LEFT OUTER JOIN INNER JOIN RIGHT JOIN"; 

} 

Answer 5

Я предлагаю использовать Regex.Replace вместо того, чтобы просто String.Replace с

\bKeyWord\b 

узора с целью лечения подстроки в качестве ключевого слова, если и только если это целого слова:

select horror 
    from torro 
    where corner = 1 
    or ordain = 2 

следует преобразовать в

SELECT horror -- even if horror has two "or" 
    FROM torro -- even if torro has two "or" 
    WHERE corner = 1 -- even if corner contains "or" 
    OR ordain = 2 -- even if ordain contains "or" 

Реализация:

string[] keywords = { 
    "SELECT", "FROM", "WHERE", "GROUP", "HAVING", "ORDER", "LEFT", "RIGHT", 
    "JOIN", "INNER", "OUTER", "ASC", "DESC", "AND", "OR","IN", "BETWEEN", 
    "BY", "NOT", "ON", "AS", "CASE", "WHEN", "ELSE", "UPDATE", "SET" 
}; 

var actualString = 
    "SELECT * FROm A Join B On C in D case e join t left outer join inner join right join"; 

foreach (var keyword in keywords) 
    actualString = Regex.Replace(
    actualString, 
    @"\b" + Regex.Escape(keyword) + @"\b", 
    keyword.ToUpper(), 
    RegexOptions.IgnoreCase);