运算符 - 类的重载

时间:2014-01-17 08:27:50

标签: c++

在我的项目中,用户输入2个浮点数(整数部分和小数部分,用(。)分隔。(+)运算符成功重载。但是在( - )中没有错误,但程序没有似乎工作正常。答案是错误的,例如,当ob1 = 216.20和ob2 = 213.45时,输出为:3.-35(这是错误的) 甚至当ob1 = 213.45和ob2 = 216.20时,输出为9999999999999999997.25。

谢谢

#include <conio.h>
#include <iostream>
#include <stdlib.h>
#include <iomanip>
#include <string.h>
using namespace std ;
class Google
{
private:
    int integer[20];
    int decimal[40];
public:
    friend istream &operator>>(istream &z,Google &t);
    Google operator-(Google T);
    friend ostream &operator<<(ostream &r,Google &p);
};// End Of Google Class
istream &operator>>(istream &z,Google &t)
{
    char a[100];
    cout<<"\n"<<" Please Enter The Google Number, Sepatare with (.)"<<"\n";
    z.get(a ,99);
    z.get();
    char  *x , *y;
    x = strtok(a,".");
    //cout<<x<<"\n";
    y = strtok('\0'," ");
    //cout<<y<<"\n";
    int k1=strlen(x);
    //cout<<k1<<"\n";
    int k2=strlen(y);
    //cout<<k2<<"\n";
    for( int i = 0; i < 20-k1; i++)
    {
        t.integer[i] = 0 ;
    }
    int j = 0;
    for( int i = 20-k1; i < 20; i ++ )
    {
        t.integer[i] = x[j] - 48 ;
        j ++ ;
    }
    for( int i = k2 ; i < 40 ; i ++ )
    {
        t.decimal[i] = 0 ;
    }
    for( int i = 0 ; i < k2 ; i ++ )
    {
        t.decimal[i] = y[i] - 48 ;
    }
    // for (int i =0; i< k2 ; i++) {
    //cout<<"\n"<<t.decimal[i]<<"\n";
    //} Checking Out Outcome
    return z ;
}
Google Google::operator-(Google T)   {
    Google M1;
    for(int k = 0 ; k < 40 ; k++ )
    M1.decimal[k] = 0 ;
    for(int p = 0 ; p < 20 ; p ++ )
    M1.integer[p] = 0 ;
    for(int j = 39 ; j > 0 ; j -- )
    {
        if (this->decimal[j]<T.decimal[j])
        {
            this->decimal[j-1] -= 1 ;
            this->decimal[j] += 10 ; 
        }
        M1.decimal[j] = this->decimal[j] - T.decimal[j] ;
    }
    if (this->decimal[0]<T.decimal[0])
    {
        M1.integer[19] -= 1  ;
        M1.decimal[0]  +=  10 ;
    }
    M1.decimal[0] = this->decimal[0]- T.decimal[0] ;
    for(int i=19 ; i> 0 ; i--)
    {
        if (this->integer[i] < T.integer[i])
        {
            this->integer[i-1] -= 1  ;
            this->integer[i] +=  10 ;
        }
        M1.integer[i] = this->integer[i] - T.integer[i] ;
    }
    return M1;
} //end of Operator 
ostream & operator <<(ostream &r ,Google &p )
{
    int  k1 = 0 , k2 = 0 ;
    for( int  i = 0 ; p.integer[i] == 0 ; i ++ )
    k1 ++ ;
    for( int  i = k1 ; i < 20 ; i ++ ) 
    r << p.integer[i] ;
    if( !p.integer[19] )
    r << "0"  ;
    for( int  i = 39 ; p.decimal[i] == 0 ; i -- )
    k2 ++ ;
    cout<<"."  ;
    for( int  i = 0 ; i < 40 - k2 ; i ++ )
    r << p.decimal[i] ;
    if( !p.decimal[0] )
    r << "0"  ;
    return  r ;
}

void main () {
    Google ob1;
    Google ob2;
    cin>>ob1;
    cin>>ob2;
    cout<<"ob1 = " <<ob1<<"\n"<< "ob2 = " <<ob2<<"\n" <<endl;
    Google ob4;
    ob4 = ob1-ob2;
    cout<<"\n"<<"Result of (-) Will Be:"<<"\n";
    cout<<ob4;
    cout<< "\n";
    getch();
} 

2 个答案:

答案 0 :(得分:1)

我发现了一些问题。 当您对decimal[0]进行减法时,从1减去M1,而对于其他小数,则使用this。可能它们都应该是相同的(可能是M1,因为当你从中减去某些东西时你不想修改this中存储的数字)。当您用新数字覆盖它时,请注意不要丢失-1

您正在做的另一件事是没有正确处理负数。如果从0中减去1,则代码将填充integer第9部分。这种格式称为9的补码,基本上是为了得到每个数字(包括小数部分)提取9整数[x]所需的负数。处理它的另一种方法是首先比较两个数字,如果结果是负交换操作数(结果现在是正数)并改变符号。

这也可能是其他问题。拿一堆非常简单的例子来调试代码。

答案 1 :(得分:0)

基本上,你在这里做定点运算。看起来你有一个问题,即进位/借位没有正确地从小数部分传播到整数部分。 (例如5.222 - 0.223)你有没有考虑过这个案子?

给你三个提示:

  • 在类中添加一个构造函数,清除两个数组。这样可以避免在“ - ”运算符或其他任何地方明确地执行此操作。
  • 考虑使用一个数字而不是两个数组,并在输入/输出数字时插入/删除小数点。这将使您的算法更简单,并使代码更易于维护。
  • 不要在代码中使用20之类的“幻数”。将static const int IntDigits=20;定义为私有类成员,并使用它。