- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
if(count($colors)>1){
if($i==2){
$col = $colors[1];
}else{
$col = $colors[0];
}
}else{
$col=$colors[0];
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+153
if(count($colors)>1){
if($i==2){
$col = $colors[1];
}else{
$col = $colors[0];
}
}else{
$col=$colors[0];
}
Вот где не надо - тьма тернарных операторов, а где их логичнее всего ожидать - вот такой огород из IF/ELSE
$col=$colors[(int)(count($colors)>1&&$i==2)];
https://www.nyse.com/quote/index/NYA
Google closure compiler может сделать «перл» даже из ECMAScript'а, причём совершенно автоматически.
Моя догадка - тут неявно предполагается, что цветов всегда не больше двух, причём индексируются они с единицы.
Если это действительно так, то при добавлении нового цвета всё посыпется. Поэтому правильный (с моей точки зрения) вариант:
http://govnokod.ru/16479#comment242321