我有一个浮点溢出问题

时间:2009-11-02 02:19:54

标签: c++ floating-point overflow

嗯,我是初学者,这是我作为计算机科学专业的一年。我正在努力做到 我的课本练习,让我使用一个名为MovieData的结构 允许我在MovieData时初始化成员变量的构造函数 结构已创建。这是我的代码的样子:

#include <iostream>
#include <iomanip>
#include <string>
using namespace std;

// struct called MovieData
struct MovieData
{
    string title;
    string director;
    unsigned year;
    unsigned running_time;
    double production_cost;
    double first_year_revenue;

    MovieData() // default constructor
    {
        title = "Title";
        director = "Director";
        year = 2009;
        running_time = 90;
        production_cost = 1000000.00;
        first_year_revenue = 1000000.00;
    }
    // Constructor with arguments:
    MovieData(string t, string d, unsigned y, unsigned r, double p, double f)
    {
        title = t;
        director = d;
        year = y;
        running_time = r;
    }
};

// function prototype:
void displayMovieData(MovieData);

// main:
int main()
{
    // declare variables:
    MovieData movie, terminator("Terminator", "James Cameron", 1984, 120, 5000000, 2000000);

    // calling displayMovieData function for movie and terminator
    // so it will display information about the movie:
    displayMovieData(movie);
    displayMovieData(terminator);

    return 0;
}

// displayMovieData function:
// It receives struct MovieData variable as
// an argument and displays that argument's
// movie information to the user.
void displayMovieData(MovieData m)
{
    cout << m.title << endl;
    cout << m.director << endl;
    cout << m.year << endl;
    cout << m.running_time << endl;
    cout << fixed << showpoint << setprecision(2);
    cout << m.production_cost << endl;
    cout << m.first_year_revenue << endl << endl;
}

这是我收到的输出:

Title
Director
2009
90
1000000.00
1000000.00

Terminator
James Cameron
1984
120
-92559631349317830000000000000000000000000000000000000000000000.00
-92559631349317830000000000000000000000000000000000000000000000.00

Press any key to continue . . .

在Microsoft Visual C ++ 2008速成版上编译。

我的问题是,这是否由于双数据类型的溢出而发生?我甚至尝试使用long double,同样的事情发生。即使我使用5mil作为production_cost和2mil作为first_year_revenue,两个数字输出都是相同的。使用我的默认构造函数正确打印出1000000.在这种情况下我使用正确的数据类型吗?我希望它是双倍的,因为它是货币数字,美元和美分。

感谢您的帮助。抱歉,我的问题很长。这是我关于SO的第一篇文章,所以关于发布问题的正确格式的任何反馈都会很棒,谢谢!

6 个答案:

答案 0 :(得分:8)

感谢您发布完整的代码,现在问题显而易见了。以下功能是问题:

MovieData(string t, string d, unsigned y, unsigned r, double p, double f)
{
    title = t;
    director = d;
    year = y;
    running_time = r;
}

您已省略以下陈述:

    production_cost = p;
    first_year_revenue = f;

如果没有这些语句,使用上述构造函数时不会初始化production_costfirst_year_revenue

此练习强调,在Stack Overflow上发布问题时,需要发布您正在使用的完全代码。您发布的代码的第一个版本不同,并且不包含此错误。

答案 1 :(得分:3)

修复阻止代码编译的错误(缺少分号,大写M而不是小写m)我得到以下内容:

#include <iostream>
#include <iomanip>

using namespace std;

struct MovieData
{
    string title;
    string director;
    unsigned year;
    unsigned running_time;
    double production_cost;
    double first_year_revenue;

    MovieData() // My default constructor
    {
        title = "Title";
        director = "Director";
        year = 2009;
        running_time = 90;
        production_cost = 1000000.00; // this one comes out ok.
        first_year_revenue = 1000000.00; // this one comes out ok as well.
    }
    // This is my constructor with arguments:
    MovieData(string t, string d, unsigned y, unsigned r, double p, double f)
    {
        title = t;
        director = d;
        year = y;
        running_time = r;
        production_cost = p;
        first_year_revenue = f;
    }
};

void displayMovieData(MovieData m)
{
    cout << m.title << endl;
    cout << m.director << endl;
    cout << m.year << endl;
    cout << m.running_time;
    cout << fixed << showpoint << setprecision(2);
    cout << m.production_cost << endl;
    cout << m.first_year_revenue << endl << endl;
}

int main()
{
  MovieData terminator(
    "Terminator", "James Cameron", 1984, 120, 5000000, 2000000);
  displayMovieData(terminator);
  return 0;
}

编译并运行会重现您的问题...:

$ g++ -Wall --pedantic z.cc
$ ./a.out
Terminator
James Cameron
1984
1205000000.00
2000000.00

$ 

请复制并粘贴我在此处提供的代码,并告诉我们发生了什么(以及使用什么编译器,平台等等 - 我在MacOSX 10.5上使用gcc 4.0.1。)

答案 2 :(得分:2)

不,你不会用任何电影的收入或制作成本超出双倍范围。

我的猜测是问题在于你的displayMovieData函数。你可以将代码发布到那个吗? 如果你打电话给像printf这样的东西,你可以得到像这样打印的奇怪值,并且它会在单打和双打之间混淆。或者,如果您传递%d而不是%f ...

答案 3 :(得分:1)

像其他答案一样,我不确定问题可能是什么,因为代码似乎没问题。但是,请尝试替换此声明:

void displayMovieData(MovieData m)

void displayMovieData(const MovieData &m)

看看你的情况是否有所改善。有关这两个声明之间差异的更多信息,请参阅最近的问题Why is it preferable to write func( const Class &value )?

我应该强调,这应该更改程序的行为,但如果有关于编译器和/或运行时环境的某些内容存在错误,则上述代码更改可能有助于隔离问题。 / p>

答案 4 :(得分:1)

您的输出与您显示的代码不符 - 您省略了'&lt;&lt;在您的输出声明包含的运行时间之后结束。这总是让我们的调试变得更难。

以下是您的代码在MacOS X 10.5.8(Leopard)上使用G ++ 4.0.1正常工作。

#include <string>
using namespace std;

struct MovieData
{
    string title;
    string director;
    unsigned year;
    unsigned running_time;
    double production_cost;
    double first_year_revenue;

    MovieData() // My default constructor
    {
        title = "Title";
        director = "Director";
        year = 2009;
        running_time = 90;
        production_cost = 1000000.00; // this one comes out ok.
        first_year_revenue = 1000000.00; // this one comes out ok as well.
    }
    // This is my constructor with arguments:
    MovieData(string t, string d, unsigned y, unsigned r, double p, double f)
    {
        title = t;
        director = d;
        year = y;
        running_time = r;
        production_cost = p;
        first_year_revenue = f;
    }
};

#include <iostream>
#include <iomanip>
using namespace std;

void displayMovieData(MovieData m)
{
    cout << m.title << endl;
    cout << m.director << endl;
    cout << m.year << endl;
    cout << m.running_time << endl;
    cout << fixed << showpoint << setprecision(2);
    cout << m.production_cost << endl;
    cout << m.first_year_revenue << endl << endl;
}

int main()
{
    MovieData def;
    MovieData terminator("Terminator", "James Cameron", 1984, 120, 5000000, 2000000);
    MovieData terminator2("Terminator 2", "James Cameron", 1984, 120, 5000000.0, 2000000.0);
    displayMovieData(def);
    displayMovieData(terminator);
    displayMovieData(terminator2);
}

我得到的输出是:

Title
Director
2009
90
1000000.00
1000000.00

Terminator
James Cameron
1984
120
5000000.00
2000000.00

Terminator 2
James Cameron
1984
120
5000000.00
2000000.00

我最好的论文(上面的数据不支持)是你在构造函数的调用中没有从'int'到'double'的转换,但我承认我不明白这是怎么回事发生。

答案 5 :(得分:0)

我会尝试将.0添加到数字并使它们成为双打。尝试从整数到双精度进行转换时可能会感到困惑。

这将反映您在默认construtor中的工作情况。