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

    +61.9

    1. 1
    2. 2
    3. 3
    std::string strFullMapName;
    if ( strstr( _strMapFileName.c_str(), "Map/" ) != _strMapFileName.c_str() )
    	strFullMapName = "Map/";

    из одного очень дорогого проекта

    pushkoff, 03 Октября 2009

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

    +54.4

    1. 1
    2. 2
    ConfigEntry::Ptr generateConfigEntry (const QVariant &val);
    QVariant genetateQVariant(const ConfigEntry::Ptr& entry);

    Страшно представить, и эта опечатка месяц пролежала в основном дереве исходников, хорошо хоть приватная функция

    Sauron, 01 Октября 2009

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

    +62.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    void func(const char* str)
    {
      std::map<std::string, int>::iterator = my_map.find(str);
    
      .....
    }
    
    std::string str = "key";
    func(str.c_str());

    Уверен что это может найти каждый из вас в своих проектах, но может не в таком очевидном виде...

    pushkoff, 25 Сентября 2009

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

    +53.4

    1. 1
    void (* signal(int __sig, void (* __func)(int))) (int)

    Объявление типа, представляющего собой указатель на функцию, возращающей указатель на функцию (может я не так понял?) из хедера signal.h.

    Говногость, 24 Сентября 2009

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

    +51.5

    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
    void CNewAnketaOrg::OnSeachOrg() 
    {
    	if(m_SeachOrg.GetCheck())
    	{
    		m_Create.SetCheck(FALSE);
    		m_SeachOur.SetCheck(FALSE);
    		m_SeachFZ.SetCheck(FALSE);
    		CFindOrganization find(this);
    		if (find.DoModal() == IDOK)
    		{
    			...
    		}
    		...
    	}
    }

    Угадайте какое сообщение привязано к методу с вызовом формы.

    Criperum, 24 Сентября 2009

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

    +60.3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    void AcquireSomething(..., boost::shared_ptr<Something> out)
    {
      ...
      out.reset(new Something());
    }

    жесть

    g26g, 23 Сентября 2009

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

    +55.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
    while (true)
      {
        m_CurrentDevice += delta;
    
        if (m_CurrentDevice < 0)
          m_CurrentDevice = devices[DEV_PLATE]-1;
    
        if (m_CurrentDevice > devices[DEV_PLATE]-1)
          m_CurrentDevice = 0;
    
        break;
      }

    Это конечно написано не специально - в результат целой кучи итераций и переделок. но результата это не отменяет

    TObject, 21 Сентября 2009

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

    +60.6

    1. 1
    2. 2
    3. 3
    static char *szClassName = new char[14];
    static char *szCurrentDirectory = new char[MAX_BUFFER];
    static char *szNewFolder = new char[MAX_BUFFER];

    глобальные указатели рулят, delete нигде не вызывается

    shomeser, 20 Сентября 2009

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

    +64.2

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    ReturnType SomeFunction(...)
    {
      ...
      {
        Mutex mutex;
        Mutex::ScopedLock lock(mutex);
        ...
      }
      ...
    }

    убер синхронизация

    g26g, 18 Сентября 2009

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

    +51.6

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    private: System::Void button2_MouseHover(System::Object^  sender, System::EventArgs^  e) {
    				 int x = rand() % 428-75; //минусуем ширину кнопки, посчитать впадлу
    				 int y = rand() % 301-23; // - высоту
    				 this->button2->Location::set(System::Drawing::Point(x,y));
    			 }
    	private: System::Void button1_Click(System::Object^  sender, System::EventArgs^  e) {						
    				 MessageBox::Show("Мы так и считали", "=)", MessageBoxButtons::OK, MessageBoxIcon::Asterisk);
    			 }

    лень

    ChakuZa, 17 Сентября 2009

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