Bigint +演算子

Question

私はbigintプロジェクトを行っています。なぜ、追加演算子がテストケースで正しく動作していないのかがわかりません。

おそらく不要なので、私は.hファイルを除外しています。

bigint.cpp

#include "bigint.h" 
#include<iostream> 
#include<fstream> 
#include<cstdlib> 
#include<cassert> 



bigint::bigint() 
{       //Default constructor that sets digit to ZERO 
    for (int i = 0; i < MAX; i++) 
    { 
    digits[i] = 0; 
    } 
} 


bigint::bigint(int n) 
{ 

for(int i = 0; i < MAX; ++i)  //Sets the digit to ZERO 
    digits[i] = 0; 

    for (int i = 0; n != 0 ; ++i) 
{ 
    digits[i] = (n % 10);  // 
      n = n/10; 
} 


} 


bigint::bigint(const char new_digits[]) 
{ 
int null = 0; 
int temp = 0; 

for(int i = 0; i < MAX; ++i) 
{ 
    digits[i] = 0; 
} 

while(new_digits[null] != '\0') 
    ++null; 
    --null; 
temp = null; 

for(int j = 0; j < MAX && temp >= 0; ++j) 
{ 
    digits[j] = new_digits[temp] - '0'; 
    temp -= 1; 
} 
} 


bool bigint::operator==(const bigint& equal) const 
{ 
int i = 0; 

while(i < MAX) 
{ 
    if(digits[i] != equal.digits[i]) 
    { 
     return false; 
    } 

    ++i; 
} 
return true; 
} 


std::ostream& operator<<(std::ostream& output, const bigint& source) 
{ 

int sub1 = MAX - 1; //subtracts 1 from the maximum size 

while(source.digits[sub1] == 0) 
{ 
    --sub1;       //EMPTY 
} 

while(sub1 > -1) 
{ 
    output << source.digits[sub1]; 
    --sub1; 
} 

std::cout << std:: endl; 

return output; 
} 

std::istream& operator>>(std::istream& in, bigint& source) 
{ 
char getdata[MAX]; 
char user_input; 
int i = 0; 



    in.get(user_input); 
    while(!in.eof() && user_input != ';') 
{ 
    in.get(user_input); 
    source.digits[i] = user_input; 
    ++i; 
} 

    source = bigint(getdata); 

    return in; 
} 

char bigint::operator[](const int i) 
{ 
return digits[i]; 
} 

bigint bigint::operator+(const bigint rhs) 
{ 
    bigint result; 
    int i = 0; 

    for(; i < MAX; ++i) 
    { 

     if((digits[i] + rhs.digits[i]) > 9) 
     { 
      digits[i+1] = digits[i+1] + 1 ; 


     } 

       result.digits[i] = (digits[i] + rhs.digits[i]); 
       result.digits[i] = result.digits[i] % 10; 
    } 

    return result; 




} 

MAIN.CPP（テストケース）プログラムがアサート（結果== "10"）でアボートこのテストケースで

int main() 
{ 
      // Setup fixture 
    bigint left("1"); 
    bigint right("9"); 
    bigint result; 

    // Test 
    result = (left + right); 

    Verify 
    assert(left == "1"); 
    assert(right == "9"); 
    assert(result == "10"); 

} 

。

しかし、私がassert（result == 10）を除いて同じテストケースを持っていれば、 プログラムが実行されます。

誰でも理由を言うことができますか？

Answer 1

まず、代入演算子bigint::operator=(const bigint&)を実装する必要があります。

は今、 operator+に、あなたはこのコードでは、左側のオブジェクトの内容を変更している：良いではありません

if((digits[i] + rhs.digits[i]) > 9) 
{ 
    digits[i+1] = digits[i+1] + 1 ; 
} 

を。あなたはこのコードを実行した場合たとえば、：

bigint x("5"); 
bigint y("6"); 

x+y; 
x+y; 

をあなたはxは17

次のものに終わるだろう、あなたはおそらくで渡す必要がありますbigint::operator引数に値渡しています参照番号（&）。

最後に、ここにあなたのインデントは、積極的に悪質なです：

while(new_digits[null] != '\0') 
    ++null; 
    --null; 

ここではループ本体には何ですか？そうですね、3行目ではありません。そのようなコードをインデントしないでください、それは子猫を泣かせます。プログラミングの子猫、少なくとも。

NB：ここには動的メモリ割り当てコードはありません。つまり、digitsはおそらく静的サイズの配列です。あなたがそれをやろうとすればそれが十分な大きさであることを確かめ、大きさを超えるとあなたが壊れることに気づくでしょう。