Есть приложение:
клиент 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:

Petro123сначала разбирают Основной прецендент. Это у теюя 80% автоматом закачка файлов без участия клиентов .
Так?
Не так, потому задача решена, когда покрыто 100% случаев.
Отсутствие 20% ручных загрузок делает бессмысленным автоматические 80%

И при чем тут анекдот про часы?
Более удачный пример: Нога сломана в бедре и в ступне. В бедре сложнее перелом. Задача: Давайте вылечим человека.
Ты: давайте вылечим только бедро!
Но ходить, то он все равно не сможет, потому что ступня не работает!

BlazkowiczfileName не нужен. size не нужен. DTO тогда для чего?
В моем конкретном случае не нужно. В общем - чтобы тупо иметь какие-то метаданные....
Может я захочу сохранить обработанный файл в какое-то хранилище, например.

Petro123Один загрузка из постоянного места с XML парсером и таймером.
Второй чтобы кинуть в то место файл.
Если не походит просто кинуть на ФТП хранилище в проводнике файл, то обзови свою тему отдельно.
Причём тут вообще XML и какой то код?
Вооот! Вот я и пытаюсь выяснить, почему нельзя файл тупо передать как параметр в сессион бин.
Почему обязательно нужно его куда-то сохранить в доступное место? (у меня, кстати, такого нет, но это уже другой вопрос)
Petro123Если не походит просто кинуть на ФТП хранилище в проводнике файл, то обзови свою тему отдельно.
Причём тут вообще XML и какой то код?
Чтобы был понятен юз кейс

В целом вообще тему можно закрывать. Ход мысли я понял...

Petro123Vladmir Kклиент Swing
в swing нет компонента для загрузки файла на сервер? В БД или просто в папку на сервер?
Я такой не знаю...
К тому же, доступ к файловой системе сервера приложений - не безопасно. имхо

Petro123Vladmir K,
т.е. при такой поставновке (2 модуля) никакого дублирования кода не происходит.
Об этом вроде топик был?
Топик был "почему плохо передавать файл как параметр". Дублирование кода - причина вопроса, причину нужно устранить. Остальное я понимаю и соглашаюсь и с тобой и Blazkowicz.

Petro123Т.е. ФТП сервер в вебе был на нашем обслуживании.
Дашь и мне местечко с поддержкой 24*7? ж)
Petro123Удачи!
Еще раз спасибо :)