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

[Lemming]

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

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

[Bober]

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

[Андре]

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

Changelog:

• fixed: на одной форме центробанк сокращен как "ЦБР", а на другой как "ЦБ" - fixed
• fixed: заголовок к форме настройки: по-русски правильно "соответствие кодов.. кодам", а не "соответствие кодов.. с кодами.."
• fixed: загрузка курсов просто закрывает форму (что правильно), а загрузка валют ругается "Ошибка импорта" (что неверно).
• fixed: У формы неплохо бы сделать column width и column height
• fixed: Либо к проекту приложить файл для импорта, скачанный с сайта ЦБ
• fixed: .. а еще и уникальными. сейчас можно для нескольких кодов валют ЦБ установить одинаковый код валюты Аксапты (с помощью validateField())
• fixed: при импорте курсов - можно указать неправильное (несуществующее) имя файла, нажать ОК - вывожу кол-во обработанных курсов.
• fixed: то есть два соседних метода...
• fixed: начальный импорт кодов, теоретически, должен быть отдельным RunBase классом
• fixed: есть поля dialogDateFrom, dialogDateTo

Цитата:

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

Нет, какие то валюты могут не импортироваться - поэтому CBRCurrencyCode может быть пустым.
CurrencyCode пустое для рублей, так как через него пересчитываем валюты. Это не очень прозрачно, но другие варианты мне пока нравятся еще меньше.

Цитата:

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

В очередном kernel rollup оно станет true, так что нафиг.