- 1
- 2
- 3
- 4
- 5
- 6
- 7
if ((ObjectType)value == ObjectType.Undefined)
{
return string.Empty;
}
{
return ((ObjectType)value).GetDescriptionAttribute();
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+139
if ((ObjectType)value == ObjectType.Undefined)
{
return string.Empty;
}
{
return ((ObjectType)value).GetDescriptionAttribute();
}
Как будто не хватает чего-то..
SSSandman 03.09.2013 17:22 # +2
guest 03.09.2013 17:31 # +3
neeedle 04.09.2013 05:55 # +2
bormand 04.09.2013 06:52 # 0
neeedle 04.09.2013 06:54 # 0
C# позволяет при этом сократить кол-во кода.
bormand 04.09.2013 07:56 # 0
Спасибо кэп. Я в курсе, что директивы else тут нет, и именно поэтому и говорю, что в строках 5 и 7 скобки не нужны.
> C# позволяет при этом сократить кол-во кода.
Звучит прям как фраза из рекламы.
И си и с++ и яваскрипт, да что уж тут говорить, даже паскаль с его begin-end, да и почти любой другой язык с блоками позволят это сделать. Это не какое-то магическое свойство c# :)
Но как я и сказал выше, в стайлгайде может быть указано, что после if'ов и while'ов скобки обязательны.
neeedle 04.09.2013 07:57 # −2
bormand 04.09.2013 07:58 # +2
neeedle 04.09.2013 07:59 # −3
bormand 04.09.2013 08:12 # +3
Вот лучше уж говностайлгайд, чем такой трешак. Собственно ради этого стайлгайды и принимают, не от хорошей жизни же. Ну и для предотвращения холиваров в духе "Вася, ты почему скобки ставишь, я вот их на дух не переношу.".
P.S. Вот я более чем уверен, что тут найдутся несколько человек, которые считают, что не ставить скобки - говно. И этот тред превратится в срач :)
neeedle 04.09.2013 09:22 # −3
;)
roman-kashitsyn 04.09.2013 09:32 # +2
vistefan 05.09.2013 17:13 # 0
Теперь кричат: О ужасный гайдлайн, он заставляет меня ставить две дополнительные скобки, когда я всего лишь хочу добавить ещё один оператор в ветку кода, а иногда и вовсе забываешь, и собираешь код с говнобагой и лишним индентом.
taburetka 03.09.2013 17:55 # 0
save2love 03.09.2013 18:16 # 0
taburetka 03.09.2013 18:24 # 0
Qwertiy 03.09.2013 21:28 # 0
save2love 03.09.2013 21:45 # 0
guest 03.09.2013 19:55 # −1
Говнокод. Надо так:
Qwertiy 03.09.2013 21:26 # 0
А вообще, я за "".
anonimb84a2f6fd141 04.09.2013 02:26 # 0
kegdan 04.09.2013 05:29 # 0
bormand 04.09.2013 06:51 # +1
http://ideone.com/pHTr47
Qwertiy 04.09.2013 08:58 # +1
А ещё, они вроде в одном из обновлений пофиксили поверение при присвыивании в String.Empty через рефлексию...
По поводу вызова свойства - проверял - разницы по времени выполнения нет.
Qwertiy 04.09.2013 09:01 # 0
bormand 04.09.2013 09:32 # 0
kegdan 05.09.2013 20:11 # 0
bormand 05.09.2013 20:13 # 0
guest 06.09.2013 06:46 # 0
Qwertiy 06.09.2013 20:36 # 0
anonimb84a2f6fd141 06.09.2013 18:55 # 0
bormand 06.09.2013 19:27 # 0
Но, в отличие от джавы, в шарпе оператор == для строк это не сравнение ссылок, а алиас для Equals(). Поэтому a == "foo" там вполне юзабельно.
Реквестирую поправки от знатоков си-с-решеткой.
kegdan 06.09.2013 19:33 # 0
Lennis 04.09.2013 09:10 # 0