Говнокод

List<Map<String, Object>> list;
...........
// берем первый элемент списка и сохраняем его
for(Map<String, Object> properties: list ) {
            map = properties;
            break;
 }

Запостил:

paralainer, 21.02.2011 (Updated 25.03.2018)

Комментарии (13) RSS

absolut 16 лет ago # 0

Нехватка typedef'а
Ответить
- Анонимус 16 лет ago # 0 ⇈
  
  лолшто?
  Ответить
  - absolut 16 лет ago # 0 ⇈
    
    Для Map<String, Object> задать говорящий тип.
    И для list тоже. Имя переменной.
    Ответить
    - Анонимус 16 лет ago # 0 ⇈
      
      говнокод не в этом
      Ответить
      - absolut 16 лет ago # 0 ⇈
        
        Я вижу в этом. Видите иначе - расскажите.
        Ответить
        
        Анонимус 16 лет ago # 0 ⇈
        
        гк в том, как получается первый элемент мапы
        Ваш К
        Ответить
        
        absolut 16 лет ago # 0 ⇈
        
        К, не мапы элемент, а листа ! 🙂 А так да, цикл одношаговый.
        Ответить
- Мистер Хэнки 16 лет ago # 0 ⇈
  
  скорее нехватка map = list.get(0)
  Ответить
hidespb 16 лет ago # 0

Какой внезапный break oO
Ответить
gegMOPO4 16 лет ago # 0
Лучше, если return внутри цикла. Вполне стереотипно и даже где-то красиво.

Можно, конечно, написать скучно:
```
if(!list.isEmpty())
    map = list.iterator().next();
```
Ответить
- paralainer 16 лет ago # 0 ⇈
  
  Там нельзя return там дальше шла обработка) list на empty до этого проверялся, его размер точно больше нуля =)
  Ответить
  - absolut 16 лет ago # 0 ⇈
    
    >его размер точно больше нуля =)
    0.5
    Ответить
  - gegMOPO4 16 лет ago # 0 ⇈
    
    Просто если бы в функции сразу шёл return -- это было бы красивее. У меня когда-то был соблазн так сделать.
    Ответить