Ошибка: элемент управления достигает конца непустой функции

Я изучаю C++ и скопировал этот код из учебника, при компиляции кода в конце появляется ошибка. Ошибка говорит:

Управление Достигает конца непустой функции

и находится в конце кода:

#include "ComplexNumber.hpp"
#include <cmath>

ComplexNumber::ComplexNumber()
{
mRealPart = 0.0;
mImaginaryPart = 0.0;
}

ComplexNumber::ComplexNumber(double x, double y)
{
mRealPart = x;
mImaginaryPart = y;
}

double ComplexNumber::CalculateModulus() const
{
return sqrt(mRealPart*mRealPart+
            mImaginaryPart*mImaginaryPart);
}
double ComplexNumber::CalculateArgument() const
{
return atan2(mImaginaryPart, mRealPart);
}

ComplexNumber ComplexNumber::CalculatePower(double n) const
{
double modulus = CalculateModulus();
double argument = CalculateArgument();
double mod_of_result = pow(modulus, n);
double arg_of_result = argument*n;
double real_part = mod_of_result*cos(arg_of_result);
double imag_part = mod_of_result*sin(arg_of_result);
ComplexNumber z(real_part, imag_part);
return z;
}

ComplexNumber& ComplexNumber::operator=(const ComplexNumber& z)
{
mRealPart = z.mRealPart;
mImaginaryPart = z.mImaginaryPart;
return *this;
}

ComplexNumber ComplexNumber::operator-() const
{
ComplexNumber w;
w.mRealPart = -mRealPart;
w.mImaginaryPart = -mImaginaryPart;
return w;
}

ComplexNumber ComplexNumber::operator+(const ComplexNumber& z) const
{
ComplexNumber w;
w.mRealPart = mRealPart + z.mRealPart;
w.mImaginaryPart = mImaginaryPart + z.mImaginaryPart;
return w;
}

std::ostream& operator<<(std::ostream& output,
                     const ComplexNumber& z)
{
output << "(" << z.mRealPart << " ";
if (z.mImaginaryPart >= 0.0)
{
    output << " + " << z.mImaginaryPart << "i)";
}
else
{
    output << "- " << -z.mImaginaryPart << "i)";
}
} //-------->>>>**"Control Reaches end of non-void function"**

c++ compiler-errors return numerical-methods
person heri-salmas    schedule 24.02.2014    source источник
comment
Из какого бы учебника это ни было, я не думаю, что это так уж хорошо.   -  person chris    schedule 24.02.2014
comment
Авторы книги Джо Питт-Фрэнсис и Джонатан Уайтли. Руководство по научным вычислениям на C++. Я читаю его, потому что я должен делать некоторые численные вычисления с использованием объектно-ориентированного подхода, и это единственный, который я смог найти. Если у вас есть какие-либо предложения, я буду рад их услышать.   -  person heri-salmas    schedule 24.02.2014
comment
Честно говоря, ученые и хорошее программирование не очень хорошо сочетаются. В конечном итоге вы получите намного больше от тех книг, которые не предназначены для обучения языку, если вы знаете его достаточно хорошо, чтобы не приобрести вредных привычек. Для языка у нас есть хороший список книг.   -  person chris    schedule 24.02.2014
comment
Спасибо, Крис! Я ученый и больше привык работать с Java и Fortran, теперь мне нужно работать над проектом, где мы используем C++. И вы правы. Даже не разбираясь в C++, я могу заметить много вредных привычек программирования. Я проверю список предложенных книг. И постарайтесь улучшить свой образ ученого в мире программирования!   -  person heri-salmas    schedule 24.02.2014

Ответы (3)


arrow_upward
1
arrow_downward

Ну, operator<< определено для возврата std::ostream&:

std::ostream& operator<<(std::ostream& output, const ComplexNumber& z)
^^^^^^^^^^^^^

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

return output ;

в конце функции. Мы можем видеть, что это неопределенное поведение из чернового стандартного раздела C++ 6.6.3 Параграф 2 инструкции return, в котором говорится:

[...] Вытекание конца функции эквивалентно возврату без значения; это приводит к неопределенному поведению функции, возвращающей значение. [...]

person Shafik Yaghmour    schedule 24.02.2014

arrow_upward
1
arrow_downward

Функция утверждает, что что-то возвращает:

std::ostream& operator<<(std::ostream& output, const ComplexNumber& z)
^^^^^^^^^^^^^

но в конце нет оператора return. Вы должны добавить один:

return output;
person Mike Seymour    schedule 24.02.2014

arrow_upward
0
arrow_downward

Эта функция

std::ostream& operator<<(std::ostream& output,
                     const ComplexNumber& z)
{
output << "(" << z.mRealPart << " ";
if (z.mImaginaryPart >= 0.0)
{
    output << " + " << z.mImaginaryPart << "i)";
}
else
{
    output << "- " << -z.mImaginaryPart << "i)";
}
}

имеет тип возврата std::ostream & Однако он ничего не возвращает. Я думаю, что это опечатка Должна быть

std::ostream& operator<<(std::ostream& output,
                     const ComplexNumber& z)
{
output << "(" << z.mRealPart << " ";
if (z.mImaginaryPart >= 0.0)
{
    output << " + " << z.mImaginaryPart << "i)";
}
else
{
    output << "- " << -z.mImaginaryPart << "i)";
}

return output;
}
person Vlad from Moscow    schedule 24.02.2014