- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
if( str.ToUpper() == "....." )
{
properties["DCS"] = "0x00";
msg = new Message( str, A, B, properties, text );
}
else
{
properties["DCS"] = "0x00";
msg = new Message( str, A, B, null, text );
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+133
if( str.ToUpper() == "....." )
{
properties["DCS"] = "0x00";
msg = new Message( str, A, B, properties, text );
}
else
{
properties["DCS"] = "0x00";
msg = new Message( str, A, B, null, text );
}
В биореактор Луговского!
taburetka 25.03.2013 16:07 # 0
SSSandman 25.03.2013 16:29 # 0
msg = new Message( str, A, B, str.ToUpper() == "....." ? properties : null, text );
норм?
scriptin 25.03.2013 18:29 # 0
guest 26.03.2013 04:45 # 0
Я вижу, что есть дублирование properties["DCS"] = "0x00";
Я вижу, что есть какая-то проверка на определенную статичную строку - str.ToUpper() == "....."
Ну и вижу то, что только для этой строки мы выводим это properties["DCS"] = "0x00";
А зачем все это я не понимаю. Поясните пожалуйста.
guest 26.03.2013 05:14 # 0
taburetka 25.03.2013 21:12 # 0
bot 26.03.2013 21:04 # 0
Ну ничего. Пусть поглубже подышит.
guest 29.03.2013 05:09 # 0