4

好吧，我是初学者，这是我主修计算机科学的一年。我正在尝试从我的教科书中做一个练习，让我使用一个名为的结构，该结构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;
}

这是我收到的输出：

标题
导向器
2009
90
1000000.00
1000000.00

终结者
詹姆斯卡梅隆
1984年
120
-925596313493178300000000000000000000000000000000000000000000000.00
-925596313493178300000000000000000000000000000000000000000000000.00

按任意键继续 。. .

在 Microsoft Visual C++ 2008 Express Edition 上编译。

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

感谢我提供的任何帮助。对不起我的长问题。这是我在 SO 上的第一篇文章，所以任何关于正确发布问题格式的反馈都会很棒，谢谢！

4

6 回答 6

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_cost并且first_year_revenue在使用上述构造函数时不会初始化。

本练习强调了在 Stack Overflow 上发布问题时发布您使用的确切代码的必要性。您发布的代码的第一个版本不同，并且不包含此错误。

于 2009-11-02T03:37:41.783 回答
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）。

于 2009-11-02T02:41:28.997 回答
2

不，你永远不会用任何电影的收入或制作成本超出两倍的范围。

我的猜测是问题出在您的 displayMovieData 函数中。您可以将代码发布到那里吗？
IIRC 如果你调用类似 printf 的东西，你会得到像这样打印的奇怪值，它会在单打和双打之间混淆。或者，如果您通过它%d而不是%f...

于 2009-11-02T02:25:54.350 回答
1

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

void displayMovieData(MovieData m)

和

void displayMovieData(const MovieData &m)

看看你的情况有没有好转。查看最近的问题为什么最好写 func( const Class &value )？有关这两个声明之间差异的更多信息。

我应该强调这不应该改变你的程序的行为，但是如果你的编译器和/或运行时环境有错误，上面的代码更改可能有助于隔离问题。

于 2009-11-02T02:47:54.050 回答
1

您的输出与您显示的代码不匹配 - 您在输出声称包含的运行时间之后省略了一个 '<< endl'。这总是让我们的调试更加困难。

这是您的代码在带有 G++ 4.0.1 的 MacOS X 10.5.8 (Leopard) 上正常工作。

#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”，但我承认我不明白这是怎么发生的。

于 2009-11-02T02:49:56.183 回答
0

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

这将反映您在工作的默认构造函数中所拥有的内容。

于 2009-11-02T04:18:15.370 回答