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

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

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

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

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

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
//////////////////////////////////////////////////////////////////////////// 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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
#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++
1
2
3
4
5
class Player
{ 
   public: 
      virtual Move move() = 0; 
}

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

Новые блоги и статьи Все статьи Все блоги /
Новый ноутбук volvo 07.12.2025 Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro	Музыка, написанная Искусственным Интеллектом volvo 04.12.2025 Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .	От async/await к виртуальным потокам в Python IndentationError 23.11.2025 Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .	Поиск "дружественных имён" СОМ портов Argus19 22.11.2025 Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .	Сколько Государство потратило денег на меня, обеспечивая инсулином. Programma_Boinc 20.11.2025 Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha Etyuhibosecyu 20.11.2025 Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .	Мысли в слух kumehtar 18.11.2025 Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .	Создание Single Page Application на фреймах krapotkin 16.11.2025 Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .	Фото: Daniel Greenwood kumehtar 13.11.2025	Расскажи мне о Мире, бродяга kumehtar 12.11.2025 — Расскажи мне о Мире, бродяга, Ты же видел моря и метели. Как сменялись короны и стяги, Как эпохи стрелою летели. - Этот мир — это крылья и горы, Снег и пламя, любовь и тревоги, И бескрайние. . .

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

Опции темы