Форум программистов, компьютерный форум CyberForum.ru

Паттерн стратегия - C++

Восстановить пароль Регистрация
 
el_gato_de_Ch
35 / 35 / 1
Регистрация: 28.04.2013
Сообщений: 110
07.02.2014, 22:43     Паттерн стратегия #1
Всем привет!

Начал изучать паттерны. Одним из наиболее понятных и интересных мне оказался паттерн Стратегия. Я начитался статей и примеров из различных источников. И как говорится запилил собственный пример на С++ и Java, с преферансом и куртизанками. =) Дабы устаканить у себя в голове полученные знания, я выставляю свой пример на обозрение опытных проггеров, чтобы вы посмотрели и указали на ошибки если таковые имеются.

main.cpp
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
#include <iostream>
#include <cstdlib>
 
#include "header/strategy.h"
#include "header/concrete_output1.h"
#include "header/concrete_output2.h"
#include "header/printer.h"
 
using namespace std;
 
int main()
{
    Printer some_printer;
    ConcreteOutput1 out1;
    ConcreteOutput2 out2;
 
    some_printer.set_output_method(&out1);
    some_printer.print();
 
    some_printer.set_output_method(&out2);
    some_printer.print();
 
    system("pause");
    return 0;
}
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
// strategy.h
 
#ifndef _STRATEGY_H_
#define _STRATEGY_H_
 
class StrategyPrint
{
public:
    StrategyPrint();
    ~StrategyPrint();
 
    virtual void output() = 0;
};
 
#endif
 
// strategy.cpp
 
#include "header/strategy.h"
 
StrategyPrint::StrategyPrint()
{
 
};
 
StrategyPrint::~StrategyPrint()
{
 
};
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
// concrete_output1.h
 
#ifndef _CONCRETE_OUTPUT1_H_
#define _CONCRETE_OUTPUT1_H_
 
#include <iostream>
#include "header/strategy.h"
 
class ConcreteOutput1 : public StrategyPrint
{
public:
    ConcreteOutput1();
    ~ConcreteOutput1();
 
    void output();
};
 
#endif
 
// concrete_output1.cpp
 
#include "header/concrete_output1.h"
 
ConcreteOutput1::ConcreteOutput1()
{
 
}
 
ConcreteOutput1::~ConcreteOutput1()
{
 
}
 
void ConcreteOutput1::output()
{
    std::cout << "You choose 1st strategy\n";
}
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
// concrete_output2.h
 
#ifndef _CONCRETE_OUTPUT2_H_
#define _CONCRETE_OUTPUT2_H_
 
#include <iostream>
#include "header/strategy.h"
 
class ConcreteOutput2 : public StrategyPrint
{
public:
    ConcreteOutput2();
    ~ConcreteOutput2();
 
    void output();
};
 
#endif
 
// concrete_output2.cpp
#include "header/concrete_output2.h"
 
ConcreteOutput2::ConcreteOutput2()
{
 
}
 
ConcreteOutput2::~ConcreteOutput2()
{
 
}
 
void ConcreteOutput2::output()
{
    std::cout << "You choose 2nd strategy\n";
}
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
// printer.h
 
#ifndef _PRINTER_H_
#define _PRINTER_H_
 
#include <cstdlib>
#include "header/strategy.h"
 
class Printer
{
public:
    Printer();
    ~Printer();
 
protected:
    StrategyPrint* behavior;
public:
    void set_output_method(StrategyPrint* strat);
    void print();
};
 
#endif
 
// printer.cpp
#include "header/printer.h"
 
Printer::Printer()
{
    behavior = NULL;
}
 
Printer::~Printer()
{
    delete behavior;
}
 
void Printer::set_output_method(StrategyPrint* strat)
{
    behavior = strat;
}
 
void Printer::print()
{
    behavior->output();
}
Надеюсь на вашу конструктивную критику!

ЗЫ. На почти безызвестном ресурсе педивикия, в примере на c++ существуют 2 класса Context и Client, я позволил себе небольшое отступление и реализовал сразу Client в виде Printer. Это же не сильно критично?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
07.02.2014, 22:43     Паттерн стратегия
Посмотрите здесь:

C++ Стратегия выборки
паттерн и строка C++ C++
C++ Паттерн Builder
Паттерн стратегия и ошибки при компиляции C++
Пошаговая стратегия(начало) C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
DU
1477 / 1053 / 45
Регистрация: 05.12.2011
Сообщений: 2,279
07.02.2014, 22:47     Паттерн стратегия #2
пример как пример. синтетический. прикапываться нет смысла, т.к. это не какое-то боевое решение.
придраться можно к другому:
вы стратегию по указатели передаете и нигде не проверяете на то, что указатель может быть
нулем. в таком случае желательно стратегии принимать по ссылке. чтобы у вызывающей стороны
не было вопросов относительно того, а работает ли принтер с нулевыми указателями.
ну или же обрабатывать как-то ситуацию, когда передали нуль указатель на стратегию.
Avazart
 Аватар для Avazart
6904 / 5144 / 253
Регистрация: 10.12.2010
Сообщений: 22,621
Записей в блоге: 17
07.02.2014, 23:09     Паттерн стратегия #3
Мм почем не сразу

C++
1
some_printer.print(&out1);
Пример неправильного использования, при таком интерфейсе как у тебя.
C++
1
2
3
4
Printer some_printer;
ConcreteOutput1 out1;
 
some_printer.print(); // упс !!! забыли установить стартегию!!
Опять же проверка указателя, и если надо нужно предусмотреть стратегию по умолчанию.
Kuzia domovenok
 Аватар для Kuzia domovenok
1883 / 1738 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
07.02.2014, 23:53     Паттерн стратегия #4
А зачем все советуют проверять указатель на NULL, если он не устанавливается в нуль по умолчанию? С таким же успехом можно проверять указатель на четыре миллиарда других невалидных значений указателя!

Добавлено через 4 минуты
Я вижу другие проблемы! Деструктор StrategyPrint не виртуальный. Почему?
Avazart
 Аватар для Avazart
6904 / 5144 / 253
Регистрация: 10.12.2010
Сообщений: 22,621
Записей в блоге: 17
07.02.2014, 23:57     Паттерн стратегия #5
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
А зачем все советуют проверять указатель на NULL, если он не устанавливается в нуль по умолчанию?
C++
1
2
3
4
Printer::Printer()
{
    behavior = NULL;
}
Кстати сразу не заметил фигню в деструкторе:

C++
1
2
3
4
Printer::~Printer()
{
    delete behavior;  /*  ??? */
}
Kuzia domovenok
 Аватар для Kuzia domovenok
1883 / 1738 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
08.02.2014, 00:04     Паттерн стратегия #6
Далее главное! Деструктор принтера зачем-то освобождает память из-под указателя на стратегию! Это ахтунг! Динамическая память тут вообще не выделяется нигде! И даж если б выделялась это не забота принтера её удалять! Он даже не знает как именно она выделена!

Добавлено через 2 минуты
Avazart, и чё? К нулю то как раз можно делитЪ применить, а о том, что в деструкторе косяк, я написал до тебя выше!
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11841 / 6820 / 771
Регистрация: 27.09.2012
Сообщений: 16,911
Записей в блоге: 2
Завершенные тесты: 1
08.02.2014, 08:41     Паттерн стратегия #7
Цитата Сообщение от Kuzia domovenok Посмотреть сообщение
К нулю то как раз можно делитЪ применить
C++
1
2
3
4
void Printer::print()
{
    behavior->output();//Тут с нулем тоже прокатит?
}
Kuzia domovenok
 Аватар для Kuzia domovenok
1883 / 1738 / 116
Регистрация: 25.03.2012
Сообщений: 5,907
Записей в блоге: 1
08.02.2014, 10:32     Паттерн стратегия #8
Croessmah, a тут разве делитъ? Ты посмотри на его деструктор!
el_gato_de_Ch
35 / 35 / 1
Регистрация: 28.04.2013
Сообщений: 110
08.02.2014, 10:51  [ТС]     Паттерн стратегия #9
По поводу delete в деструкторе, это меня всё java попутала со своими some_func(new SomeObject())
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
08.02.2014, 12:56     Паттерн стратегия
Еще ссылки по теме:

Паттерн Singleton C++
C++ Паттерн Embedded Value
Стратегия выделения/освобождения памяти C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Avazart
 Аватар для Avazart
6904 / 5144 / 253
Регистрация: 10.12.2010
Сообщений: 22,621
Записей в блоге: 17
08.02.2014, 12:56     Паттерн стратегия #10
Цитата Сообщение от el_gato_de_Ch Посмотреть сообщение
По поводу delete в деструкторе, это меня всё java попутала со своими some_func(new SomeObject())
Никто не мешает делать как в яве, только в таком случае лучше уже сразу предавать умный указатель.
Yandex
Объявления
08.02.2014, 12:56     Паттерн стратегия
Ответ Создать тему
Опции темы

Текущее время: 03:18. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru