Нужен совет с реализацией логики программы

Здравствуйте уважаемые пользователи! Мне нужен ваш совет, но сначала прошу вас прочитать про работу моей программы, чтобы понять суть моей проблемы и после этого возможно вы что-нибудь мне посоветуете...

Я создал приложение. Суть программы - работа с таблицей (изменение, редактирование ячеек)
Приложение связывается с базой данных, но через GET запросы. Основную часть работы с базой данных выполняет PHP. Программа функционирует отлично, проблем в коде нет. В программе есть различные переходы в окна.

Я примерно нарисовал как сейчас работает программа

Форма 1 - авторизация. Работает один раз и передает данные в форму Main(имя и какие-то данные)

Формы 3,4,5 создают в базе имена, фамилии, и какие-то данные

Форма Main - основное окно. В нём есть главная таблица над которой управляем. Она вызывает формы 3,4,5 для создания

Форма monitoring - дополнительное окно. В окне содержится таблица которая показывает отчёт о конкретной группе пользователей

Форма main,3,4,5,monitoring связаны с engine

Класс engine имеет тип partial. Я разделил класс на несколько частей для удобства. Engine передает GET запросы на сервер и выводит данные в таблицу сам. То есть main посылает команду в engine, engine выгружает данные и сам всё меняет.

Сам класс с модификатором static. Методы и все поля с модификатором static. При старте программы, некоторые компоненты(таблица, листы, кнопки, текста) передаются в engine.
Управлять таблицей из формы main может любой класс и форма. Метод из класса может вызвать любой класс и любая форма

Как думаете, правильно ли реализована логика программы или можно было сделать намного проще? У меня была идея переписать код программы под какой-нибудь паттерн для удобства, и не знаю, стоит ли. Или я просто слишком придирчив к коду и стремлюсь к идеальному коду которого не существет?

Миниатюры

 

0

Сообщение от Bilyalov правильно ли реализована логика программы или можно было сделать намного проще

"Правильно" и "проще" это не одно и тоже.

Сообщение от Bilyalov некоторые компоненты(таблица, листы, кнопки, текста) передаются в engine.

Это напрягает.

Сообщение от Bilyalov Engine

Не очень понятно что это за класс, и каковы его функции. Привидите код методов этого класса. Как он связан с формами? Формы вызывают его методы, или он вызывает их методы, или у него есть события, или он обрабатывает события.

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

Если работает и нет проблем - не стоит.

Сообщение от Bilyalov какой-нибудь паттерн

Можно было бы конечно посоветовать какой нибудь паттерн типа MVC, MVP и т.д. Но без понимания арихитектуры системы в целом, ее функций, выбрать лучший паттерн невозможно.

0

Bilyalov, Обычно если класс Partial, то в него слишком много напихано всего и его лучше разделить на отдельные классы. А так совет правильный работает - не трогай. Если нужно будет внести изменения, и вы почувствуете, что где-то нужно костыльными методами это делать - значит пришла пора менять. Если изменения происходят легко и без последствий - то нет причин беспокоиться.

0

Storm23, engine посылает гет запросы на сервер и парсит данные, затем эти данные сам вбивает в таблицу . В общем все что нужно вытащить из бд, выполняет класс engine

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

Сообщение от Storm23 Не очень понятно что это за класс, и каковы его функции. Привидите код методов этого класса. Как он связан с формами? Формы вызывают его методы, или он вызывает их методы, или у него есть события, или он обрабатывает события.

Формы вызывают методы класса Engine. Например мне нужно ввести что-то в ячейке #1. Форма вызовет метод из класса engine, передав ему ячейку. Engine сам все реализует(изменит ячейку, отправит данные на сервер, обновит ячейку, изменит цвет если нужно)

0

Bilyalov, Короче не знаю как паттерн, а вот антипаттерн God object у вас получился Стоит только на схему внимательнее посмотреть

0

Сообщение от Psilon Bilyalov, Короче не знаю как паттерн, а вот антипаттерн God object у вас получился Стоит только на схему внимательнее посмотреть

Вы правы

Добавлено через 12 минут
Собсно, вопрос. Стоит ли так всё оставлять? Или нужно переписать всё под паттерн?

0

Сообщение от Bilyalov Стоит ли так всё оставлять? Или нужно переписать всё под паттерн?

лучшее - враг хорошего. если работает, и качество работы устраивает - не трогай.

хотя для самообучения можно и переписать

0

Bilyalov,

Сообщение от Psilon Обычно если класс Partial, то в него слишком много напихано всего и его лучше разделить на отдельные классы.

0