Как убрать ошибки взаимодействия между классами?

Question

[colorname]

Есть 2 класса: Screen (screen.h)и Window_mgr (window_mgr.h)
В классе Window_mgr объявлена функция clear принимающая параметр типа ScreenIndex (std::vector::size_type)
В классе Screen эта функция объявлена как дружественная
В файле window_mgr.cpp дано определение этой функции.

Подскажите в чем проблема и как ее решить?

#ifndef WINDOW_MGR_H
#define WINDOW_MGR_H

#include <vector>
#include "screen.h"

class Window_mgr
{
public:
	using ScreenIndex = std::vector<Screen>::size_type;
	void clear(ScreenIndex);
};

#endif

#include "window_mgr.h"

void Window_mgr::clear(ScreenIndex i)
{
	// some code
}

#ifndef SCREEN_H
#define SCREEN_H

#include "window_mgr.h"

class Screen
{
	friend void Window_mgr::clear(ScreenIndex);
};

#endif

При компиляции показываются 2 ошибки:

Answer 1

[code_panik]

Если структуру классов не нужно менять, то можно сделать как в рабочем примере.
Перед Window_mgr достаточно объявить (forward declare) class Screen. Просто нужно иметь ввиду, что в файле класса Window_mgr это неполный тип (incomplete type).
И во friend объявлении не хватает Window_mgr::ScreenIndex.

Comments

[code_panik]

colorname

А подскажите если это возможно, как в этой ситуации пофиксить код https://godbolt.org/z/hYPaGM8nr

Вынесите определение конструктора в window_mgr.cpp

Window_mgr::Window_mgr()
 : screens{ Screen(24, 80, ' ') } { }

Answer 2

[Dmitrii]

Нет необходимости делать публичную функцию дружественной. У вас что-то не так с заголовочными файлами, видимо
Попробуйте вынесите using из класса

Comments

[colorname]

Dmitrii Это все примеры из книги. Если вынесли using за класс то ничего не меняется, все те же ошибки. Если сделать весь класс дружественным то ошибок нет, но хочется разобраться все таки почему в таком исполнении не работает код

[Dmitrii]

colorname, ну как по мне выглядит так, что using как-то не так работает. Можете попробовать без него

[res2001]

colorname, У вас заголовки включают друг друга. В windows_mgr.h подключается screen.h, а в screen.h - windows_mgr.h. Это приводит к тому, что в screen.h не определен класс Windows_mgr и наоборот. Избавьтесь от этой зависимости.

[Dmitrii]

res2001, если там если там pragma once, то проблема ли это?

[res2001]

Dmitrii, Может и не проблема, надо проверить. Но прагму надо хотя бы написать.
colorname, Добавьте в самое начало обоих заголовков
#pragma once
include guard в этом случае надо убрать.
И сообщите о результатах компиляции.

Но правильнее все таки избавиться от такой перекрестной зависимости.

[colorname]

res2001, добавил pragma, ошибки те же самые. Я не понимаю почему в книге описан код именно так. Я скачал исходный код для текущей главы, так там вообще нет никаких инклудов, ни screen.h ни window_mgr.h не подключатся нигде. Не понимаю как вообще это должно работать.
Это еще и часть кода, если добавить и другие функции то ошибок там навалит с десяток другой. Хотелось бы все таки разобраться во всем.

[res2001]

colorname, Что за книжка?
Избавится от такой зависимости достаточно не сложно.
Надо выбрать класс в котором меньше всего зависимостей от соседа.
Например удалим зависимости в screen.h:
Тогда из screen.h удаляем включение заголовка windows_mg.h.
Возможно надо добавить в screen.h предварительное объявление класса Windows_mgr:
class Windows_mgr;
Как решили выше, объявление friend не нужно - удаляем. Реализацию методов Screen, которые используют Windows_mgr выносим в screen.cpp. В screen.cpp подключаем как screen.h так и windows_mgr.h.
Таким образом зависимость перенесена в screen.cpp и теперь screen.h может быть успешно подключен в windows_mgr.h.
Как-то так в общих чертах.