Проблема с удалением символов из строки

Question

Я видел много ответов о том, как обращаться с редактированием символов внутри строки с помощью C#.

Однако мне нужен ответ, который учитывает только метод класса string remove (также, чтобы избежать использования строкового редактора).

Это только потому, что я вижу, что проблема с этим конкретным кодом, который, похоже, не избавляет от среднего пробела в телефонном номере. Код также удаляет любой другой символ, который не является числом.

Я до сих пор не смог понять, почему пробелы или два смежных пробела не будут удалены. Я подозреваю, что самозадача одной строки может быть проблемой, но я не уверен.

//leave only the digits 
for (int i = 0; i < enteredPhone.Length; i++) 
{ 
    switch (enteredPhone[i]) 
    { 
     case '0': 
     case '1': 
     case '2': 
     case '3': 
     case '4': 
     case '5': 
     case '6': 
     case '7': 
     case '8': 
     case '9': 
      break; 

     default: 
      //gets rid of any other type of character 
      enterPhone = enteredPhone.Remove(i, 1); 
      break; 
     } 
    } 

Заранее спасибо.

Answer 1

«почему пробелы или два смежных пробела не будут удалены» - потому что все показатели персонажа после i уменьшены на 1, когда вы .Removei.

Прогулка по низу назад - от Length-1 до 0.

Answer 2

Попробуйте вычесть 1, когда вы удалите caracter. Потому что, если вы удалите caracter в позиции 5 и i ++, значение i теперь равно 5, а следующий символ не будет найден в цикле. Как это:

default: 
     //gets rid of any other type of character 
     enterPhone = enteredPhone.Remove(i, 1); 
     i--; 
     break; 

Answer 3

Игнорирование ваши странные требования:

Regex.Replace(inputString, @"\D", string.Empty) 

Answer 4

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

int index = 0; 
while(index < enteredPhone.Length) 
{ 
    switch (enteredPhone[index]) 
    { 
     case '0': 
     case '1': 
     case '2': 
     case '3': 
     case '4': 
     case '5': 
     case '6': 
     case '7': 
     case '8': 
     case '9': 
      index++; 
      break; 

     default:     
      enteredPhone = enteredPhone.Remove(index, 1); 
      break; 
    } 
} 

Также вы можете использовать Char.IsDigit, чтобы упростить этот цикл:

int index = 0; 
while(index < enteredPhone.Length) 
{ 
    if (Char.IsDigit(enteredPhone[index])) 
    { 
     index++; 
     continue; 
    } 

    enteredPhone = enteredPhone.Remove(index, 1); 
} 

Answer 5

var onlyDigits = new string(enteredPhone.Where(Char.IsDigit).ToArray()); 

должен быть немного более эффективным, чем регулярное выражение.

Answer 6

Вы также можете попробовать. Но я считаю, что решения Linq или Regex - лучшие варианты.

//leave only the digits 
for (int i = enteredPhone.Length - 1; i >= 0; i--) 
{ 
    if (!Char.IsDigit(enteredPhone[i])) 
    { 
     enterPhone = enteredPhone.Remove(i, 1); 
    } 
}