关于友元类的成员函数

[coffee_sugar]

小弟又来请教诸位大神了……
这次是关于友元函数的。
首先，我知道友元函数怎么去声明了，但是我写的这两个类让我十分纠结应该怎么搞……

class Screen {
public:
//注意这个声明
    friend void Window_mgr::clear(ScreenIndex);
    typedef std::string::size_type pos;
    Screen() = default;
    Screen(pos ht,pos wd,char c):height(ht), width(wd),
        contents(ht*wd, c){}
    Screen(pos ht,pos wd):height(ht),width(wd),contents(ht*wd,' '){};
    char get() const
        { return contents[cursor];}
    inline char get(pos ht,pos wd) const;
    inline Screen &move(pos r, pos c);
    Screen &set(char c);
    Screen &set(pos ht,pos wd,char c);
    Screen &display(std::ostream &os) {
        do_display(os);
        return *this;
    }
    const Screen &display(std::ostream &os) const {
        do_display(os);
        return *this;
    }
private:
    pos cursor = 0;
    pos height = 0, width = 0;
    std::string contents;
    void do_display(std::ostream &os) const{
        os<<contents;
    }
};

Screen &Screen::move(pos r, pos c) {
    pos row = r * width;
    cursor = row + c;
    return *this;
}

char Screen::get(pos r, pos c) const{
    pos row = r * width;
    return contents[row +c];
}

第二个类

class Window_mgr{
public:
    using ScreenIndex = std::vector<Screen>::size_type;
    void clear(ScreenIndex);
private:
//注意这里，如果没有这个private的话我就能搞定了
    std::vector<Screen> screens{Screen(24, 80, ' ')};
};

求大神能当做一个简单的健脑题帮忙解答下……我今天刚入手了c++ primer第五版的习题集，但那个练习题要比我这个简单，看了下也没啥参考价值。所以特来此地求解

[勤奋的小游侠]

把这句void clear(ScreenIndex); 直接换成void clear(Screen); 就容易多了

[羽飞]

我感觉SCreen的clear不应该是friend，直接public成员，让Window_mgr调用就好了

[coffee_sugar]

引用 2 楼 hnwyllmm 的回复:

我感觉SCreen的clear不应该是friend，直接public成员，让Window_mgr调用就好了

那个clear的实现是这样的：

void Window_mgr::clear(ScreenIndex i) {
    Screen &s=screens[i];
    s.contents =std::string(s.height*s.width,' ');
}

就这个方法来说，我觉着contents是private的应该没啥问题。是不是遇到这种情况就是类的设计有问题？

[fly_dragon_fly]

contents是private，但没有成员函数可以修改

[coffee_sugar]

其实我是在想，是不是有的时候友元是声明不出来的。因为以前只碰过c#和一点点的java，友元的使用上我没啥经验。这个东西是不是应该尽量避免使用(我是这么认为的)？

[羽飞]

引用 3 楼 kzl332058242 的回复:

[quote=引用 2 楼 hnwyllmm 的回复:] 我感觉SCreen的clear不应该是friend，直接public成员，让Window_mgr调用就好了

那个clear的实现是这样的：

void Window_mgr::clear(ScreenIndex i) {
    Screen &s=screens[i];
    s.contents =std::string(s.height*s.width,' ');
}

就这个方法来说，我觉着contents是private的应该没啥问题。是不是遇到这种情况就是类的设计有问题？[/quote] 明显改成public void cliear(void); 这样不是更简洁.......