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

    +162

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    Allow* AccessSection::check(CONNECTION * connection, char *username, char *password)
{
...
                if (current->username != "" && username != "" && current->username != username)
                        continue;
...

    Кусок кода из прокси-сервера Shweby

    Запостил: gpr, 07 Июня 2010

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

    • ava Мистер Хэнки 07.06.2010 19:31 # +1

      username != ""

      похапэшная обезьянка детектед
      Ответить

    • ava guest 07.06.2010 19:33 # +1

      А как надо?
      Ответить

      • ava guest 07.06.2010 20:26 # 0

        if ( strlen ( username ) > 0 && ... )
        Ответить

      • ava legolegs 07.06.2010 20:56 # +1 

        Allow* AccessSection::check(CONNECTION * connection, const std::string & username, const std::string & password)
{
...
                if (!current->username.empty() && !username.empty() && current->username != username)
                        continue;
...

        К.О.
        Ответить

    • ava absolut 08.06.2010 07:55 # 0

      Сравнение строк кучерявое кстати :
      current->username != username
      P.S. В исходном варианте имеется в виду
      Ответить

      • ava gpr 08.06.2010 09:21 # 0

        Справедливости ради стоит отметить, что current->username это string, так что тут все круто.

        Кстати, если в коде есть что-то типа:
        username = "";
        то сравнение username с "" может даже и работать как надо :)
        Ответить

        • ava absolut 08.06.2010 11:03 # 0

          Справедливости ради обратите внимание на постскриптум.
          Ответить

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