Функция друга С++ не имеет доступа к частным членам

Предполагается, что это строковый класс с множеством операторов и функций, включая две функции-друга. И эти два причиняют мне некоторую проблему, потому что компилятор говорит, что они не могут получить доступ к закрытым членам. Вот мой string.h:

#include <iostream>
#ifndef STR_H
#define STR_H

namespace MyStr
{
class Str
{
private:
    unsigned int length;
    char *data;
public:
    Str();
    Str(const Str&);
    Str(const char*);
    Str(char c, unsigned int db);
    ~Str();
    char* cStr() const;
    unsigned int getLength() const;

множество неучтенных функций здесь...

    friend int operator/ (const Str&, char);
    friend std::ostream& operator<< (std::ostream&, const Str&);
};
}
#endif /* STR_H */

здесь main.cpp:

#include <iostream>
#include "Str.h"

using namespace std;
using namespace MyStr;

ostream& operator<< (ostream& out,const Str& str)
{
    for (int i=0; i<str.length; i++)
    {
        out<<str.data[i];
    }
    out<<endl;
    return out;
}

int operator/ (const Str& str, char c)
{
    for (int i=0; i<str.length; i++)
    {
        if(str.data[i]==c) return i;
    }
    return -1;
}

Этот код не будет компилироваться, компилятор утверждает, что члены Str являются закрытыми.

Ответы

Ответ 1

Вы должны уделять больше внимания пространствам имен.

class Str {
private:
    unsigned int length;
    char *data;
public:
    Str(){}
    Str(const Str&){}
    Str(const char*){}
    Str(char c, unsigned int db){}
    // maybe something more...
    friend int operator/ (const Str&, char);
    friend std::ostream& operator<< (std::ostream&, const Str&);
};

ostream& operator<< (ostream& out,const Str& str)
{
    for (int i=0; i<str.length; i++)
        out<<str.data[i];
    out<<endl;
    return out;
}

int operator/ (const Str& str, char c)
{
    for (int i=0; i<str.length; i++)
        if(str.data[i]==c) return i;

    return -1;
}

int main()
{
    Str s;
    cout<<s;
    return 0;
}

Вы получаете ошибку из-за несогласованных пространств имен. Если вы предпочитаете придерживаться MyStr, тогда вы должны добавить пространство имен MyStr для перегруженных операторов друзей. Так вы можете это сделать: (операторы должны быть определены в пространстве имен MyStr)

namespace MyStr {  
    ostream& operator<< (ostream& out,const Str& str)
    {
        for (int i=0; i<str.length; i++)
        {
             out<<str.data[i];
         }
        out<<endl;
        return out;
    }

    int operator/ (const Str& str, char c)
    {
        for (int i=0; i<str.length; i++)
        {
            if(str.data[i]==c) return i;
        }
        return -1;
    }
}

Ответ 2

Когда вы объявляете функции друга внутри Str, они считаются находящимися в непосредственном вложенном пространстве имен, MyStr.

Операторы, которые вы определяете, находятся в глобальном пространстве имен, поэтому компилятор считает, что это два совершенно разных оператора, а не друзья.

Вы можете решить эту проблему, добавив

namespace MyStr
{

}

вокруг операторов в файле .cpp.