Mikhail Sukhov
|
Дата: 22.03.2012
ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься.
|
|
Спасибо:
|
|
|
|
|
ra81
|
Дата: 22.03.2012
Mikhail Sukhov ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься. Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все. А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :)
|
|
Спасибо:
|
|
|
|
|
Sergey Masyura
|
Дата: 22.03.2012
|
|
|
|
ra81 Mikhail Sukhov ra81 Какие предложения по совмещению? Я пока плохо представляю сей процесс. 1) Выписываются фичи, которые есть в альтернативном коннекторе, и отсутствуют в основном (уже сделано). 2) Выписываются фичи, что есть в основном коннекторе, и что отсутствуют в альтернативном (поговорю с Сергеем). 3) Обсуждается, какие фичи правильнее. 4) Код переноситься. Совершенно верно. В итоге мы получим мою версию коннектора или придется дополнять версию базовую. Поскольку как мне известно все фичи что есть в базовом есть у меня. Сергей писал под задачу. Что надо было то сделал. Он не ставил целью сделать пупер либу. Мне нужно было больше функционала, я расширил его код. Вот собственно и все. А вообще предлагаю с ним и поговорить. Пусть он озвучит свое видение. Он же тоже замешан во всем этом :) Скрещивать два коннектора смысла мало в силу различий в архитектуре и авторов - получится нечто еще более сложное и менее рабочее. У Родиона есть хорошия решения, которые можно позаимствовать. Предлагаю оставить два коннектора, а я по мере возможностей приведу коннектор из транка в порядок.
|
|
Спасибо:
|
|
|
|
|
ra81
|
Дата: 23.03.2012
Новый коммит http://stocksharp.codepl...changeset/changes/15775
1) Исправил проблему озвученную выше с переносом на другой ПК. теперь проблемы быть не должно. 2) Исправил работу с CancelTime, ExpiryDate. Время истечения заявки задается в ExpiryDate. Время отмены заявки приходит в CancelTime. Было иначе. Следует учитывать что терминал не возвращает реальное время отмены, поэтому при получении ордера, коннектор ставит время отмены сам. Если коннектор был запущен и в терминале есть несколько ордеров отмененных, то у них у всех будет время отмены равно текущему времени. Но стоп/старт экспорта не будет менять время отмены ордера. Оно обновляется только один раз, когда коннектор получает данные о том что ордер отменился. По умолчанию время отмены равно null.
|
|
|
|
|
OvcharenkoVI
|
Дата: 23.03.2012
Спасибо, Родион. Будем тестить.
|
|
Спасибо:
|
|
|
|
|
Mikhail Sukhov
|
Дата: 23.03.2012
Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.
Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.
Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.
Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить.
|
|
Спасибо:
|
|
|
|
|
ra81
|
Дата: 23.03.2012
|
|
|
|
Mikhail Sukhov Посмотрел вчера код альтернативного коннектора. Чтож. Написано много.[laugh] Я бы даже сказал, есть какой-то элемент овердизайна.
Сергей обещал перетащить свои изменения из транка в дев. Там сделана асинхронная отправка заявок. Ваш коннектор с локами - это путь к дедлогам (что и случилось в маркет котировании). Поэтому, мне кажется, основной коннектор так и остается основным коннектором.
Идея с таблицами интересная. Советую посмотреть на структуру метаданных в квике и плазе. Потому что сейчас выглядит как-то недоделанно.
Вести два коннектора накладно. Предлагаю доделывать только один. Его тестить, его и ревьюить. Овердизайн остался похоже от полностью асинхронной модели. Планировал так сделать. Локи есть только на метода терминала. Дедлок Не возможен при моей архитектуре асинхронной передачи данных и приема ордеров. Дедлоки были, но в другом месте. Я избавился о них. Да и они были временным решением. А без локов возможны зависания терминала. ЧТо я и наблюдал :). Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :). Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)
|
|
Спасибо:
|
|
|
|
|
Mikhail Sukhov
|
Дата: 23.03.2012
ra81 Таблицы квика я не могу увидеть. Таблицы Алора и плазы я смотреть. Решил не наворачивать подобные конструкции отдав предпочтение простоте решения. Работать удобно и реализовать было быстро :).
Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки. ra81 Впрочем если будет рабочее решение мне неважно какое оно будет. Работало бы всё сразу, не было бы второго коннектора :)
Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать?
|
|
Спасибо:
|
|
|
|
|
ra81
|
Дата: 23.03.2012
Цитата:Быстро реализовать - это как сейчас в основном коннекторе, ввиде строчки. Это совсем грубо. И неудобно работать. И главное непрозрачно в коде. Цитата:Если что-то не работало в основном коннекторе, то надо было его лечить. Зачем новый создавать? Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :). Сергей думал смерджить, посмотрел и понял, что не мерджится. Вот как-то так.
|
|
Спасибо:
|
|
|
|
|
Mikhail Sukhov
|
Дата: 23.03.2012
ra81 Собственно еще раз повторюсь. Я взял базовый коннектор, и вылечил всё, что не работало. В итоге получился альтернативный коннектор :). Я это понял сразу.[rolleyes] Почему это было неправильно: 1. Два коннектора, оба далеки от 100% готовности. 2. Ревьюровать есть время только один коннектор. И это скорее всего будет основной. 3. Для юзеров опять же недоступен сейчас альтернативный коннектор. 4. С учетом малого кол-ва рук, скорость разработки медленная для каждой из версий коннектора.
|
|
Спасибо:
|
|
|
|