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

    −113

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if ([_categories count] != 0) {
    for (NSString *item in _categories) {
        [path appendFormat:@"categories/%@/", item];
    }
}

    Случайно обнаружил у себя :)

    Запостил: zummenix, 10 Апреля 2013

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

    • ava Lure Of Chaos 10.04.2013 18:56 # +3

      обнаружил у себя все, кроме родильной горячки (ц)
      Ответить

    • ava byss 10.04.2013 19:11 # 0

      В чем говно, кроме лишнего if'а?
      Ответить

      • ava zummenix 10.04.2013 19:22 # 0

        Как раз в этом и есть.
        Ответить

      • ava krypt 10.04.2013 21:15 # 0

        И не только.
        Есть такой метод, зовётся stringByAppendingPathComponent. Делает ровно то, что написано в её имени.
        Дал бы ссылку на референс, но какого-то фига сайт Эппла не открывается.
        Ответить

        • ava bormand 10.04.2013 21:23 # 0

          > stringByAppendingPathComponent
          Блин, как вы живете с такими именами...
          Ответить

          • ava LispGovno 10.04.2013 21:39 # 0

            string_by_appending_path_component ?
            Ответить

            • ava bormand 10.04.2013 21:48 # 0 

              path = path / "categories" / item
              feel the power of the dark side.
              Ответить

              • ava Vindicar 11.04.2013 11:19 # 0

                Гыг, путь как подкласс строки с переопределенным оператором деления?
                Предлагаю тогда:
                path1 + path2 => слияние путей
                path1 / path2 => тоже (синтаксический сахар)
                path1 - path2 => преобразовать путь path1 в относительный с базой path2
                path-- => убрать последнюю часть пути
                Ответить

                • ava bormand 11.04.2013 11:50 # +1

                  > Гыг, путь как подкласс строки с переопределенным оператором деления?
                  http://www.boost.org/doc/libs/1_53_0/libs/filesystem/doc/index.htm
                  Ответить

                  • ava guest 11.04.2013 23:33 # −1

                    Если правильно понимаю, то это алиас на /= , который в свою очередь оператор, соответствующий ф-ии "append".

                    Что не очень понятно - зачем тогда перегружать / ?
                    Ответить

                    • ava defecate-plusplus 12.04.2013 06:49 # +1

                      разницы между / и /= не видишь чтоли?
                      namespace fs = boost::filesystem;
void cache(fs::path const &);
// 1
cache(basepath / hash(filename) / filename);
// 2
fs::path tmp = basepath;
tmp /= hash(filename);
tmp /= filename;
cache(tmp);
                      Ответить

          • ava krypt 10.04.2013 23:51 # 0

            Спагетти-код? Не, не слышал )
            Ответить

          • ava Xtasy 11.04.2013 09:40 # 0

            Это _очень_ короткое имя для метода по меркам Obj-C.
            Ответить

            • ava bormand 11.04.2013 09:42 # 0

              У нее просто слишком мало параметров... судя по всему всего один?
              Ответить

            • ava QuickNick 11.04.2013 11:23 # +2

              +(NSMutableURLRequest *) formatRequest:(NSString *) urlString withRequestType:(NSString *) type withData:(NSString *) dataStr withParam:(NSMutableArray *) param withContentType:(NSString *) content autodetectProject:(BOOL)autodetectProjec t proRequest:(BOOL)isProRequest

              Один из эпичнейших методов в старом проекте, который едва не перенесли в новую библиотеку.
              Ответить

              • ava zummenix 11.04.2013 11:31 # 0

                :) А как заменили?
                Ответить

                • ava QuickNick 11.04.2013 11:39 # 0

                  Сделали UrlBuilder, которому последовательно ставили страницу, накидывали параметры запроса "ключ-значение" и в конце концов вызывали create.

                  Из-за того, что на тот момент не очень Objective-C знал, получился не шедевр, но жалоб не поступало.
                  Ответить

              • ava wvxvw 12.04.2013 01:53 # 0

                На фоне того, как пару дней назад Эппл пинками выгнали из АппСтора какую-то невзрачную контору со всего-то десятком миллионов пользователей, глядя на пытки которым подвергаются рядовые ОбжСи программисты невольно задумываешься о третьей волне.
                Ответить

                • ava blackhearted 12.04.2013 15:46 # 0

                  Зато гугль не такой, жидята за нас, простых девелоперов стоят горой.
                  Совсем уже яблокофобы ебанулись.
                  Ответить

        • ava byss 11.04.2013 12:03 # 0

          Не факт, что хотели добавить именно path component. Хотя, если это действительно так, то согласен.
          Ответить

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