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

    +15

    1. 1
    2. 2
    3. 3
    String testName;
//...
std::swap(testName,  _testName);

    String из thirdparty-библиотеки, а swap везде в нашем коде. По очевидным причинам получаем подение производительности.

    LispGovno, 16 Марта 2013

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

  2. C++ / Говнокод #12746

    +23

    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
    struct mystream: public std::ostream 
{
    mystream(std::ostream & o): std::ostream(o.rdbuf()) {}

    template <class T>
    mystream & operator << (T const & arg)
    {
        if (enabled_) as_std_ostream() << arg;
        return *this;
    }

    // дерьмо STX
    mystream & operator << (std::ostream & (*f)(std::ostream &))
    {
        if (enabled_) as_std_ostream() << f;
        return *this;
    }

    mystream & operator << (std::ios & (*f)(std::ios &))
    {
        if (enabled_) as_std_ostream() << f;
        return *this;    
    }

    mystream & operator << (std::ios_base & (*f)(std::ios_base &))
    {
        if (enabled_) as_std_ostream() << f;
        return *this;
    }
    // дерьмо ETX

    void enable() { enabled_ = true; }
    void disable() { enabled_ = false; }

protected:
    bool enabled_;
    std::ostream & as_std_ostream() { return *this; }
};

    а так хотелось хоть сегодня рыбки поесть захерачить вместо трёх перегрузок
    template <class O>
    mystream & operator << (O & (*f)(O &)) { ...

    defecate-plusplus, 13 Марта 2013

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

  3. C++ / Говнокод #12736

    +28

    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
    class atoi_func
{
public:
    atoi_func(): value_() {}

    inline int value() const { return value_; }

    inline bool operator() (const char *str, size_t len)
    {
        value_ = 0;
        int sign = 1;
        if (str[0] == '-') { // handle negative
            sign = -1;
            ++str;
            --len;
        }

        switch (len) { // handle up to 10 digits, assume we're 32-bit
            case 10:    value_ += (str[len-10] - '0') * 1000000000;
            case  9:    value_ += (str[len- 9] - '0') * 100000000;
            case  8:    value_ += (str[len- 8] - '0') * 10000000;
            case  7:    value_ += (str[len- 7] - '0') * 1000000;
            case  6:    value_ += (str[len- 6] - '0') * 100000;
            case  5:    value_ += (str[len- 5] - '0') * 10000;
            case  4:    value_ += (str[len- 4] - '0') * 1000;
            case  3:    value_ += (str[len- 3] - '0') * 100;
            case  2:    value_ += (str[len- 2] - '0') * 10;
            case  1:    value_ += (str[len- 1] - '0');
                value_ *= sign;
                return value_ > 0;
            default:
                return false;
        }
    }
private:
    int value_;
};

    standard atoi()
    79142 milliseconds

    class atoi_func
    131 milliseconds.

    Если приходится велосипедить стандартные функции, то это камень в огород С++. Видать кресты писали гении ассемблерной оптимизации.

    LispGovno, 13 Марта 2013

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

  4. C++ / Говнокод #12733

    +14

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    void ThumbnailAdapter::clearCache(size_t index) {
    if ((size_t)-1 == index) {
        mImages.clear();
    } else {
        ImagesMap::iterator it = mImages.find (index);
        if (mImages.end() != it) {
            mImages.erase(it);
        }
    }
}

    годная очистка map'ы

    shomeser, 12 Марта 2013

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

  5. C++ / Говнокод #12716

    +16

    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
    class Context;

class AbstractState
{
	Context * m_context;

protected:
	Context * context() const { return m_context; }

public:
	AbstractState(Context * context) : m_context(context) { };
	virtual ~AbstractState() { }
	virtual void doSomething() = 0;
};

class Context
{
	std::unique_ptr<AbstractState> m_state;

public:
	enum State
	{
		State1,
		State2,
	};
	Context() { switchToState(State1); }
	void switchToState(State newState);
	void doSomething() { m_state->doSomething(); }
	void someCleanup() { }
};

class ConcreteState1 : public AbstractState
{
public:
	ConcreteState1(Context * context) : AbstractState(context) { }
	virtual void doSomething()
	{
		context()->switchToState(Context::State2);
		context()->someCleanup();
	}
};

class ConcreteState2 : public AbstractState
{
public:
	ConcreteState2(Context * context) : AbstractState(context) { }
	virtual void doSomething()
	{
		context()->switchToState(Context::State1);
		context()->someCleanup();
	}
};

void Context::switchToState(State newState)
{
	switch(newState)
	{
	case State1:
		m_state.reset(new ConcreteState1(this));
		return;
	case State2:
		m_state.reset(new ConcreteState2(this));
		return;
	}
}

    Бывает, на меня находит состояние "сначала делай, потом думай", благо результат был быстро обнаружен отладчиком.

    Xom94ok, 10 Марта 2013

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

  6. C++ / Говнокод #12713

    +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
    struct base {
   template <class Foo>
	base() {}
};

struct derived {
	derived()
		: base::base<int>()		// why not?? WHHYYYY?
	{}
};

base b1 = base::base<int>();
base b2<int>();

    долбанный комитет
    им проще запретить, чем продумать нормальный способ вызова шаблонного конструктора

    defecate-plusplus, 09 Марта 2013

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

  7. C++ / Говнокод #12712

    +12

    1. 1
    http://pastebin.com/kG05YmBX

    Поиск подстроки в строке, написано однокурсником

    jQuery, 09 Марта 2013

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

  8. C++ / Говнокод #12711

    +21

    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
    BOOL Space::SetShattle(Shattle* sh)
{
	if(!sh)
		return 1;

	if(!dynamic_cast<Shattle*>(sh) )
		return 2;

	if(shattle)
		delete shattle;
	shattle=sh;

	return 0;
}

    Как освоить и закрепить знания о приведении типов? Скажем, вот так...

    Bart, 09 Марта 2013

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

  9. C++ / Говнокод #12707

    +21

    1. 1
    2. 2
    3. 3
    4. 4
    if (g_bCanAcceptUnderScore)
			SetIgnoreChars("№@`$%#^&*()~[]{}:;,.!?><|\\//-=+'\" \n");
		else
			SetIgnoreChars("№@`$%#^&*()~[]{}:;,.!?><|\\//-=+'\" \n_");

    lifemaker, 08 Марта 2013

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

  10. C++ / Говнокод #12704

    +10

    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
    {//НАЧАЛО ВРЕМЕННОГО БЛОКА ПОЛУЧЕНИЯ КОДА
try
{
	int q=quan-pos;
	char* temp=new char[q+1];//Временный буфер под код.
	for(int i=0;i<q;temp[i]=buf_temp[pos+(i++)]);
	//В цикле копируем в этот буфер символы кода.
	temp[q]=0;//Завершаем строку буфера нулём.
	code=atoi(temp);
	delete[]temp;
}
catch(...)
{
	//Проверка на вшивость, учитывая возможный размер кода, излишня, но всё же не помешает, поскольку не тормозит систему.
	code=0;
}
}//КОНЕЦ ВРЕМЕННОГО БЛОКА ПОЛУЧЕНИЯ КОДА

    Когда-то давно (школопериод) выпало изучать обмен данными по сети на С++.
    В задании выше - кусок функции ListenSockets(), которая принимала по сети форматированную строку и парсила.
    В те годы про strtok() я еще не слыхал...

    Bart, 08 Марта 2013

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