- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
public string removeSpecialCharacters(string orig) {
string rv;
rv = orig.Replace("\\", " ");
rv = rv.Replace("(", " ");
rv = rv.Replace(")", " ");
rv = rv.Replace("/", " ");
rv = rv.Replace("-", " ");
rv = rv.Replace(",", " ");
rv = rv.Replace(">", " ");
rv = rv.Replace("<", " ");
rv = rv.Replace("-", " ");
rv = rv.Replace("&", " ");
rv = rv.Replace("'", "");
rv = rv.Replace("\u2019", ""); // smart forward (possessive) quote.
rv = rv.Replace(" ", "");
rv = rv.Trim(' '); // Remove leading and trailing spaces.
return (rv);
}
guest 27.10.2009 15:20 # 0
guest 27.10.2009 16:34 # 0
fade 27.10.2009 16:40 # 0
guest 27.10.2009 17:48 # 0
fade 27.10.2009 18:05 # +1
"Примечание: Данный метод не изменяет значение текущего экземпляра. Вместо этого возвращается новая строка, в которой все вхождения oldValue заменяются на newValue."
В этом говнокоде создается как минимум 14 (!!) копий исходной строки. И если вызывать его для большого текста или для большого листа строк - перфоманс приложения (асп.нет страницы) сделает торжественное сеппуку.
guest 27.10.2009 23:55 # −1
fade 28.10.2009 15:30 # 0
Вобщем ахтункъ, да хД
guest 04.11.2009 15:58 # 0
Код читается и всем понятно что он делает.
В остальном коде наверняка есть более "узкие" места - их и надо доводить до приемлимой производительности. Профайлер в руки и вперед.
guest 04.11.2009 18:43 # 0
Да, не надо.
Надо изначально писать нормальный код.
StringBuilder как раз для таких целей предназначен, и его использование никак не ухудшает производительность труда или читаемость.
Про регэкспы уже молчу.
guest 06.11.2009 07:35 # 0
guest 28.10.2009 08:35 # 0
return orig.Replace("\\", " ")..Replace("(", " ").Replace(")", " ").Replace("/", " ").Replace("-", " ") .......
guest 31.10.2009 04:14 # −1
guest 06.11.2009 20:46 # 0
guest 02.11.2009 15:34 # +1
вариант с регексом не предлагать, так как медленный.
guest 02.11.2009 19:25 # 0
guest 04.11.2009 19:31 # 0
guest 04.11.2009 19:34 # 0
guest 05.11.2009 18:38 # +1
a ne cajetsea sto
rv = rv.Replace("(", " ");
.....................
rv = rv.Replace(" ", "");
==
rv = rv.Replace("(", ""); ???
a ecio lu4e rv = rv.Replace("(", string.Empt);
guest 22.11.2009 00:19 # 0
Ужас.