AXForum  
Вернуться   AXForum > Microsoft Dynamics AX > DAX: База знаний и проекты
All
Забыли пароль?
Зарегистрироваться Правила Справка Пользователи Сообщения за день Поиск Все разделы прочитаны

 
 
Опции темы Поиск в этой теме Опции просмотра
Старый 22.12.2009, 17:06   #21  
Андре is offline
Андре
Moderator
Сотрудники компании GMCS
 
2,375 / 464 (20) +++++++
Регистрация: 03.12.2001
Всем спасибо за замечания. Я выложил новый проект.

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, так что нафиг.
Старый 22.12.2009, 17:20   #22  
Андре is offline
Андре
Moderator
Сотрудники компании GMCS
 
2,375 / 464 (20) +++++++
Регистрация: 03.12.2001
Цитата:
Честно говоря, на вашем месте (я имею ввиду GMCS) я бы уделил больше времени подготовке подобного проекта.
Следующий раз обязательно так и сделаем.

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

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

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

Согласен, что перед выкладыванием решения на форум я уделил недостаточно времени "чистке коду". На будущее постараюсь учесть.
Старый 22.12.2009, 17:51   #23  
blokva is offline
blokva
Пенсионер
Аватар для blokva
SAP
NavAx Club
 
743 / 167 (7) ++++++
Регистрация: 04.06.2003
Адрес: Беларусь
Не понятна фраза мелким шрифтом внизу:
Цитата:
*Компания GMCS не несет ответственности за достоверность информации, содержащейся в файле.
Иными словами вот вам сервис, но за достоверность не отвечаем ... т.е. это просто побаловаться?
__________________
Законы природы еще никто не отменял!
А еще у меня растет 2 внучки!!! Кому интересно подробности тут:
http://www.baby-shine.com/
Старый 22.12.2009, 17:55   #24  
Андре is offline
Андре
Moderator
Сотрудники компании GMCS
 
2,375 / 464 (20) +++++++
Регистрация: 03.12.2001
Цитата:
Иными словами вот вам сервис, но за достоверность не отвечаем ... т.е. это просто побаловаться?
Защита от юридического преследования на случай, если форматы файлов неожиданно поменяются, наш сервер взломают или ЦБ выдаст некорректные курсы валют. Стандартная фраза, включаемая в любой договор.

Там же написано, что если вы обнаружите проблемы и сообщите нам о них, мы постараемся их устранить. Ряд наших клиентов используют этот сервис, поэтому мы заинтересованы в поддержании его работоспособности.
Старый 23.12.2009, 11:59   #25  
Rezervforall is offline
Rezervforall
Участник
 
142 / 26 (1) +++
Регистрация: 09.06.2009
а мне понравилось.
Главное - поставил - работает. Можно много писать про код и про остальное.

Я - не программист. Мне главное - скачал БЕСПЛАТНО - поставил - работает. Значит сделано хорошо. Скачал БЕСПЛАТНО - поставил - не работает. Значит сделано плохо.
Работал и первый вариант кстати.
Опять же раздел вроде называется "База знаний и проекты", а не "Куски кода и прочие непонятки".

А в разделе то кроме Tabax и этого проекта вроде готового и нет ничего. Так чтобы "Plug&Play".
Может я не прав? или этот раздел только для программистов?

Когда эта тема открывается - снизу похожие темы есть. Ни в одной я ничего даже приблизительно похожего не нашел. Просто какие то терки про код. А решения то где?

Последний раз редактировалось Rezervforall; 23.12.2009 в 12:01.
Теги
скачать (download), курс

 

Похожие темы
Тема Автор Раздел Ответов Посл. сообщение
Импорт курса валют innavish DAX: Функционал 18 13.10.2009 09:25
Загрузка курсов валют Alenka DAX: Функционал 5 20.01.2009 11:50
Кэширование курсов валют Tony Green DAX: Программирование 8 09.02.2007 10:02
Закачка курсов валют с Интернета. Hans DAX: Программирование 1 22.09.2006 12:08
Импорт из текстового файла - один их вариантов. YVAS DAX: Программирование 5 13.01.2004 16:31
Опции темы Поиск в этой теме
Поиск в этой теме:

Расширенный поиск
Опции просмотра

Ваши права в разделе
Вы не можете создавать новые темы
Вы не можете отвечать в темах
Вы не можете прикреплять вложения
Вы не можете редактировать свои сообщения

BB коды Вкл.
Смайлы Вкл.
[IMG] код Вкл.
HTML код Выкл.
Быстрый переход

Рейтинг@Mail.ru
Часовой пояс GMT +3, время: 21:01.