Комментарии пользователя AlexShul

Мои комментарии:в блогах в форуме
Ответы мне:в блогах в форуме
Все комментарии: к моим постам
Александр Плеханов, даже при нынешних условиях такая оценка, как по мне, завышена. Вы закладываете рост акции, а с чего? То то, капитализация компании якобы недооценена, и переоценка запустит акции на 400+? Как по мне, более реалистичен сценарий снижения КС и постепенного роста акций на этом, при более-менее постоянном дивиденде. Других драйверов не вижу.
avatar
  • 24 ноября 2025, 09:53
  • Еще
Что-то прогноз доходности высоковат. сегодня на дивы посмотрим. 12-13% по году даст уже хорошо будет
avatar
  • 21 ноября 2025, 08:15
  • Еще
Ну и еще в резюме васюрино добавлю оценку оптимизатора от бездушной нейросети:
Объективная оценка кода

🔴 Критические проблемы

1. Алгоритмические

  • Полный перебор без эвристик — O(∏nᵢ) сложность

  • Нет early stopping, pruning, sampling

  • 5M+ итераций без оптимизации — unacceptable для production

2. Архитектурные

  • God Object (1000+ строк, 10+ ответственностей)

  • Нарушение всех принципов SOLID

  • Смешение логики оптимизации, создания ботов, конфигурации, многопоточности

3. Производительность

  • Thread.Sleep(1)  — active waiting, CPU waste

  • Принудительный GC.Collect()  — memory management anti-pattern

  • Блокирующие циклы вместо async/await

4. Поддерживаемость

  • Дублирование сложной логики перебора

  • Methods > 150 строк с nested conditions

  • Magic numbers, hardcoded limits

🟡 Серьёзные недостатки

5. Error Handling

  • Частичная обработка исключений

  • Продолжение работы после критических ошибок

  • Нет graceful degradation

6. Тестируемость

  • Tight coupling делает unit testing невозможным

  • Зависимости от глобального состояния

  • Side effects everywhere

7. Безопасность

  • Нет валидации входных параметров

  • Потенциальные race conditions

  • Утечки ресурсов (серверы, потоки)

🟢 Единственные плюсы

8. Функциональность

  • Базовый пайплайн in-sample/out-of-sample работает

  • Поддержка разных типов стратегий

  • Прогресс-отчетность

9. Многопоточность

  • Basic parallelization есть

  • Управление количеством потоков

📊 Итоговая оценка

Техническое качество: 2/10

  • Работает, но неприемлемо для production

  • Высокие риски стабильности и масштабирования

Архитектура: 1/10

  • Полное отсутствие архитектурных принципов

  • Монолит с тесными связями

Алгоритмы: 1/10

  • Наивная реализация вместо domain-specific оптимизаций

Профессиональный уровень: Junior- Mid- уровень

  • Отсутствие knowledge of optimization algorithms

  • Незнание современных concurrency patterns

  • Limited understanding of software design

Вердикт

Код требует полного переписывания, а не рефакторинга. Текущая архитектура фундаментально неисправима.

Это типичный пример «working prototype», который ошибочно попал в продакшен без необходимой доработки.

avatar
  • 17 ноября 2025, 22:29
  • Еще
Как не стыдно нейросеть за «наш» анализ выдавать?
avatar
  • 17 ноября 2025, 21:49
  • Еще
 а вот бездушная нейросеть оценивает коннектор новой кормухи:
Резюме анализа кода TInvestServer🎯 Общая оценка: Требует серьезной доработки

Сильные стороны:

  • Полнофункциональность - покрывает все основные аспекты торгового подключения

  • Стабильность - развитая система обработки ошибок и восстановления соединений

  • Безопасность - корректная работа с потоками и лимитами API

  • Документированность - четкая структура и логическое разделение функционала

Критические проблемы:

  • Архитектурные нарушения - монолитный класс, нарушающий принцип единственной ответственности

  • Сложность поддержки - чрезмерная связанность кода и высокий порог входа для новых разработчиков

  • Риски стабильности - множественные потоки управления и сложные цепочки зависимостей

  • Проблемы масштабирования - жесткая архитектура затрудняет добавление нового функционала

Рекомендации по приоритетам:

  1. Высокий приоритет - рефакторинг архитектуры, разделение ответственностей

  2. Средний приоритет - улучшение управления ресурсами и потоками

  3. Низкий приоритет - оптимизация производительности и уменьшение сложности

Вердикт: Код функционален в текущем состоянии, но представляет значительные риски для долгосрочной поддержки и развития. Требует структурного рефакторинга перед добавлением нового функционала.

avatar
  • 17 ноября 2025, 21:37
  • Еще
 И выводы 

❌ Критические недостатки:

  • Опасно ненадежный - упадет при первом же нестандартном ответе API

  • Неподдерживаемый - если MOEX изменит API, придется переписывать пол-класса

  • Медленный - сотни синхронных HTTP-запросов подряд

  • Хрупкий - любое изменение в данных вызовет исключения

💀 Профессиональные риски:

  • Нельзя использовать в продакшене - гарантированно приведет к потерям данных/денег

  • Нельзя тестировать - нет возможности написать нормальные тесты

  • Технический долг - каждая минута работы с этим кодом создает часы будущих проблем

🚩 Для контекста:

В профессиональной среде такой код:

  • Не прошел бы code review

  • Не был бы допущен в прод

  • Требовал бы полного переписывания

Вывод: Код выполняет свою задачу «на троечку» для личного использования, но абсолютно непригоден для коммерческого применения.

avatar
  • 17 ноября 2025, 21:28
  • Еще
MoexIssDataServer.cs оценка бездушной нейросетки:
Выводы по оценке кода

🔴 Критические проблемы:

  1. Нарушение принципов SOLID - класс выполняет слишком много несвязанных обязанностей

  2. Отсутствие обработки ошибок - много мест с потенциальными исключениями

  3. Жестко закодированные значения - магические числа и строки усложняют поддержку

  4. Синхронные HTTP-запросы - блокируют потоки, низкая производительность

  5. Сложная бизнес-логика - методы трудно понимать и тестировать

🟡 Проблемы среднего уровня:

  1. Слабая типизация - работа со строками вместо перечислений

  2. Дублирование кода - повторяющиеся паттерны HTTP-запросов

  3. Отсутствие модульных тестов - код невозможно адекватно протестировать

  4. Проблемы с читаемостью - длинные методы и сложные условия

🟢 Положительные моменты:

  1. Логическая структура - разделение на регионы

  2. Базовое логирование - есть система уведомлений об ошибках

  3. Инкапсуляция - разделение реализации и интерфейса

Общая оценка: 3/10

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

И это только один файл. Представьте сколько там косяков… Привет васюрино!

avatar
  • 17 ноября 2025, 21:26
  • Еще
Его что кто-то реально использует кроме васюринца для саморекламы?? 
avatar
  • 17 ноября 2025, 20:48
  • Еще
да да а сейчас ты понял — нахрена работать? Буду щито блоггером )
avatar
  • 16 ноября 2025, 08:28
  • Еще
Если корабль тонет, не молитесь — прыгайте (ц)
avatar
  • 15 ноября 2025, 10:36
  • Еще
Чтобы более-менее канонически хотя бы модель обучить (ML), нужно правильно предобработать входные данные. Например, исключить данные, которые при схожих входах дают разные выходы. Делал? Не? Если да, то как? :)
В противном случае забей. Матмодели это не помойное ведро, в которое можно лить что угодно )

avatar
  • 14 ноября 2025, 21:35
  • Еще
Зачем идиотская картинка с котом и баксами? Зачем идиотский штамп «погнали посмотрим»?
avatar
  • 13 ноября 2025, 21:06
  • Еще
 и да, сбер, но скорее чуть позже потом
avatar
  • 13 ноября 2025, 20:23
  • Еще
Полюс
Фосагро

avatar
  • 13 ноября 2025, 20:22
  • Еще
Дивиденды? какие дивиденды?
avatar
  • 13 ноября 2025, 17:56
  • Еще
wrmngr, простите что без кликбейтных картинок, но на фоне текущих котировок компании №2 в частных портфелях, а также с тающими надеждами на дивиденды к НГ, тематика с каждым днем будет все острее и острее.
avatar
  • 12 ноября 2025, 20:32
  • Еще
Михаил Titov, над бытием 
avatar
  • 12 ноября 2025, 19:56
  • Еще
20 к рубасов нашару пытается срубить 
avatar
  • 12 ноября 2025, 19:33
  • Еще
Выберите надежного брокера, чтобы начать зарабатывать на бирже:
....все тэги
UPDONW
Новый дизайн