1. C++ / Говнокод #23196

    0

    1. 1
    https://github.com/catboost/catboost

    Похоже яндекс заопенсорсил свежую версию аркадии. Давайте обсирать эти их велосипеды.

    Запостил: FrauSchweinhund, 19 Июля 2017

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

    • ava FrauSchweinhund 19.07.2017 22:14 # 0

      https://github.com/catboost/catboost/blob/master/CPP_STYLE_GUIDE.md
      > Don't use boost, and limit use of STL (Use optimized counterparts available in /util instead of some of the parts that are poorly implemented in STL, such as std::string -> TString, std::streams -> TInputStream, TOutputStream). We don't encourage reinventing the wheel (we've already invented everything and just check the /util).
      Ответить

      • ava Antervis 20.07.2017 04:50 # 0

        yeah, they sure discourage reinventing the wheel
        Ответить

        • ava j123123 20.07.2017 12:24 # 0

          > yeah, they sure discourage reinventing the wheel

          If wheel made from frozen shit, then it makes sense
          Ответить

    • ava FrauSchweinhund 19.07.2017 22:31 # +1 

      if (errorHappened) {
    ythrow TSomeException(numericCode) << "error happened (" << usefulDescription << ")";
}

      Они кидают исключения специальным оператором йкинуть.
      Ответить

    • ava AnalPerOral 20.07.2017 08:03 # 0

      Обосрал лицо твоей мамки, проверь.
      Ответить

    • ava j123123 20.07.2017 12:07 # 0 

      https://github.com/catboost/catboost/blob/37378d278a57a2464547e07e48a2c779d888e174/contrib/libs/cxxsupp/openmp/kmp_environment.c#L183


      Классика
      return ( ( value == NULL ) ? ( 0 ) : ( 1 ) );
      Ответить

      • ava j123123 20.07.2017 12:13 # 0

        но это конечно к самому catboost отношения не имеет. Это так, какая-то левая херня из зависимостей
        https://github.com/catboost/catboost/blob/master/catboost/libs/algo/error_functions.cpp эта хрень на плюсах написана, но в каком-то сишечном стиле, без всяких там std:kokoko

        Вот тут
        https://github.com/catboost/catboost/blob/master/catboost/libs/algo/greedy_tensor_search.cpp#L25 уже есть парашная плюсовая лямбда
        Ответить

        • ava d_fomenok 20.07.2017 12:43 # 0

          Надо было использовать Bing
          Ответить

        • ava subaru 20.07.2017 16:21 # 0

          > но в каком-то сишечном стиле
          Оно было бы в сишечном стиле, если бы все переменные были однобуквенные, вместо const double* войд звездочка, и пара маллоков без парных фри. А так обычные плюсы.
          Ответить

          • ava Antervis 20.07.2017 19:57 # 0

            не путайте лабораторный си с толковым
            Ответить

            • ava subaru 20.07.2017 20:16 # 0

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

        • ava roman-kashitsyn 20.07.2017 18:43 # 0

          > в каком-то сишечном стиле

          for (auto& curSoftmax : *softmax)

          > без всяких там std:kokoko

          Во втором файле тоже ни одного std::кококо нет
          Ответить

        • ava Antervis 20.07.2017 19:55 # 0

          Первый же метод в первом же файле:
          1. Сигнатура: void(const yvector &, yvector *), когда можно было спокойно сделать yvector(const yvector &). Даже проверка валидности входных данных отсутствует. Типа быстрее?
          2. Находим юз MaxElement, эта шо за покемон? Переходим, voila:
          https://github.com/catboost/catboost/blob/master/util/generic/algorithm.h
          файлик, на 90% состоящий из переименовывания stl. Но юзать stl некошерно же. На дефайнах бы хоть компилилось быстрее. Там в общем-то вся generic папка - супертонкие врапперы над stl
          Ответить

          • ava subaru 20.07.2017 20:17 # 0

            > Сигнатура: void(const yvector &, yvector *), когда можно было спокойно сделать yvector(const yvector &).
            Эта кодовая база появилась до с++11, если что.
            Ответить

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