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

[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.

[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, так что нафиг.