Говнокод

- (void) setLevel:(NSInteger)level {
    if (level > (long)[_ranksDictionary count] - 1) level = (long)[_ranksDictionary count] - 1;
    if (level < 0) level = 0;

    _level = level;
    
    self.currentRank = [_ranksDictionary objectForKey:[NSNumber numberWithInteger:_level]];
    
    if (rankDataItem.integerValue != -_level)
        rankDataItem.integerValue = _level;
}

Я совсем хуевый?

Запостил:

ExT, 28.01.2015 (Updated 28.03.2018)

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

nshipster 12 лет ago # 0

rankDataItem.integerValue = _level -> это какая-то странная магия
Ответить
- pr0gl 11 лет ago # 0 ⇈
  
  Грустно слышать это от человека с таким ником
  Ответить
guest 12 лет ago # 0
Можно
```
self.currentRank = [_ranksDictionary objectForKey:[NSNumber numberWithInteger:_level]];
```
заменить на
```
self.currentRank = _ranksDictionary[@(_level)];
```
Ответить
- ExT 12 лет ago # 0 ⇈
  
  Спасибо, весь этот участок кода уже переправлен.
  Ответить
pr0gl 11 лет ago # 0
Почему бы не писать так?
```
- (void)setLevel:(NSInteger)level
{
        if(level >= self.ranksDictionary.count)
                level = self.ranksDictionary.count - 1;
        else if(level < 0)
                level = 0;

        _level = level;

        self.currentRank = self.ranksDictionary[@(_level)];

        if(rankDataItem.integerValue != -_level) // откуда взялось это  rankDataItem?
                rankDataItem.integerValue = _level;
}
```
Ответить
- Gerchicov-bp 11 лет ago # 0 ⇈
  
  многих интересует вопрос, почему бы не заменить:
  rankDataItem.integerValue = _level;
  
  на что-нибудь вроде:
  rankDataItem = @(_level);
  
  потому как rankDataItem.integerValue больше похож на геттер, а не свойство (даже если работает как свойство)
  Ответить
  - guest 11 лет ago # 0 ⇈
    
    >> больше похож на A, а не B (даже если работает как A)
    - так про шо угодно в обжектив си можно сказать, в общем-то
    Ответить
Desktop 3 года ago # 0

я б вообще последний иф не писал, смысл?
Ответить
- Desktop 3 года ago # 0 ⇈
  
  да и integerValue это read-only
  Ответить