- 1
- 2
- 3
- 4
- 5
foreach ($user as $field => $value) {
if ($field == 'username') {
$user[$field] = md5($value);
}
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+165
foreach ($user as $field => $value) {
if ($field == 'username') {
$user[$field] = md5($value);
}
}
Обучаю стажера, который целый год кодил на битрикс, частенько у него проскользает такой подход для обновления значений в ассоциативном массиве.
someone 18.11.2014 19:22 # +2
http://blogs.msdn.com/b/oldnewthing/archive/2011/12/27/10251210.aspx
3.14159265 18.11.2014 20:04 # +1
Антипаттерном я бы назвал скорее спорное решение, в котором зачастую сознательно отказываются от паттернов, допустим пишут свищ вместо создания фабрики.
Или любовь Тараса к structам с публичными полями. Само по себе это не есть плохо.
Анонимус 18.11.2014 22:03 # +1
А что за говнопатерн "бульщит"?
bormand 18.11.2014 23:11 # +1
Анонимус 18.11.2014 23:14 # +2
someone 18.11.2014 23:48 # +5
Очевидно же!
inkanus-gray 19.11.2014 01:23 # 0
Или так: http://ideone.com/dEPIoe
Xom94ok 18.11.2014 22:37 # +1
bormand 18.11.2014 23:13 # +1
kipar 19.11.2014 00:07 # +1
а, ну да, программы же нет. Не подходит.
bormand 19.11.2014 06:22 # +1
Мы ее здесь просто не видим. Она может быть в каком-то массиве.
niggerzilla 19.11.2014 13:01 # 0
bot 19.11.2014 21:36 # 0
Наверное, isset($user['username']); ну или array_key_exists('username', $user);
Анонимус 19.11.2014 23:21 # 0
inkanus-gray 18.11.2014 21:15 # +2
wvxvw 18.11.2014 21:49 # 0
bormand 18.11.2014 23:16 # +2
http://ideone.com/LA18Uj
inkanus-gray 19.11.2014 00:40 # 0
1024-- 19.11.2014 03:02 # 0
bormand 19.11.2014 06:41 # +1
0 == "хуй" и 0 == "",
но 0 != true
Твой пых. Твои правила.
Анонимус 19.11.2014 16:59 # +5
defecate-plusplus 19.11.2014 17:18 # +8
roman-kashitsyn 19.11.2014 17:19 # +7
inkanus-gray 19.11.2014 17:27 # 0
defecate-plusplus 19.11.2014 17:32 # 0
Анонимус 19.11.2014 17:32 # 0
guest 21.11.2014 17:16 # +1
Анонимус 21.11.2014 18:46 # +1
Кстати Python не имеет изначально отнощения к змее. Он про монти пайтон
bormand 21.11.2014 18:50 # 0
Но теперь даже Гвидо в этом убедили...
roman-kashitsyn 21.11.2014 18:54 # +2
Особенно хорошо это заметно на логотипе языка.
Анонимус 21.11.2014 18:56 # 0
roman-kashitsyn 21.11.2014 19:00 # +2
Анонимус 21.11.2014 19:01 # +1
roman-kashitsyn 21.11.2014 19:12 # +1
BOLD SARCASM
bormand 21.11.2014 19:12 # 0
guest 26.11.2014 11:13 # 0
Анонимус 19.11.2014 23:22 # 0
barbarbar 27.11.2014 15:17 # 0
Анонимус 27.11.2014 15:18 # 0
barbarbar 27.11.2014 15:29 # 0
1) Раньше были и другие условия в цикле. Либо предполагается, что в скором времени они добавятся. Вобщем, пример явно подправленный или вырван из контекста.
2) Программист действительно больной дебил.