c ++ friend функция - перегрузка операторов istream >> - PullRequest
Новая
       37

c ++ friend функция - перегрузка операторов istream >>

12 голосов
/ 22 июля 2010

Мой вопрос касается функций друзей, а также перегрузки << и >>.Из моего понимания я думал, что функции друзей могут (и должны) обращаться к закрытым переменным-членам напрямую.Однако в том случае, если у меня есть здесь, компилятор примет мой файл .cxx, только когда я использовал функции «get» для получения каждой закрытой переменной.

Вот мой заголовочный файл 

class BigNum 
public:

// CONSTRUCTORS and DESTRUCTORS
    BigNum();                            
    BigNum(int num, size_t optional_base = 10);                         
    BigNum(const char strin[], size_t optional_base = 10);

// MEMBER FUNCTIONS
    size_t get_digit(size_t index) const;
    size_t get_used() const;
    size_t get_capacity() const;
    size_t get_base() const;
    bool get_sign() const;

// FRIEND FUNCTIONS
    friend std::ostream& operator<<(std::ostream &os, const BigNum &bignum);
    friend std::istream& operator>>(std::istream &is, BigNum &bignum);

private:
    size_t base;            
    size_t *digits;          
    bool positive;          
    size_t used;

Здесьмой соответствующий файл .cxx с реализациями для функций-друзей 

#include "file.h"
#include <cstdlib>
#include <iostream>
#include <string>
#include <cstring>

using namespace std;

std::ostream& operator <<(std::ostream &os, const BigNum &bignum)
{
if (bignum.get_sign() == false)
    os << '-';

for (size_t i = 0; i < bignum.get_used(); ++i)
    os << bignum.get_digit(bignum.get_used() - i - 1);

return os;
}

std::istream& operator >>(std::istream &is, BigNum &bignum)
{
for (size_t i = 0; i < bignum.get_used(); ++i)
    is >> bignum.digits[i];

return is;
}

Так что в этом отношении операторы-друзья выше скомпилированы правильно.Однако почему мой оператор >> может напрямую обращаться к одной закрытой переменной (is >> bignum.digits [i]), а остальные приватные переменные необходимо получить с помощью функции get

Ниже,когда я пытаюсь написать операторы перегрузки в этом отношении (как я думал, функции-друзья должны правильно вызывать закрытые переменные): 

std::ostream& operator <<(std::ostream &os, const BigNum &bignum)
{
if (bignum.positive == false)
    os << '-';

for (size_t i = 0; i < bignum.used; ++i)
    os << bignum.digits[used - i - 1];

return os;
}

std::istream& operator >>(std::istream &is, BigNum &bignum)
{
for (size_t i = 0; i < bignum.used); ++i)
    is >> bignum.digits[i];

return is;
}

Я получаю следующие ошибки.

BigNum2.cxx: In function `std::ostream&
   csci2270_hw1B::operator<<(std::ostream&, const csci2270_hw1B::BigNum&)':
BigNum2.cxx:201: error: `used' undeclared (first use this function)
BigNum2.cxx:201: error: (Each undeclared identifier is reported only once for
   each function it appears in.)
BigNum2.cxx: In function `std::istream&
   csci2270_hw1B::operator>>(std::istream&, csci2270_hw1B::BigNum&)':
BigNum2.cxx:208: error: syntax error before `)' token

Используемый мной компилятор - g ++ (версия 3.3.1).Спасибо за любую помощь, спасибо.

Исправлено:

Я обновил код, чтобы объект bignum мог обращаться к закрытым переменным.Я сделал следующее с перегрузкой оператора друга <<, и она скомпилировалась нормально.Спасибо за комментарии, это была ошибка новичка. </p> 

std::ostream& operator <<(std::ostream &os, const BigNum &bignum)
{
if (bignum.positive == false)
    os << '-';

for (size_t i = 0; i < bignum.used; ++i)
    os << bignum.digits[bignum.used - i - 1];

return os;
}

Однако компилятор все еще выдает ошибки для >> оператора

BigNum2.cxx: в функции std::istream& csci2270_hw1B::operator>>(std::istream&, csci2270_hw1B::BigNum&)': BigNum2.cxx:208: error: syntax error before) 'токен

Предполагается, что >> читает число, а закрытая переменная-член 'used' должна записывать длину массива.Я все еще несколько озадачен тем, почему компилятор принимает 

std::istream& operator >>(std::istream &is, BigNum &bignum)
{
for (size_t i = 0; i < bignum.get_used()); ++i)
    is >> bignum.digits[i];

return is;
}

вместо: 

std::istream& operator >>(std::istream &is, BigNum &bignum)
{
for (size_t i = 0; i < bignum.used); ++i)
    is >> bignum.digits[i];

return is;
}

Есть мысли?спасибо.

Ответы [ 3 ]

9 голосов
/ 22 июля 2010

Функция друга имеет доступ к закрытым данным класса, но она не получает указатель this, чтобы сделать это автоматическим, поэтому каждый доступ к данным класса (частным или другим) имеет быть квалифицированным Например это: 

os << bignum.digits[used - i - 1];

должно быть: 

os << bignum.digits[bignum.used - i - 1];
5 голосов
/ 22 июля 2010

Вы не указали used в первой функции - должно быть bignum.used. Перегрузки операторов определены в глобальной области видимости, поэтому они не получают указатель this. Однако функции друзей имеют доступ к закрытым членам класса. 

std::ostream& operator <<(std::ostream &os, const BigNum &bignum) 
{ 
    if (bignum.positive == false) 
        os << '-'; 

    for (size_t i = 0; i < bignum.used; ++i) 
        // Note "bignum.used", instead of "used".
        os << bignum.digits[bignum.used - i - 1];     
    return os; 
} 

std::istream& operator >>(std::istream &is, BigNum &bignum) 
{ 
    for (size_t i = 0; i < bignum.used; ++i) 
        is >> bignum.digits[i]; 

    return is; 
}
2 голосов
/ 29 ноября 2014

Кажется, что в следующей строке сразу после bignum.used есть лишний ')'. 

 for (size_t i = 0; i < bignum.used**)**; ++i)
...