4

我必须创建一个程序来将罗马数字转换为十进制数字，我将垃圾值作为输出。事实是我已经仔细检查了我的逻辑，它似乎是正确的。

我该如何纠正？

这是我的代码：

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

class RomanType
{
  char str[10];
  int d;
public:
  void accept()
  {  
    cout<<"Enter Roman No. in capitals:"<<endl;
    cin>>str;
    convert(str);
  }

  void convert(char str1[10])
  {
    int j=0;

    for(j=0;j<strlen(str1);j++)
    {

    if( str1[j]=='I')
    {
        if(str1[j+1]=='V' || str1[j+1]=='X')
        {
            d=d-1;
            cout<<j<<endl;
        }
        else
        {
            d=d+1;
            cout<<d<<endl;
        }
    }

    if ( str1[j]=='V')
        d=d+5;

    if(str1[j]=='X')
    {
        if(str1[j+1]=='L' || str1[j+1]=='C')
            d=d-10;
        else
            d=d+10;
    }

    if(str1[j]=='L')
        d=d+50;

    if( str1[j]=='C')
    {
        if(str1[j+1]=='D' || str1[j+1]=='M')
            d=d-100;
        else
            d=d+100;
    }

    if(str1[j]=='D')
        d=d+500;

    if(str1[j]=='M')
        d=d+1000;
    }
  }

  void display()
  {
    cout<<"It's decimal equivalent is="<<d<<endl;
  }
};

main()
{
  RomanType obj;
  obj.accept();
  obj.display();
}
4

4 回答 4

2

几点：

  • 不要直接跳到解析高价值的罗马字。I仅以、V和开头X（即首先目标 1 到 10，然后是 11 到 20，然后是 21 到 39、40 到 99、100 到 499 等）
  • 不要假设 ifI是给定的，它是在 or 之前或之后给出V的X。它可能是为自己给出的（例如II- 你其他部分假设了一些东西）。
  • 赋值为零_ d_
  • 做单步调试，观察d变量值等。如果调试器不好或不可用，请在每个步骤/迭代中输出值。
  • [添加]您不需要传递str给函数convert，因为它们属于同一类，并且convert可以/将读取相同的内容。
于 2014-11-12T15:32:50.063 回答
1

您没有将 d 初始化为 0

在转换中将 d=0 放在开头

于 2014-11-12T15:32:03.853 回答
0

您没有初始化d为 0。请在函数顶部添加以下内容convert：

void convert(char str1[10])
{
    int j=0;
    d = 0;
    . . .
于 2014-11-12T15:31:26.137 回答
0

好的，感谢您的帮助。现在已经解决了。我犯了一个错误，并在 convert() 中再次初始化了 d，因此它已成为局部变量。见评论：

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

class RomanType
{
char str[10];
int d;
public:
void accept()  // UNNECESSARILY NOT PASSING ANY STRING
{
    cout<<"Enter Roman No. in capitals:"<<endl;
    cin>>str;
    convert();
}

void convert()
{
     d=0;// PREVIOUSLY WRIITEN int d=0; so that was the mistake. Yay! it's solved :D


    for(int j=0;j<10;j++)
    {

    if( str[j]=='I')
    {
        if(str[j+1]=='V' || str[j+1]=='X')
        {
            d=d-1;
           // cout<<d<<endl;
        }
        else
        {
            d=d+1;
            //cout<<d<<endl;

        }

    }

    else if ( str[j]=='V')
        d=d+5;

    else if(str[j]=='X')
    {
        if(str[j+1]=='L' || str[j+1]=='C')
            d=d-10;
        else
            d=d+10;
    }

    else if(str[j]=='L')
        d=d+50;

    else if( str[j]=='C')
    {
        if(str[j+1]=='D' || str[j+1]=='M')
            d=d-100;
        else
            d=d+100;
    }

    else if(str[j]=='D')
        d=d+500;

    else if(str[j]=='M')
        d=d+1000;
    }


}

void display()
{
    cout<<"It's decimal equivalent is="<<d<<endl;
}
};

main()
{
RomanType obj;
obj.accept();
obj.display();

}
于 2014-11-12T16:30:09.637 回答