Еще один импорт курсов валют

[Андре]

GMCS представляет бесплатный сервис для импорта курсов валют в Microsoft DAX для тех, у кого ограничен доступ в интернет.

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

Установка

Проект не перекрывает стандартные объекты системы, что позволяет безболезненно импортировать его практически на любое приложение. После импорта необходимо добавить два пункта меню (MunuItem), входящие в состав проекта, в меню системы. Проект тестировался на Axapta 4 SP2.

Настройка

Настройка решения заключается в установке соответствия между кодами валют с сайта Центробанка и кодами валют в системе:



Перечень валют, курсы которых предоставляет сайт Центробанка можно скачать по ссылке и импортировать в систему, нажав клавишу "Взять из файла". При сохранении файла убедитесь, что он сохранился в формате XML.

Использование

1. Создаем файл, содержащий курсы валют за выбранный диапазон дат и загружаем его.
2. Открываем форму "Импорт котировок ЦБ":



3. Выбираем загруженный файл.

В случае указания определенного кода валюты будет импортирован только ее курс. Если же поле ввода оставили пустым будут обновлены курсы всех валют для которых настроено соответствие их кодов (см. пункт Настройка).

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

Все вопросы и замечания относительно проекта можно оставлять мне в этой ветке.

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

[Андре]

Проект от 22 декабря (17:00).

[axbegin]

В GMCS стало все настолько плохо, что вы готовы раздавать свои доработки бесплатно ?

[Андре]

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

[Zabr]

Разрешите побрюзжать :
1 - на одной форме центробанк сокращен как "ЦБР", а на другой как "ЦБ"
2 - заголовок к форме настройки: по-русски правильно "соответствие кодов.. кодам", а не "соответствие кодов.. с кодами.."

updated: формы выбора файлов для загрузки списка валют и для загрузки курсов по-разному обрабатывают кнопку "Отмена": загрузка курсов просто закрывает форму (что правильно), а загрузка валют ругается "Ошибка импорта" (что неверно). Такое ощущение, что список валют и их загрузку писал один программист, а загрузку курсов - другой.

[Андре]

Судя по заданным мне вопросам из описания не очень понятно - зачем, собственно, нужен этот сервис.

Дело в том, что у нас есть ряд клиентов, у которых интернет, в общем то, есть, но не в той сетке, где установлена Axapta. DMZ, например.

Этим клиентам подошел бы вариант со скачиванием файла с данными и последующим импортом данных в систему, но мы не нашли сервиса центробанка, который бы выдавал курсы всех валют за запрошенный диапазон дат. Либо все валюты на дату, либо одна валюта на диапазон дат.

Если у клиента в системе 20 валют, то скачивать и импортировать 20 файлов будет не очень удобно, поэтому наш сервис запрашивает 20 XML-файлов у центробанка, собирает из них один XML-файл и отдает его посетителям для последующего импорта в систему.

[Андре]

Цитата:

Разрешите побрюзжать

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

[Bishop]

Цитата:

Сообщение от Андре

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

Честно говоря, на вашем месте (я имею ввиду GMCS) я бы уделил больше времени подготовке подобного проекта.
Содержание кода не выдерживает критики ни в плане единого стиля в рамках проекта, ни в плане стиля стандартной аксапты.
По факту получается, что, помимо бета-тестирования, вы невольно организовали также "бета-кодревьюинг", так как "ляпов" немало (отсутствие меток, форма не ресайзится, грамматические ошибки, префиксы новых объектов в разном регистре - GM_ и gm_ и т.д.)

[Zabr]

Цитата:

Сообщение от Bishop

префиксы новых объектов в разном регистре - GM_ и gm_ и т.д.)

- вот, еще одно доказательство что несколько разных программистов работало над проектом. Хотя, казалось бы, небольшая ведь разработка...

[oip]

Еще совсем мелкие замечание. У формы неплохо бы сделать column width и column height, у таблицы GM_AxCurCode2CBRCurCode поля CurrencyCode и CBRCurrencyCode по логике должны быть мандатори. В проекте в трех местах используются метки @GMC*, однако меточного файла не приложено, да и в других местах метки не используются. Надо бы однообразие. Или метки и меточный файл или везде текст.

Форма сейчас работает по совсем неочевидной для пользователя логике. Создаем записи, задав только CurremcyCode, но при следующем открытии формы она откроется пустой из-за ренджа в ините, но создавать новые записи уже не получится, так как в таблице они есть. Надо бы как-то иначе.

Upd. Ой, не так с мандатори . CBRCurrencyCode == "" - это RUR? Тоже не очень прозрачно и хорошо.

[Zabr]

Цитата:

Сообщение от oip

В проекте в трех местах используются метки @GMC*

Не нашел. Утром проект скачивал. Может поменяли уже ?

[Zabr]

Цитата:

Сообщение от oip

Надо бы как-то иначе.

Вряд ли у ЦБ поменяются коды валют, так что можно было бы облегчить жизнь юзерам: при первом запуске формы сразу заполнять справочник (зашить список в код). Либо к проекту приложить файл для импорта, скачанный с сайта ЦБ.

[Zabr]

Цитата:

Сообщение от oip

у таблицы GM_AxCurCode2CBRCurCode поля CurrencyCode и CBRCurrencyCode по логике должны быть мандатори

.. а еще и уникальными. сейчас можно для нескольких кодов валют ЦБ установить одинаковый код валюты Аксапты.

А еще: и при импорте списка валют, и при импорте курсов - можно указать неправильное (несуществующее) имя файла, нажать ОК и получить вместо сообщения об ошибке сообщение "импорт завершен". Как будто все в порядке. Какая тут есть опасность: например, кто-то регулярно выгружает файл с курсами, а задание на загрузку в Аксапте стоит в пакете. Если изменится местоположение каталога для выгрузки или файл назовут неправильно, а пакет продолжает работать, то никто не узнает, что файл перестал загружаться.

[oip]

Цитата:

Сообщение от Zabr

Не нашел. Утром проект скачивал. Может поменяли уже ?

EDT GM_CBRCurrencyCode, таблица GM_AXCurCode2ExtCurCode и ее группа Common.

[oip]

Если уж совсем по мелочам, чтоб причесать, то есть два соседних метода...

static GM_AxCurCode2ExtCurCode find(CurrencyCode _currencyCode)
{
    gm_AxCurCode2ExtCurCode gm_AxCurCode2ExtCurCode;
    ;
    select gm_AxCurCode2ExtCurCode
        where gm_AxCurCode2ExtCurCode.CurrencyCode == _currencyCode;

    return gm_AxCurCode2ExtCurCode;
}

и

static GM_CBRCurrencyCode FindCBRCurrencyCode(CurrencyCode _currencyCode)
{
    GM_AxCurCode2ExtCurCode gm_AxCurCode2ExtCurCode;

    ;

    select gm_AxCurCode2ExtCurCode where gm_AxCurCode2ExtCurCode.CurrencyCode == _currencyCode;

    return gm_AxCurCode2ExtCurCode.CBRCurrencyCode;
}

Ну, то что писали, возможно, разные люди (перенос строк в селекте), это одно. А другое - я бы второй метод вообще бы убрал, он лишний, использовав первый. А в первом бы добавил "if (currencyCode)" и _forupdate с firstonly в селекте. Все-таки у метода find есть стандартная структура.

[S.Kuskov]

Цитата:

Сообщение от oip

я бы второй метод вообще бы убрал, он лишний

Не, убирать может и не нужно, но переназвать и переписать используя find стоит

static GM_CBRCurrencyCode CBRCurrencyCode(CurrencyCode _currencyCode)
{
    return GM_AxCurCode2ExtCurCode::find(_currencyCode).CBRCurrencyCode;
}

P.S.: Ещё немного и к теме можно будет добывить тег: "законченный пример"

[oip]

Цитата:

Сообщение от S.Kuskov

Не, убирать может и не нужно

А зачем он? Вызывается один раз и возвращает просто поле самой таблицы.

Цитата:

Сообщение от S.Kuskov

P.S.: Ещё немного и к теме можно будет добывить тег: "законченный пример"

Ну, еще немного кодревью... Ну там подчеркивания в передаваемых параметрах в методах, импорт, вероятно, лучше делать в транзации, где-то там в паре мест при делении на переменную не проверяется, что у нее ненулевое значение, зачем-то объявляется переменная с начальным значение false, хотя оно и так false... Есть еще разные мелочи.

[S.Kuskov]

Цитата:

Сообщение от oip

А зачем он? Вызывается один раз и возвращает просто поле самой таблицы.

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

CBRCurrencyCode = GM_AxCurCode2ExtCurCode::CBRCurrencyCode(CurrencyCode)

Цитата:

Сообщение от oip

зачем-то объявляется переменная с начальным значение false, хотя оно и так false...

У такого подхода есть плюсы. Отсекается возможность случайно забытой инициализации и как следствие код лучше читается. Ну и потом пароноидальные мании... на компилятор надейся, а сам не плошай Я сам стараюсь не использовать не инициализарованные переменные.

[Lemming]

+5 копеек, по мелочи, ИМХО, вместо clicked на кнопке формы, начальный импорт кодов, теоретически, должен быть отдельным RunBase классом, а не вызывать "волшебный" статический метод fillCBRCurrencyCodeFromInternet, который к слову не "фром интернет", а "фром локал файл"

В объявлении класса gm_GetCurrencyFromFileCBR есть поля dialogDateFrom, dialogDateTo, которые нигде больше не используются, так же зачем-то там-же объявили саму переменную dialog.

[Андре]

Цитата:

Честно говоря, на вашем месте (я имею ввиду GMCS) я бы уделил больше времени подготовке подобного проекта.

Следующий раз обязательно так и сделаем.

Цитата:

По факту получается, что, помимо бета-тестирования, вы невольно организовали также "бета-кодревьюинг", так как "ляпов" немало (отсутствие меток, форма не ресайзится, грамматические ошибки, префиксы новых объектов в разном регистре - GM_ и gm_ и т.д.)

Не совсем так. В первую очередь нас интересовала (и интересует) новая модель распространения ПО, точнее некоторых его частей. Тестирование, code review, предложения по развитию, правка багов в данном случае побочный, хотя, не скрою, полезный эффект. Но этот эффект присущ всем open source приложениям, поэтому мне не очень понятно, что Вас удивило. Все советы и замечания к коду были сделаны добровольно. Если Вам интересна данная модель разработки - Вы можете выложить свои проекты на форум - данный раздел открыт для всех.

Цитата:

Сколько программистов будет уволено из GMCS по результатам разбора примера ?

Думаю, как инициатор я буду первым. А что, есть предложения ?

Цитата:

...еще одно доказательство что несколько разных программистов работало над проектом. Хотя, казалось бы, небольшая ведь разработка...

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

Согласен, что перед выкладыванием решения на форум я уделил недостаточно времени "чистке коду". На будущее постараюсь учесть.