Альтернативный Альфа-Коннектор

ra81: Какие предложения по совмещению? Я пока плохо представляю сей процесс.

1. Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано).
2. Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем).
3. Обсуждается, какие фичи правильнее.
4. Код переноситься.

Mikhail Sukhov:

ra81: Какие предложения по совмещению? Я пока плохо представляю сей процесс.

1. Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано).
2. Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем).
3. Обсуждается, какие фичи правильнее.
4. Код переноситься.

Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все.

А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :)

ra81:

Mikhail Sukhov:

ra81: Какие предложения по совмещению? Я пока плохо представляю сей процесс.

1. Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано).
2. Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем).
3. Обсуждается, какие фичи правильнее.
4. Код переноситься.

Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все.

А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :)

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

Новый коммит http://stocksharp.codeplex.com/SourceControl/changeset/changes/15775

1. Исправил проблему озвученную выше с переносом на другой ПК. теперь проблемы быть не должно.
2. Исправил работу с CancelTime, ExpiryDate. Время истечения заявки задается в ExpiryDate. Время отмены заявки приходит в CancelTime. Было иначе. Следует учитывать что терминал не возвращает реальное время отмены, поэтому при получении ордера, коннектор ставит время отмены сам. Если коннектор был запущен и в терминале есть несколько ордеров отмененных, то у них у всех будет время отмены равно текущему времени. Но стоп/старт экспорта не будет менять время отмены ордера. Оно обновляется только один раз, когда коннектор получает данные о том что ордер отменился. По умолчанию время отмены равно null.

Спасибо, Родион. Будем тестить.

Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.

Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.

Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.

Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить.

Mikhail Sukhov: Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.

Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.

Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.

Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить.

Овердизайн остался похоже от полностью асинхронной модели. Планировал так сделать.

Локи есть только на метода терминала. Дедлок Не возможен при моей архитектуре асинхронной передачи данных и приема ордеров. Дедлоки были, но в другом месте. Я избавился о них. Да и они были временным решением. А без локов возможны зависания терминала. ЧТо я и наблюдал :).

Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :).

Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)

ra81: Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :).

Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки.

ra81: Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)

Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать?

Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки. Это совсем грубо. И неудобно работать. И главное непрозрачно в коде.

Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать? Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :). Сергей думал смерджить, посмотрел и понял, что не мерджится. Вот как-то так.

ra81: Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :).

Я это понял сразу.[rolleyes]

Почему это было неправильно:

1. Два коннектора, оба далеки от 100% готовности.
2. Ревьюровать есть время только один коннектор. И это скорее всего будет основной.
3. Для юзеров опять же недоступен сейчас альтернативный коннектор.
4. С учетом малого кол-ва рук, скорость разработки медленная для каждой из версий коннектора.