Есть приложение:
клиент Swing, Сервер - Ejb

Допустим есть функционал: загрузить курсы валют из xml файла.
Сервер тянет по расписанию.
Пользователь тоже может загрузить файл, выбрав его со своего локального диска.

Сейчас в приложении логика по обработке файлов дублируется:
- Клиент разгребает файл локально и обращается к серверу только сохранить уже сформированные сущности.
- У сервера, собственно, свой фасадный метод, который не используется клиентом.

Мне такое дублирование кода не нравится. Я бы передал с клиента файл (например даже через byte[]), делегировав всю логику серверу.
Передавать файл на сервер не нравится архитектору.

Проясните меня, пжста, чем ему не нравится?

BlazkowiczVladmir Kклиент Swing, Сервер - Ejb

Протокол RMI?
Да

BlazkowiczVladmir KМне такое дублирование кода не нравится.
Ну, не дублируйте код, кто вас заставляет?
Сделать общий класс, использовать его и на сервере и на клиенте?
Меня смущает - что логика на сервере в сешионбине, в котором инжектится правильно логер, т.д. Как это все богатство затянуть на клиент?

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

BlazkowiczНу, тут есть разумное зерно. Зачем делать на сервере то что можно сделать на клиенте. XML парсинг не самая быстрая штука.
Вот я всегда наивно полагал обратное, что клиент - чем тупее, тем лучше....
Сервер - по определению мощнее, зачем на клиенте делать "небыстрые" штуки?

BlazkowiczVladmir KСделать общий класс, использовать его и на сервере и на клиенте?

Не обязательно класс. Мавеном что ли не пользуетесь? Общий код отдельным модулем.
Класс - модуль. Суть та же. Ясно.

BlazkowiczЗачем вам логировать детали разбора XML.
Да я и пытаюсь объяснить, что помимо простого разбора xml, есть еще подготовительные этапы:
считать доп параметры, принять решение, что делать в зависимости от параметров, т.д.

BlazkowiczНу, что-то всё равно останется на сервере. А то что от сервера не зависит, можно пользовать на клиенте. Промежуточный результат передавать. Что не так-то?
Видимо, так и придется рефакторить....
Я дублирование логики так и нашел - открылся баг. Начал копать. В двух разных классах (один на клиенте, второй на сервере) 90% кода совпадает. Даже методы такие же (по семантике). Общего модуля нет...
BlazkowiczСервер труднее масштабировать.
Сервер на горячую можно передеплоить.
А если расширяемость тугая, тут уж архитектурой попахивает, имхо

BlazkowiczНагрузки на клиента минимальные.
Ага, лишь бы не забыться... и не переборщить.
BlazkowiczВ 90х у нас были полностью тупые клиенты-браузеры. И где они сейчас?
Хорошее было время! :)
Графику оптимизировали до боли, комментарии удаляли, чтоб облегчить файлы!
И да, они были тупыми! По мне - так это хорошо :)


BlazkowiczСплошной DHTML
В такие моменты я с ужасом начинаю думать, что если переписывать такой толстый клиент (как у нас со Свинга на веб), то ... мама не горюй!

Спасибо за разъяснения.

Petro123не всегда.
Вот я за расшифровкой и обратился на форум.

Petro123Поэтому у тебя и есть архитектор.
Благодарю :)

Petro123Плюс вариации про 3-х звенку и т.д.
Ну вот я и пытаюсь собрать инфу о камнях от умных людей при трехзвенке с БЛ на сервере при передаче сериализированного файла от клиента серверу.

Пока собрал
- масштабируемость сервера и нагрузку. Blazkowicz, как обычно, +1

Leonid KudryavtsevЕсли в процессе загрузки могут происходить какие-то события, требующие диалога с пользователем (GUI), то перенести на сервер "все не так очевидно" ( C ) дочь офицера
В данном случае, пользователь выбирает только файл. Остальное делается по алгоритму, без доп уточнений пользователя.

Petro123- при десктопе минимальные классы на сервере и все классы на клиенте.
Фасады больше не в моде?

Petro123Фасад обычно прикрывает какю то кухню внутри или скрывает её недостатки.
- Считать доп настройки для конкретного подразделения: смещать ли дату (например на завтра) курса из файла, локальная валюта подразделения (RUB, GBP, etc)
- Распарсить xml
- собрать модель валюты.

Сойдет за кухню?


Petro123Модель данныйх какая для курса валют?
Табличка Дата и Курс?
примерно да:
дата - валюта С - валюта В


Petro123Тогда на сервере только INSERT, а всё остальное на клиенте.
ИМХО, зачем такой клиент? Переход на другой клиент = новый проект!

Petro123Т.к. парсер XML к Модели данных на сервере не имеет отношения
тут я согласен.
Вопрос только: как организовать автоматическую загрузку по расписанию? Если логика и парсинг на клиенте, а запуск задачи на сервере...

Petro123тут надо определятся что в приоритете.
80% автоматом
20% пользователь
Petro123Если вдруг с клиента надо подтолкнуть, то просто данный шедулер выставляется на время минутой позже чем сейчас.
В динамике выставлять расписание шедуллера, при этом такие привилегии должны быть у юзера? Сомнительно.

А моя фраза
Vladmir KПользователь тоже может загрузить файл, выбрав его со своего локального диска.
вообще перечеркивает такую концепцию! :( Потому что не подтолкнуть, а совсем другой файл загрузить...

В итоге, подожду аргументы архитектора, и скорее всего, будет некий общий модуль, как и советовал Blazkowicz, ну что и очевидно...
Хотя мое мнение, что фасад тут совсем не лишний, а серьезных технических камней против сериализации файла в том или ином виде, я не услышал.....

Petro123дал поиск по слову сериализация по топику. Нет ни слова.
Petro123ИМЯ_ПАКЕТА.ЗагрузитьВсёВ_общейПапке;
Ты правда боишься вызвать эту хранимку с клиента?
Нет, хранимку, конечно, не боюсь

Но вот если какой-то внешний шедуллер, например в данном проекте, quartz, то доступ к конфигурационнику не хочется давать клиенту. Ибо это привилегия одминов.
Petro123я лично не вижу у одной строки выше фасада)).
Удачи!
спасибо :)

Petro123дал поиск по слову сериализация по топику. Нет ни слова.

Leonid KudryavtsevИ как файл можно сериализовать? Это вообще как?

Я так рассуждаю:
передать файл, как File, от клиента к серверу через RMI не получится, потому что он проксироватьсян не будет. Ну если конечно путь будет не одинаковый с сервера и клиента, а это утопия.
Значит надо как-то сериализировать...

Варианта, на мой взгляд, два приходят на ум:
1. В частном случае, файл текстовый - можно передать в виде строки
2. Можно создать некий DTO: