Архитектура программы. Правильно ли сделал?

Собствено с архитектурой вроде определился. Хотелось бы конструктивной критики.

Задача по сути следующая:
- Считывать данные геометрических объектов из файла.
- Вычислить некоторое число для двух геометрических объектов.

Базовый класc для геометрических объектов имеет одну функцию которая возвращает число, плюс конструкторы деструкторы и т.д.

Это как бы "ядро" никаких проверок корректности данных при создании объекта не произвожу. Важна производительность. Максимально общий, чтобы можно было добавить любые возможные подклассы.

Все проверки в "интерфейсном" классе, он заточен под данную задачу:
(правильно ли называть его интерфейсом?)

Reader - статический объект, используется функцией ReadFromFile, считывает данные из файла и проверяет их корректность, создает объекты сGeomObj.

Calculator - статический объект, используется функцией CalcGoalValue, вычисляет интересующее значение для двух объектов cGeomObj.


Еще вопрос. Если я свой клас сGeomObj сделаю в виде шаблона, я могу задать список допустимых типов для параметра типа?
Я не нашел, но это по моему странно. Если я не хочу чтобы в мой шаблон пихали не понять что, то что же делать?

Еще. Сейчас в cGeomObj используется #define константа, ее лучше сделать параметром шаблона?

0

Сообщение от loou Важна производительность

я бы тогда создал отдельно модуль который бы по полной сразу проверял загружаемый или создаваемый ресурс, а в программе для работы с этим ресурсом полностью отключил бы проверки, в угоду производительности

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

Сообщение от loou Все проверки в "интерфейсном" классе, он заточен под данную задачу: (правильно ли называть его интерфейсом?)

Интерфейс Interface Общий метод для структурирования компьютерных программ для того, чтобы их было проще понять
http://ru.wikipedia.org/wiki/%... 0%B8%D1%8F

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

Сообщение от loou Еще. Сейчас в cGeomObj используется #define константа, ее лучше сделать параметром шаблона?

а хотите ли вы чтобы её видели участки кода которые будут использовать ваш класс/шаблон

1

Сообщение от loou Это как бы "ядро" никаких проверок корректности данных при создании объекта не произвожу. Важна производительность. Максимально общий, чтобы можно было добавить любые возможные подклассы.

Нужно сделать его абстрактным. virtual double GetVal()=0;

Сообщение от loou Сейчас в cGeomObj используется #define константа, ее лучше сделать параметром шаблона?

Делайте её чем хотете, но только не #define... макросы в С++ лучше не использовать

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

Сообщение от loou Еще вопрос. Если я свой клас сGeomObj сделаю в виде шаблона, я могу задать список допустимых типов для параметра типа? Я не нашел, но это по моему странно. Если я не хочу чтобы в мой шаблон пихали не понять что, то что же делать?

Нету смысла в шаблоне, в который можно пихать только определенные типы. Подумайте, может вам вовсе не шаблон нужен?

1

Сообщение от Van111 я бы тогда создал отдельно модуль который бы по полной сразу проверял загружаемый или создаваемый ресурс, а в программе для работы с этим ресурсом полностью отключил бы проверки, в угоду производительности

То есть, как то так?
Спасибо!

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

Нужно сделать его абстрактным. virtual double GetVal()=0;

Ага, сделана, просто в пост virtual не попало

Делайте её чем хотете, но только не #define... макросы в С++ лучше не использовать

Спасибо!

Нету смысла в шаблоне, в который можно пихать только определенные типы. Подумайте, может вам вовсе не шаблон нужен?

ну вот double, int и т.д. хочу, а вот class : cXZ в качестве параметра шаблона не хочу. У меня там корни, степени и т.д. что они будут с объектами делать?

0

Сообщение от loou ну вот double, int и т.д. хочу, а вот class : cXZ в качестве параметра шаблона не хочу. У меня там корни, степени и т.д. что они будут с объектами делать?

А ничего не будет делать. Не скомпилируется. Но ограничить шаблон только числовыми типами можно

0

Но ограничить шаблон только числовыми типами можно

Скажите как, если не трудно. Ну либо ключевое слово для поиска. А то пока найти не могу.

0

В большей степени критики, пожалуй, заслуживает не реализация, а постановка задачи.

Сообщение от loou - Считывать данные геометрических объектов из файла. - Вычислить некоторое число для двух геометрических объектов.

Вопрос звучит как "мне нужно считать данные и произвести вычисления, давайте обсудим архитектуру". Т.е. информации практически ноль, и при этом какие-то разговоры об архитектуре.

Последняя фраза мне вообще не ясна. "Вычислить для двух объектов" - это выбрать из всего массива два объекта и для каждого из них посчитать по отдельности, либо речь о функции с двумя аргументами f(x, y)?

Сообщение от loou C++ 1 2 3 4 5 6 7 class сGeomObj { public: * * ... * * double GetVal(); protected: * * ... };

Само название GetVal - ни о чем не говорит. Т.е. вопрос такой: а это должно быть методом класса? Или может будет лучше сделать внешней функцией? Забегая вперед, скорее всего метод объявлен в классе справедливо, но задумываться о таких вещах стоит.

Сообщение от loou C++ 1 2 3 4 5 6 7 8 9 10 11 class iGeomObjFromFile { public: * * ... * * int * * *ReadFromFile(char* f_name); * * double CalcGoalValue(int i, int j); protected: * * list < cGeomObj > gList; * * static cReaderFromFile * * * *Reader; * * static cGoalValueCalculator *Calculator; * * ... }

Сразу же вопрос: а нужен ли этот класс? Он выглядит как свалка всего, что просто не знал где разместить.
Первая же архитектурная проблема: класс предполагает определенный порядок вызова методов. Т.е. вызывать метод вычисления имеет смысл только после того, как выполнено чтение из файла.
Вообще чтение из файла я бы сделал отдельной функцией:
Не вижу повода использовать std::list. Вектор вполне подойдет.

Далее - почему reader статический? Он один общий на все файлы? Каковы его функции? Он stateless? А если нет, если в нем реализована какая-нибудь своя буферизация и при начале обработки следующего файла в нем остается некоторое состояние от предыдущего чтения?
Короче, на один файл - один reader.
Опять же, при такой постановке не вижу необходимости в cGoalValueCalculator как классе. Предполагаю, что вместо него вполне могут быть обычные функции.

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

Сообщение от loou Скажите как, если не трудно. Ну либо ключевое слово для поиска.

Type traits http://en.cppreference.com/w/cpp/types

2

Спасибо, за комментарии. На счет постановки - я привел в общих чертах архитектуру, вы ее по критиковали. Был бы очень рад если бы вы продолжили в том же духе.

Последняя фраза мне вообще не ясна. "Вычислить для двух объектов" - это выбрать из всего массива два объекта и для каждого из них посчитать по отдельности, либо речь о функции с двумя аргументами f(x, y)?

Да функция двух переменных, иначе класс iGeomObjFromFile точно не нужен.

Само название GetVal - ни о чем не говорит. Т.е. вопрос такой: а это должно быть методом класса? Или может будет лучше сделать внешней функцией? Забегая вперед, скорее всего метод объявлен в классе справедливо, но задумываться о таких вещах стоит.

Класс имеет множественное наследование, единственное что объединяет все производные классы - эта функция. Тут, по моему, все очевидно.

Про cGeomObjFromFile:

Сразу же вопрос: а нужен ли этот класс? Он выглядит как свалка всего, что просто не знал где разместить.

Ну, примерно так и есть. Я хотел в нем производить выбор производного от cGeomObj класса и соответствующего конструктора в зависимости от считанных из файла данных.

C++ 1 std::vector<cGeomObj> ReadFromFile(const std::string& filename); Не вижу повода использовать std::list. Вектор вполне подойдет.

Спасибо! Наверно так и сделаю. Тогда выбор конструктора тоже на нее ложится.

0