Сдаём в продакшен #2. Отчёт по стандартам написания кода. Коннекторы к OsEngine #75

Второй отчёт из четырёх, который надо сдать при написании коннектора к OsEngine. В этот раз речь пойдёт о стандартах написания кода.

Где мы?

Всего должно быть четыре файла на сдаче. Файл типа word office. Сегодня говорим про второй.

Скачать заготовки можно по этой ссылке: https://disk.yandex.ru/d/f0G-lpI1lYReNw

Отчёт по стандартам написания кода.

Коннектор: ALOR Open Api

Тип коннектора в проекте: Alor

Разработчик: Иван Иваныч Иванов

Телеграмм:

Телефон:

Дата:

1) Чужой код закрытый коммерческими лицензиями.

1. Использовался? Нет

2) Создание переменных.

1. https://smart-lab.ru/company/os_engine/blog/968171.php
2. Создание членов объекта во время его создания – отсутствуют.
3. Переменные типа VAR – отсутствуют.

3) Регионы внутри коннектора.

1. https://smart-lab.ru/company/os_engine/blog/969184.php
2. Всего штук – 12.
3. Нумерация – есть.
4. Названия из стандартов – да.
5. Доп регионы:

• Helpers – конвертеры.

• WebSocket check alive – коннектору не нужно посылать ping на уровне приложения.
• Queris – все вызовы REST сделаны на месте, где они требуются. Без общих методов.

4) Вложенность методов.

1. https://smart-lab.ru/company/os_engine/blog/969583.php
2. Методы в методе – отсутствуют.
3. New Action внутри метода не применяется.

5) Файлы и папки.

1. https://smart-lab.ru/company/os_engine/blog/969978.php
2. Стандарт выдержан – да.

6) Многопоточность.

1. https://smart-lab.ru/company/os_engine/blog/970542.php
2. Task, Asunc – отсутствуют.
3. Доп поток – разбор входящих из API сообщений.
4. Доп поток – раз в 20 минут запрос обновления торгового токена.

7) Циклы и Linq.

1. https://smart-lab.ru/company/os_engine/blog/971828.php
2. Применение Linq – отсутствует.
3. Цикл foreach – отсутствует.

8) Json / классы с описанием базовых данных API

1. https://smart-lab.ru/company/os_engine/blog/972551.php
2. Всё лежит в одной папке – Да.
3. Классы только для данного коннектора – Да.
4. Все типы переменных String – Да.
5. Convert.ToDecimal() – Отсутствует.
6. Конвертация ленты сделок и стаканов в классы OsEngine отдельным потоком – Да.

9) HTTP запросы.

1. https://smart-lab.ru/company/os_engine/blog/972792.php
2. Есть? – Да.
3. HttpClient создаётся в коде многократно? – Нет.
4. В качестве класса обёртки RestClient – Да.

10)  WebSocket.

1. https://smart-lab.ru/company/os_engine/blog/973202.php
2. Есть? – Да.
3. В качестве класса обёртки WebSocket4Net? – Да.
4. Событие Connected вызывается после открытия всех сокетов? – Да.
5. Событие Closed у сокета меняет статус на Disconnet? – Да.
6. Надо ли в лоб подавать пинги и есть ли для этого поток? – Нет.

11)  Защита от спама.

1. https://smart-lab.ru/company/os_engine/blog/973568.php
2. Подписка на бумаги. Есть защита? – Да.
3. Выставление ордеров. Есть защита? – Да.
4. Отзыв ордеров. Есть защита? – Да.
5. Запрос исторических данных по бумагам. Есть защита? – Да.
6. Запрос состояния портфеля. Есть защита? – Да.

12)  Версия C# и среды.

1. https://smart-lab.ru/company/os_engine/blog/973935.php
2. Сишарп выше 6 версии применялся? – Нет.

13)  Очищение памяти в методе Dispose.

1. https://smart-lab.ru/company/os_engine/blog/975564.php
2. Сокеты удалены по правилам? – Да.
3. Ссылки на события из-под объектов убраны? – Да.
4. Очищены ли ранее созданные массивы? – Да.

Удачных алгоритмов!

Пост из серии «Конекторы к OsEngine»

Серия о том, как стать настоящим программистом и изменить свою профессию.

Оглавление и смыслы здесь: https://smart-lab.ru/company/os_engine/blog/959953.php

Комментарии открыты для друзей, добавляйтесь!

OsEngine: https://github.com/AlexWan/OsEngine
Поддержка OsEngine: https://t.me/osengine_official_support

Регистрируйся в АЛОР и получай бонусы: https://www.alorbroker.ru/open
Сайт АЛОР БРОКЕР: https://www.alorbroker.ru
Раздел «Для клиентов»: https://www.alorbroker.ru/openinfo/for-clients
Программа лояльности от АЛОР БРОКЕР и OsEngine: https://smart-lab.ru/company/os_engine/blog/972745.php