1. Objective C / Говнокод #12881

    −102

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    @interface CORERatingImages : NSObject
{
    UIImage *imageForOne;
    UIImage *imageForTwo;
    UIImage *imageForThree;
    UIImage *imageForFour;
    UIImage *imageForFive;
}
+(CORERatingImages *) ratingImages;
-(UIImage *) getRatingImage:(int) ratings;
@property (nonatomic, retain) UIImage *imageForOne;
@property (nonatomic, retain) UIImage *imageForTwo;
@property (nonatomic, retain) UIImage *imageForThree;
@property (nonatomic, retain) UIImage *imageForFour;
@property (nonatomic, retain) UIImage *imageForFive;
-(void) releaseResources;
@end

static CORERatingImages *ratingImages = nil;

@implementation CORERatingImages
@synthesize imageForOne;
@synthesize imageForTwo;
@synthesize imageForThree;
@synthesize imageForFour;
@synthesize imageForFive;

+(CORERatingImages *) ratingImages
{
    @synchronized(self)
    {
        if (ratingImages == nil)
        {
            ratingImages = [[self alloc] init];
        }
    }
    return ratingImages;
}
-(id) init
{
    if (self = [super init])
    {
        self.imageForOne = [UIImage imageNamed:@"1.png"];
        self.imageForTwo = [UIImage imageNamed:@"2.png"];
        self.imageForThree = [UIImage imageNamed:@"3.png"];
        self.imageForFour = [UIImage imageNamed:@"4.png"];
        self.imageForFive = [UIImage imageNamed:@"5.png"];
    }
    return self;
}
-(UIImage *) getRatingImage:(int) ratings
{
    if (ratings == 1)
    {
        return imageForOne;
    }
    else if (ratings == 2)
    {
        return imageForTwo;
    }
    else if (ratings == 3)
    {
        return imageForThree;
    }
    else if (ratings == 4)
    {
        return imageForFour;
    }
    else if (ratings == 5)
    {
        return imageForFive;
    }
    else
    {
        return [UIImage imageNamed:@"0.png"];
    }
}
-(void) dealloc
{
    NSLog(@"release Images");
    [imageForOne release];
    [imageForTwo release];
    [imageForThree release];
    [imageForFour release];
    [imageForFive release];
    [super dealloc];
}
-(void) releaseResources
{
    [ratingImages release];
    ratingImages = nil;
}

@end

    Массив или stringWithFormat: @"%d.png"?

    Не, не слышал.

    Запостил: QuickNick, 11 Апреля 2013

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

    • ava tirinox 11.04.2013 13:02 # +1

      паттерн говнолтон
      Ответить

    • ava Xtasy 11.04.2013 13:09 # 0

      Еще и целый синглтон вместо одной однострочной функции (можно в категорию на UIImage).
      Ответить

    • ava krypt 11.04.2013 13:17 # +1

      Да у вас там целая говношахта, блин ) Хотя аффтар пол года назад уволился и ресурс не возобновляемый )
      Ответить

      • ava QuickNick 11.04.2013 13:37 # +1

        Сушеный помет можно использовать в качестве топлива.
        Следовательно, мы - топливная сверхдержава!
        Ответить

      • ava pee 13.04.2013 10:12 # 0

        Вы(с QuickNick) много пишите об этом товарище. И то, что вы откапываете его код из проекта. У вас вообще нет codereview, шоле? оО
        Ответить

        • ava krypt 13.04.2013 10:43 # 0

          При нём не было. Вообще. А сейчас я там уже не работаю.
          Ответить

          • ava pee 13.04.2013 10:50 # 0

            Это грустно. Хотя иногда даже кодревью не спасает:)
            Ответить

        • ava QuickNick 15.04.2013 11:03 # 0

          На момент написания кода было очень наивное ревью.

          Потом архитектурные правила усложнялись, ревью мал-помалу вводилось, но автор оказался очень устойчив к изменениям - все замечания он отклонял, приходилось пробивать через начальство, и на обсуждение какой-нибудь ерунды запросто мог уйти час. Помню эпическое сражение "Код UITableViewController*ов у тебя не оформлен согласно MVC" :)
          Ответить

    • ava clockworkman 11.04.2013 16:54 # 0

      Индусятина?
      Ответить

      • ava krypt 11.04.2013 17:04 # 0

        Да если бы. Аффтар известен в лицо.
        Ответить

        • ava eth0 11.04.2013 19:42 # +3

          Как он относится к коровам?
          Ответить

          • ava krypt 11.04.2013 19:56 # 0

            Гм... Не хочу оскорблять человека, всё таки у меня претензии к его коду, а не к нему самому и свои ассоциации я придержу при себе... Но священными не считает, нет.
            Ответить

    • ava QuickNick 12.04.2013 09:18 # 0

      2 строки превратить в 100 - это рекорд.
      Ответить

    Добавить комментарий