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

    +165

    1. 1
    log::write(std::string() + "Opened \"" + filename + "\"");

    История не моя, но приложу её к коду:
    >Несколько лет назад у нас в проекте была такая строчка кода
    Функция log::write() принимала форматную строку и набор аргументов, как printf.
    Если имя файла содержит %, возможны неприятности, но такие файлы обычно не встречаются.
    Я когда это увидел, очень резко прокоментировал способности и интелектуальный уровень того, кто так написал.
    Как же так, если файл называется скажем "%s", программа грохнется с вероятностью близкой к 100%!

    Как оказалось, этот кусок кода писала жена начальника.
    Морские свинки и программистки, ага.
    Кажется она сильно обиделась, меня уволили.

    Запостил: Говногость, 21 Августа 2011

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

    • ava CPPGovno 21.08.2011 02:16 # +5

      Cool story, Bro.
      Ответить

    • ava Lure Of Chaos 21.08.2011 08:34 # 0

      >Opened
      клозет
      Ответить

    • ava CPPGovno 21.08.2011 11:19 # −2

      Кстати, а какая замена этому коду? Вроде он так нормальный...
      Ответить

    • ava absolut 21.08.2011 11:44 # 0

      А как вам приведение к std::string через конкатенацию?
      Ответить

      • ava CPPGovno 21.08.2011 11:56 # 0

        можно пример? мы не в сишарпе
        тогда уж создание string переменной с нужными данными и последующей передачей в функцию лога
        Ответить

      • ava gegMOPO4 21.08.2011 12:04 # 0

        Это как раз нормально. Не самый эффективный способ, но разница не принципиальна в данном случае. Лаконичность стоит тактов.
        Ответить

    • ava CPPGovno 21.08.2011 11:58 # −2

      С++0х:
      log::write("Opened \""s + filename + "\"");
      Ответить

      • ava CPPGovno 21.08.2011 12:03 # 0

        Тем кто не верит, то привожу выдержку:
        >C++0x supports ``user-defined literals'' through the notion of literal operators that map literals with a given suffix into a desired type. For example:

        constexpr complex<double> operator "" i(long double d)	// imaginary literal
	{
		return {0,d};	// complex is a literal type
	}

	std::string operator""s (const char* p, size_t n)	// std::string literal
	{
		return string(p,n);	// requires free store allocation
	}
        Ответить

    • ava CPPGovno 21.08.2011 12:01 # 0

      Это конечно шутка, навеянная enum'ом FORCE_DWORD=0x7FFFFFFF из некоторых системных библиотек и библиотеки DirectX:
      log::write(FORCE_STRING+"Opened \"" + filename + "\"");
      Ответить

    • ava bugmenot 21.08.2011 13:01 # 0

      ололо, на башорк!
      Ответить

    • ava guest8 09.04.2019 11:21 # −999

      показать все, что скрытоvanished
      Ответить

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