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

    +68

    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
    #include <iostream>
    
    using namespace std;
    
    unsigned ololo_div(unsigned what, unsigned by) {
        unsigned cnt=0;
        while( what>by ) {
            what-=by;
            cnt++;
        }
        if( what==0 ) cnt++;
        return cnt;
    }
    
    int main() {
        cout << ololo_div(10,0) << endl;
        return 0;
    }

    "че там делить то)))"

    http://codepad.org/nL06uX2s

    Elvenfighter, 23 Апреля 2014

    Комментарии (4)
  2. C++ / Говнокод #15794

    +64

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    QSqlQuery my_query;
    my_query.prepare(
              QString("INSERT INTO table1 (number, address, age) VALUES (%1, '%2', %3);")
                              .arg(fromInput1).arg(fromInput2).arg(fromInput3)
              );

    Жаль, но похоже автор не осилил экранирование от SQL-иньекций.

    LispGovno, 19 Апреля 2014

    Комментарии (29)
  3. C++ / Говнокод #15793

    +70

    1. 1
    qDebug() << QString("%1-%2").arg("%1").arg("foo");

    LispGovno, 19 Апреля 2014

    Комментарии (37)
  4. C++ / Говнокод #15778

    +72

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    auto val = map_.at(std::distance(map_.begin(),
    	std::min_element(map_.begin(), map_.end(), [](std::vector<int> a, std::vector<int> b)
    	{
    		return b.at(std::distance(b.begin(), std::min_element(b.begin(), b.end()))) > a.at(std::distance(a.begin(), std::min_element(a.begin(), a.end())));
    	})));
    
    return val.at(std::distance(val.begin(), std::min_element(val.begin(), val.end())));

    Разыменование итератора для слабаков!

    runewalsh, 18 Апреля 2014

    Комментарии (31)
  5. C++ / Говнокод #15776

    +61

    1. 1
    std::cout << std::cout << "Hello, world";

    С гейдевчика, странно, что здесь не было.

    DlangGovno, 17 Апреля 2014

    Комментарии (5)
  6. C++ / Говнокод #15774

    +77

    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
    #define SK_SIMPLE_SIZE SK_BIG : case SK_SMALL : case SK_MIDDLE
    
    switch (some_shit)
    {
      case SK_TINY : 
      {
        do_smth();
      }
      break;
      case SK_SIMPLE_SIZE :
      {
        do_smth2();
      }
      break;
    }

    странно что вроде бы тут не было

    TarasB, 17 Апреля 2014

    Комментарии (14)
  7. C++ / Говнокод #15763

    +14

    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
    typedef CStatisticsCalculator* (*TCalcCreator)(string&, const TUltimateStatDataPtr&);
    typedef std::map <CStatisticsCalculator::eCalcKind, boost::tuple<string, TUltimateStatDataPtr, TCalcCreator>> TCalcImplMap; 
    
    static TCalcImplMap CalcsMap;
    
    CStatisticsCalculator* CStatisticsCalculator::MakeNewCalculator(CStatisticsCalculator::eCalcKind Kind, 
                                                                    const TUltimateStatDataPtr& Data )
    {
        if (CalcsMap.empty())
        {
          CalcsMap[eCalcKind::eExtremeDealPrice] = 
            boost::make_tuple(string("be.commons.calculators.extreme_prices"), Data,
                   [](string& Name, const TUltimateStatDataPtr& Data_)->CStatisticsCalculator* 
                       { return new CExtremePricesCalculator(Name.c_str(), Data_);});
        
          CalcsMap[eCalcKind::eTurnOver] = 
            boost::make_tuple(string("be.commons.calculators.turnover"), Data,
                   [](string& Name, const TUltimateStatDataPtr& Data_)->CStatisticsCalculator* 
                       { return new CTurnOverCalculator(Name.c_str(), Data_);});
          
        };             
        
        auto i = CalcsMap.find(Kind);
        
        if (i == CalcsMap.end())
            return 0;
        
        auto& tpl = i->second;
        
        return tpl.get<2>()(tpl.get<0>(), tpl.get<1>());
    };

    Фабричный метод теперь выглядит так

    ABBAPOH, 16 Апреля 2014

    Комментарии (4)
  8. C++ / Говнокод #15758

    +9

    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
    template<...many params...>
    class Foo
    {
      template <typename T>
      class __Bar 
      {
        T t;
      public :
        __Bar(T t) : t(t) {}  
      };
    
    public :
      template <typename T>
      __Bar<T> Bar(T t)
      {
        return __Bar<T>(t); // просто __Bar(t) нельзя, вывод не сработает
      }
    
      template <typename T>
      void Buzz (T t)
      {
      }
    };
    
    ...
    Foo f;
    f.Buzz(Bar(5));

    А как вы выкручиваетесь из неумения определять тип результата по типу параметров конструктора?

    TarasB, 15 Апреля 2014

    Комментарии (87)
  9. C++ / Говнокод #15753

    +9

    1. 1
    BOOST_FOREACH(auto &v, pt.get_child("root"))

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

    laMer007, 14 Апреля 2014

    Комментарии (4)
  10. C++ / Говнокод #15750

    +12

    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
    int main()
    {
    MyList<MySubList*> *list;
    ...
    int error=makeZeroItemsNull(list)+deleteZeroItems(list);
    delete list;
    return error;
    }
    
    int makeZeroItemsNull(MyList *list)
    {
    //Удаляем указатели в списке и обнуляем
    if(list==0||error)return 1;
    return deleteZeroItems(list); //На всякий случай проверяем остались ли не удаленные элементы
    }
    
    int deleteZeroItems(MyList *list)
    {
    //Удаляем из списка обнуленные элементы
    if(list==0||error)return 2;
    return makeZeroItemsNull(list); //На всякий случай проверяем остались ли не обнуленные элементы
    }

    Учим все функции возвращать int(0) когда нет ошибок.

    IGHOR, 14 Апреля 2014

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