@TheFunGun36

std::queue::pop() не возвращает значение:

Перегрузку сделать не надо, класс queue написан не вами, и изменить его удастся только если создать класс наследник. Не думаю, что это то что вам нужно.

А вот что вам нужно, так это пересмотреть выбор структуры данных. printMonsters() - функция, которая выводит всех монстров, одновременно их всех удаляя. Предположу, что задача стояла просто вывести. Тогда, можно сделать так:
Да и вообще, почему бы не перегрузить operator<<, как это было сделано с монстрами выше?

С вашего позволения, некоторые рекомендации по дальнейшему написанию кода:
1) методы, не изменяющие сам объект при своём выполнении, лучше обозначать const: int Get_Health() const { ... };
2) передачу структуры выполнять через константную ссылку, если вы не собираетесь менять то, что вам дала вызывающая сторона: friend ostream& operator<<(ostream& out, const Monster& name) (name меняться не будет, поэтому передаём константную ссылку - const TypeName&);
3) поизучайте стандартные STL контейнеры - их много, и каждый из них имеет свои преимущества и недостатки в разных ситуациях: vector, list, forward_list, map, multimap, set и так далее;
4) старайтесь придерживаться единого стиля именования переменных/методов/функций. У вас в Monster есть поле monster с маленькой буквы, и поле Health с заглавной. Так-же, у вас есть метод Get_Monster с разделением в виде нижнего подчёркивания, и AddM без такового. Если у вас нет каких-либо предпочтений в этом вопросе, рекомендую использовать snake_case (все буквы малые, слова разделяются нижним подчёркиванием) для переменных, полей, функций и методов; PascalCase для имён типов (классы, using) и SCREAM_CASE для директив #define

это, как раз таки, не понятно. Что за приоритет? Когда был добавлен? Если да, замените в функции Battle::AddM monsters.push_front(monster); на monsters.push_back(monster);

Вернуться к обсуждению:
Как сделать грамотно вывод очереди в с++ C/C++

1