VPF::Подскажите что не так в этом коде

_Ghost_

Дата 8.5.2008, 21:29 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 27
Регистрация: 24.12.2007

Репутация: нет
Всего: нет

Подскажите что не так в этом коде?
вроде все компилится

Код


bool login(char* input, char* domain)
{
    Authenticator* authenticator = new Authenticator();
    int userlen = 0;
    char user[255];
    char password[255];
    char* p = input;

    if(input[0] == '#')
      // no username specified
      return false;

    try
    {
        // Get length of username
        while(p != NULL)
        {
            if(*p == '#')
              break;
            userlen++;
            p++;
        }

        // Extract username from input
        for(int i=0; i <= userlen; i++)
        {
            user[i] = input[i];
        }

        // Extract password from input
        strcpy(password, p);

        authenticator.login(user, password);
    }
    catch(...)
    {
        throw LoginException();
    }

    delete authenticator;
    return true;
}

Rififi

Дата 8.5.2008, 21:38 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1254
Регистрация: 9.3.2008

Репутация: 11
Всего: 36

Подскажите что не так в этом коде?
у него плохая аура.

JackYF

Дата 8.5.2008, 22:32 (ссылка)

(нет голосов)

Загрузка ...

полуавантюрист

Профиль
Группа: Участник
Сообщений: 5814
Регистрация: 28.8.2004
Где: страна тысячи озё р

Репутация: 18
Всего: 162

Rififi, курсив - это хорошо, но есть кнопка "быстрая цитата" ;)

Цитата(_Ghost_ @ 8.5.2008, 20:29

)

Подскажите что не так в этом коде?

Проблемы-то какие?

--------------------

Пожаловаться на меня как модератора можно здесь.

Ln78

Дата 9.5.2008, 09:23 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 274
Регистрация: 25.11.2006

Репутация: 13
Всего: 15

Цитата(_Ghost_ @ 8.5.2008, 21:29

)

authenticator.login(user, password);

Результат выполнения этого метода никак не важен, сам метод login возвращает всегда true - это, конечно, может быть и правильным, но несколько смущает. Более очевидные реализации вернули бы именно результат выполнения authenticator.login

mes

Дата 9.5.2008, 11:09 (ссылка)

(нет голосов)

Загрузка ...

любитель

Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250

Цитата(Ln78 @ 9.5.2008, 09:23

)

сам метод login возвращает всегда true

не всегда ))

Цитата(Ln78 @ 9.5.2008, 09:23

)

Код


    if(input[0] == '#')
      // no username specified
      return false;

к тому же если используется ассинхроное программирование , то результат операции сообшает лишь о том, что команда отправлена на выполнение.
а authenticator.login() вполне вероятно, что возврашает void -так как выдать результат "логина" он вернуть "физически" не может

Это сообщение отредактировал(а) mes - 9.5.2008, 11:11

--------------------

http://opendots.net

Ln78

Дата 9.5.2008, 11:19 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 274
Регистрация: 25.11.2006

Репутация: 13
Всего: 15

mes, первый возврат я видел, в данном случае речь не о нём (хотя цитата с этим кодом не моя smile

).
И про то, что authenticator.login обязан возвращать значение категоричных утверждений не было. Просто исходный вопрос предполагал, что в коде что-то не так. У тебя есть какие-то другие конструктивные варианты ответа на заданный вопрос?

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »