Pull to refresh

Comments 6

Не знаю зачем все это , но выглядит не нужно )

Ну а так ни kafka , ни timescaledb , ни тайминговых агрегационно запросов при изменение цен...

Если статья про единый апи для всех бирж, то есть бесплатные профессиональные сервисы которые в отличии от вашего кода не потеряют свечи при "отключении интернета" а так же биржи это не всё , сейчас ликвида много и на dex

Ну, я бы не был так категоричен. Человек учится, пробует, экспериментирует, показывает другим. Потом может это перерастёт в конкурентоспособный продукт, а это в свою очередь конкуренция и развитие прогресса, технологий да и мира всего. Как говорится путешествие начинается с маленького шага. Мне было интересно, спасибо! Жду продолжения.

извините, а что именно не понравилось в коде?

Не могу сказать, что все плохо, но вот именование переменных вида "m", "p", и т.д. выглядит не очень. Можно не экономить буквы.

Еще, архитектурно все находится в куче, в одном проекте, если потом захочется сделать веб клиент, то придется переделывать все. Не хватает разделения на слои.

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

Весь код принудительно синхронный, что выглядит странно, еще и Thread.Sleep-ами заторможен.

Это то, что бросилось в глаза, может, если открою ide, то что-то еще вылезет.

Отличная статья!

Было бы совсем замечательно, если бы автор продолжил публикации по данной теме, например, о том, как работать с аккаунтом пользователя в Cucoin, получать инфу о состоянии счета, совершать сделки.

Sign up to leave a comment.

Articles