1. Список говнокодов пользователя NAlexN

    Всего: 4

  2. Objective C / Говнокод #14250

    −100

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    -(void)savePictureWithId:(NSString*)pictureId{                       // pictureId - строка типа @"4629"
    Picture * picture = [pictures objectForKey:pictureId];           // pictures - словарик, хранит какие то (не все) доступные объекты Picture
    if(picture == nil)
        return;
    NSString * identif = picture.id;                                 // id найденного объекта, который всегда содержит строку совпадающую с параметром pictureId
    if((picture.viewed == NO) &&
       (NO == [viewedPicIdsPack containsObject:identif]))            // viewedPicIdsPack - специальный NSMutableArray для хранения Picture, причем которые не "viewed"
    {
        [viewedPicIdsPack addObject:identif];                        // Про NSMutableSet не знаем, судя по проверке
    }
}                                                                    // При чем здесь "savePicture"?

    К этой функции прилагается еще 5 кг спагетти-кода работы с этой кучей NSMutableArray на все случаи жизни (в функции еще не все такие контейнеры засветились)
    Писал человек с 3+ года опыта коммерческой разработки.

    NAlexN, 19 Декабря 2013

    Комментарии (6)

  3. Objective C / Говнокод #13762

    −111

    1. 1
    Character * c = Character.alloc.init.autorelease;

    К чему приводит излишняя либеральность синтаксиса

    NAlexN, 10 Сентября 2013

    Комментарии (11)

  4. Objective C / Говнокод #13554

    −121

    1. 1
    2. 2
    3. 3
    4. 4
    - (id) init {
    if ((self = [super init]) == nil) return nil;
    ...
}

    Как автор сюда еще тернарный оператор взгромоздить не додумался?

    NAlexN, 05 Августа 2013

    Комментарии (0)

  5. Objective C / Говнокод #12757

    −96

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    int arrayCount = [itemArray count];
if( currentHeight < 10000 ) {
    currentItemType = random() % arrayCount;
} else if( currentHeight < 50000 ) {
    currentItemType = random() % arrayCount;
} else if( currentHeight < 100000 ) {
    currentItemType = random() % arrayCount;
} else {
    currentItemType = random() % arrayCount;
}

    Чем выше горы - тем рандомней сборы.

    NAlexN, 16 Марта 2013

    Комментарии (17)