Почему это компилируется и вообще как работает место в коде и что, так можно было (QSharedPointer)?

Question

[Kalombyr]

Добрый день!

Есть код:

.....
typedef QSharedPointer<QDataStream> AnswerData;
.....
Executor::AnswerData Executor::sendSync(int timeout)
{
    if ( !_serial->isConnected() ) return false;
    .......
}
.......
double Heating::getCurrentTemp()
{
    ......
    Executor::AnswerData data = _executer->sendSync();
    ......
    if (!data)  return -1;
    ......
}

Как видите, функция sendSync должна возвращать QSharedPointer, но в случае ошибки она возвращает false,
ну а в getCurrentTemp проверяется результат на false и выходит в случае чего.
И главное, это работает (вроде бы).

Я не нашёл в документации на QSharedPointer , почему так можно было сделать? А можно ли вообще (не будет ли каких-либо "сюрпризов")?

P.S. Компилятор MinGW, Qt 5.10.0

Answer 1

[Vitaly]

Вообще непонятно почему у вас так. Попробуйте чистый пример вроде:

QSharedPointer<QDataStream> doFoo()
{
    return false;
}

Возможно у вас что-то не то с флагами, или окружением, или компилятором, или версией Qt. Дело в том, что у QSharedPointer нет такого конструктора. Сделаю предположение, что у вас MSVC вместо компилятора :)

Советую возвращать какой-нибудь DefaultAnswer или просто nullptr.

Roman, те ссылки, что вы привели не релеванты. Они про приведение типа к bool, но не наоборот.

Comments

[Roman]

Ок

Сделаю предположение, что у вас MSVC вместо компилятора :)

В VS компилируется.

[Евгений Шатунов]

Vitaly, а ты ведь помнишь что boolнеявно приводится к int? И в случае с false ноль все еще остается литералом, который может трактоваться как NULL по причинам совместимости.

[Vitaly]

Евгений Шатунов, Всё, что ниже C++11 я трактую как outdated. Так ведь и есть, ведь прошло уже 8 лет с момента принятия 11го стандарта. С 11ым стандартом этот код не совместим.
Сейчас 11 стандарт не поддерживается только лимитированным набором компиляторов, которые в основном используются для каких-нибудь кастомных железяк, где мало памяти и ленивые вендоры компиляторов.

[Kalombyr]

Компилятор MinGW 53, Qt 5.10.0
Ваш "чистый" пример компилируется, вызывается и обрабатывается =)

[Vitaly]

Kalombyr, хорошо, а теперь включите C++11 и добавьте флаги wall, wextra и pedantic. Это чтобы стандарту следовать.

Answer 2

[Никита]

Дополнение:
Опыты показали, что во всем виноват конструкторQSharedPointer(std::nullptr_t), как только к своему классу добавил аналогичный конструктор, стало возможно преобразование "return false" в объект класса.

nullptr_t ff2(0); //так можно
nullptr_t ff1(false); //так вроде нельзя, но MSVC не ругается

Старый и неверный ответ:
Ну так судя по документации QSharedPointer имеет соответствующие операторы преобразования, а именно

bool QSharedPointer::operator bool() const
bool QSharedPointer::operator!() const

Returns true if this object is not null. This function is suitable for use in if-constructs, like:
if (sharedptr) { ... }

Comments

[Roman]

Никита,

Roman, те ссылки, что вы привели не релеванты. Они про приведение типа к bool, но не наоборот.

[Никита]

Roman, понял, вопрос про "return false;".

[Никита]

Roman, изменил свой ответ с учетом вашего замечания, просьба глянуть.

[Roman]

Никита,

с учетом вашего замечания

Не моего а Vitaly,

Answer 3

[SerJook]

Согласно стандарту C++11 такое не должно компилироваться. Но у Майкрософта своё видение стандартов.
GCC выдаст ошибку

error: could not convert ‘false’ from ‘bool’ to ‘QSharedPointer<QDataStream>'

Comments

[Vitaly]

Всё верно.
Я знаю, что майкрософта тоже можно выставить диалект C++, там даже можно было выставить что-то со словом "latest", чтобы самый последний стандарт использовать. Ну и всякие расширения компилятора тоже можно поотключать, равно как и аналоги wall, wextra и pedantic выставить. Я правда не помню как, уже давно MSVC не пользовался и под винду не писал.