Паттерны проектирования (Фабрики)

@Lotus34 · Регистрация: 26.10.2012

Author24 — интернет-сервис помощи студентам

Добрый вечер. Есть игра "Морской бой" из учебника Павловской Т.А .
Задача: добавить в ней какой-либо шаблон проектирования.

Я подумал, что лучшим решение будет реализовать паттерн Фабрика, но у меня возникают проблемы.

Есть заголовочный файл:

C++

//////////////////////////////////////////////////////////////////////////// Navy.h
#include "Ship.h"
#include <assert.h>
#define DECK '+' // исправная клетка-палуба
#define DAMAGE 'X' // разрушенная клетка-палуба
#define MISS 'o' // пустая клетка, в к-рую упал снаряд
typedef unsigned char Field[N][N]; // игровое поле
typedef std::map<Cell, int> ShipMap; // словарь
// ассоциаций "клетка - индекс корабля"
enum CurrentState { Miss, Damage, Kill }; // результат попадания в цель
 
 
struct Space { // ------------ Класс Space - информационное пространство для обмена
public: // информацией между игроками
static Cell u_fire; // огонь от пользователя
static Cell r_fire; // огонь от робота (компьютера)
static CurrentState u_state; // состояние пользователя
static CurrentState r_state; // состояние робота
static int step;
};
 
enum Navy_ID { UserNavy_ID=0, RobotNavy_ID };
 
 
 
class Navy : public Space { //---------------------------------- Базовый класс Navy
 
public:
    virtual ~Navy() {}
    static Navy* createNavy( Navy_ID id );
Navy();
void AllocShip( int, int, std::string ); // разместить корабль
void Show() const; // показать поля ownField и enemyField
int GetInt(); // ввод целого числа
bool IsLive() { return ( nLiveShip > 0 ); } // мы еще живы?
Rect Shell( Rect ) const; /* вернуть "оболочку" для заданного прямоугольника
(сам прямоугольник плюс пограничные клетки) */
void AddToVetoSet( Rect ); // добавить клетки прямоугольника
// в множество vetoSet.
protected:
Ship ship[10]; // корабли флота
Field ownField; // мое игровое поле
Field enemyField; // игровое поле неприятеля
ShipMap shipMap; // словарь ассоциаций "клетка - индекс корабля"
CellSet vetoSet; // множество "запрещенных" клеток
CellSet crushSet; // множество "уничтоженных" клеток
int nLiveShip; // количество боеспособных кораблей
 
 
 
};
 
 
class UserNavy : public Navy { //---------------------------------- Класс UserNavy
public:
UserNavy() { Allocation(); }
void Allocation();
void FireOff(); // выстрел по неприятелю
void ResultAnalys(); // анализ результатов выстрела
void GetFire(); // "прием" огня противника
void FillDeadZone( Rect r, Field& ); // заполнить
// пробелами пограничные клетки для r
};
 
 
class RobotNavy : public Navy { // -------------------------------- Класс RobotNavy
public:
RobotNavy();
void Allocation();
void FireOff(); // выстрел по неприятелю
void ResultAnalys(); // анализ результатов выстрела
void GetFire(); // "прием" огня противника
private:
bool isCrushContinue; // предыдущий выстрел был успешным
bool upEmpty; // у поврежденного корабля противника
// нет "живых" клеток в верхнем направлении
};
 
 
Navy* Navy::createNavy( Navy_ID id )
{
    Navy * p;
    switch (id)
    {
        case UserNavy_ID:
            p = new UserNavy();           
            break;      
        case RobotNavy_ID:
            p = new RobotNavy();           
            break;                  
        default:
            assert( false);
    }
    return p;
}
 
class Singleton
{
    public:
        static Singleton& instance();
        void print(char*);
    
    protected:
        static Singleton Instance;
        
    private:
        Singleton();
        ~Singleton();
    
};

Так вот, т.к классы usernavy и RobotNavy наследуются от класса navy я подумал, что можно было бы сделать из них фабрику. Пробовал уже по разному задавать, но в конечном итоге я не могу наладить использование этой фабрики в главном файле.

C++

#include <iostream>
#include "Navy.h"
#include <locale>
#include <vector>
using namespace std;
int main() {
setlocale(LC_ALL,"Russian");
 
UserNavy userNavy; // Начальная позиция
RobotNavy robotNavy;
userNavy.Show();
while ( userNavy.IsLive() && robotNavy.IsLive() ) {
if ( Space::u_state != Miss ) { // ----------------- Выстрел пользователя
cout << "пропускается...: <Enter>" << endl;
cin.get(); }
else {
userNavy.FireOff();
robotNavy.GetFire();
userNavy.ResultAnalys();
if ( !robotNavy.IsLive() ) { userNavy.Show(); break; }
}
if ( Space::r_state != Miss ) // ------------------ Выстрел робота
cout << "\nВыстрел неприятеля: пропускается..." << endl;
else {robotNavy.FireOff();
userNavy.GetFire();
robotNavy.ResultAnalys();
}
userNavy.Show();
}
if ( userNavy.IsLive() ) cout << "\n:-))) Ура! Победа!!! :-)))" << endl;
else { cout << "\n:-((( Увы. Непрятель оказался сильнее." << endl;
}
cin.get();
}

Подскажите пожалуйста как тут поступить или мб есть паттерны по-проще и более уместные.

П,С Цель просто вставить шаблон, пусть он там и не будет особо нужен.

@castaway · 17.01.2015, 17:24

Не вижу у тебя никакой фабрики, вижу только синглтон, который не используется.

@Lotus34 · 17.01.2015, 18:20 **[ТС]**

Сообщение от castaway

Не вижу у тебя никакой фабрики, вижу только синглтон, который не используется.

Синглтон там вставлен так как я пытаюсь воткнуть разные шаблоны, и когда копировал код пытался сделать синглтон, но с ним я , в итоге, ещё сильнее запутался.

А фабрику я пытался инициализировать через:

C++

Navy* Navy::createNavy( Navy_ID id )
{
    Navy * p;
    switch (id)
    {
        case UserNavy_ID:
            p = new UserNavy();           
            break;      
        case RobotNavy_ID:
            p = new RobotNavy();           
            break;                  
        default:
            assert( false);
    }
    return p;
}

Делал я это по примерам, так что вполне могу ошибаться. С паттернами столкнулся вообще впервые и пытаюсь понять по примерам, но слабо получается.

@EVP · 18.01.2015, 13:44

Сообщение от Lotus34

П,С Цель просто вставить шаблон, пусть он там и не будет особо нужен.

По паттернам есть книжка Приемы объектно-ориентированного проектирования. Паттерны проектирования.

Пока язык плохо знаешь, за паттерны лучше не браться.
На мой взгляд, для фабрики скорее подходит игрок(Player), а не игра (Navy).
Игра должна работать с абстрактным игроком:

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <memory>
#include <random>
 
enum class EPlayer{ UserPlayer, RobotPlayer };
 
class Move
{
public:
    Move(int _x, int _y) :x(_x), y(_y){}
    int x;
    int y;
};
 
 
class Player{
public:
    virtual Move move() = 0;
    //сюда можно добавить методы инициализации поля и взаимодействия с игрой
};
 
 
class UserPlayer :public Player{
    Move move()override{ std::cout << "I'm User player" << std::endl; return Move(0, 0); }
};
 
class RobotPlayer :public Player{
    Move move()override{ std::cout << "I'm Robot player" << std::endl; return Move(std::rand(), std::rand()); }
};
 
std::unique_ptr<Player> createPlayer(EPlayer _player)
{
    switch (_player)
    {
    case EPlayer::UserPlayer:
        return std::make_unique<UserPlayer>();
    default:
    case EPlayer::RobotPlayer:
        return std::make_unique<RobotPlayer>();
    }
}
 
 
int main() {
    auto player1 = createPlayer(EPlayer::UserPlayer);
    auto player2 = createPlayer(EPlayer::RobotPlayer);
 
    auto move = player1->move();
    move = player2->move();
 
    system("pause");
}

@NeonLost · 18.01.2015, 14:39

пару строк тебе подкину

C++

1
2
3

UserNavy userNavy = Navy::createNavy(0); // Начальная позиция
RobotNavy robotNavy = Navy::createNavy(1);
userNavy->Show();

стоить обратить внимание, на айдишники(их следуеть поменять на соответствующие)
и на то, как вызываются функции теперь

@Avazart · 18.01.2015, 15:08

Сообщение от NeonLost

на айдишники(их следуеть поменять на соответствующие)

на enum поменять...
Ну и не

C++

1 2	UserNavy userNavy = Navy::createNavy(0); // Начальная позиция RobotNavy robotNavy = Navy::createNavy(1);

А

C++

1 2	BaseNavy* userNavy = createNavy(ntUser); BaseNavy *robotNavy = createNavy(ntRobot);

- Указатели на базовый/абстрактный класс.

Добавлено через 1 минуту

Сообщение от EVP

C++

class Player
{ 
   public: 
      virtual Move move() = 0; 
}

Виртуальный деструктор забыт.

@castaway 4986 / 3093 / 456 Регистрация: 10.11.2010 Сообщений: 11,169 Записей в блоге: 10
	17.01.2015, 17:24	2
	Не вижу у тебя никакой фабрики, вижу только синглтон, который не используется. 0

	18.01.2015, 15:08