👏 Дія відкрила код - Diia code review 🤦
👉 Група для обговорення Diia: t.me/+f6d5MCe-euthOTFi
👉 Github експерта: github.com/tshemsedinov
👉 Канал на youtube: / timurshemsedinov
👉 Metarhia комьюніті: t.me/metarhia
👉 Група Node.js UA: t.me/nodeua
👉 Твітер: / tshemsedinov
👉 Linkedin: / shemsedinov
Пікірлер: 93
Потім вони напишуть - ми спеціально виклали стару версію щоб перевірити як відреагує суспільство, що ми все пофіксили і ми дуже вдячні за фідбек блаблабла
З приводу самого рев'ю - в цілому все по ділу. Єдине що, треба буде всі недоліки пріоритезувати грамотно і виправляти перш за все самі критичні. Дещо нове для себе дізнався з відео (особливо про локалізацію), тож піду читати.
Тимуре, дуже дякую за Вашу роботу. Мені особисто було цікаво глянути такого роду аналіз із конкретикою задля свого професійного розвитку і розуміння нормальних підходів в програмуванні :)
Приємно вражений, що українською! Сподіваюсь надалі відео теж будуть соловʼїною 👏🏻
@TimurShemsedinov
2 ай бұрын
Будуть тією мовою, якою зручно мені та глядачам, я і італійською можу, але мене дивляться у Казахстані та Азербайджані, наприклад, у Іспанії та Ізраїлю
@JuniorSoftwareVlogger
2 ай бұрын
Viva Italia❤ @@TimurShemsedinov
@solution-n-social
2 ай бұрын
@@TimurShemsedinov італійською було б цікаво, якраз хочу вивчитИ щоб читатИ книжки Леонардо у оригіналі.
Повністю згоден, перше що спало на думку - а де опис процесу, як правильно вносити/пропонувати свої до зміни?
Дякую!
Хороший початок як на мене. Цілком по опенсорсному - зарелізити хоч щось а потім потроху розвивати. Головне щоб знайшлось пару фанатів в команді, які б побачили в цьому цінність. За ревʼю автору велике дякую. Якісні і точні зауваження по коду. Загалом все виглядає значно краще ніж я собі думав, все ж це державний проект.
Лайк, підтримка, коментар
Коротше відмили бабок на дії немало. Але в принципі, враховуючи код вінди 95 яким він був з самого почату, дія виглядає непогано. Непогано, щоб переписати))
Лайк, спасибо за интервью у Николая Фельдмана.
@vugpgu
2 ай бұрын
Фельдман из мутной компании Романенко-Дацюка-Арестовичей
дякую) став менше комплексувати через свій старий код)) а крапку з комою( ; ) ставити в кінці рядку вже не в тренді?
@TimurShemsedinov
2 ай бұрын
😊😃🤣
@DmytroZhyvonitko
2 ай бұрын
TS компілятор вміє це робити за вас, тому хто як робить.
@TimurShemsedinov
2 ай бұрын
@@DmytroZhyvonitko є випадки, коли не вміє і все ломається
Цікаво чи вони чули про якийсь code quality tools, хоча б SonarQube?
Дякую. Видалив Дію )
Лайк, підтримка, коментар v.2
дякую за відео. дещо корисне і для себе дізнався
57:57 синхроные функции и их использование, тут надо внимательно смотреть на юзкейсы
Будь які взаємодії мають бутИ прописані контрактами (структурами) починаючі з скалярних і розширюючи їх, тоді і typescript не потрібен. :)
@TimurShemsedinov
2 ай бұрын
💯%
А відомо про команду розробників? Хто там керував процесом, підтверджував реквести і тд?
@TimurShemsedinov
2 ай бұрын
Нам все відомо, у нас скрізь свої люди
@CodeBeep
2 ай бұрын
@@TimurShemsedinov привіт передали?))))
Дякую за відео українською ❤
Ну, по-перше, це, звісно, не відкритий код (опенсорс), а просто "публічно доступний код". І головне, що керівництво проекту мало б спочатку замовити експертну оцінку, аніж поспішати з публікацією.
страшно такими сервісами користуватись, тому не користуюсь
@od1ez
2 ай бұрын
Один з вагомих елементів майбутнього цифрового концлагеру
@TimurShemsedinov
2 ай бұрын
Усі цифрові системи повинні бути в руках громадян, а не в руках держави
схоже що вони виклали свій смітник, що не увійшло в продакшен
В пул реквестах є такі слова як bad and other adjectives, а як ми знаємо немає ідеального коду. Пропоную відразу додавати імплементацію якогось ішшю, як це покращити, а не просто коментувати, щоб краще донести думку до мейнтейнерів.
@TimurShemsedinov
2 ай бұрын
Ні хто не збирається мерджити, це технічно неможливо, я про це кажу у відео
@vugpgu
2 ай бұрын
Чого пишеш суржиком? Пиши вже англійською як не вмієш донести думку
Дякую за відео українською)
@pavel.88
2 ай бұрын
чувствуется, шо выдавливает из себя, как может, но все равно режет слух
@AndrewFloatrx
2 ай бұрын
@@pavel.88якщо ріже слух то не слухай 😂… дивний комент…
@TimurShemsedinov
2 ай бұрын
Слава Аллаху, Господу світів!
Чекаємо якихось офіційних пояснень
@TimurShemsedinov
2 ай бұрын
Виправдовуватись вони штучні мовні моделі змушують
тепер зрозуміло чому дія часто видае сталася помилка спробуйте пізніше, мабудь пізніше коли цей код розгребуть
@TimurShemsedinov
2 ай бұрын
Це тому, що там мікросервіси, а оркестрації нормально не зроблено, ось і можно видати тілько повідомлення: Ой, щось сталосі
53:36 молчим уже про транзакционность в промис алл
PR на те й називається PR (піар), ось вам і опис процесу.. Не варто очікувати, щось від людей зі світу PR, бо так скоро будемо на Марсі з вами смакувати кавусю.
Дуже крутий викладач. Дякую за навчання.
@TimurShemsedinov
2 ай бұрын
Усі навчальні матеріали тут github.com/HowProgrammingWorks
1:09:09 а есть пример из практики работы с каким-то реестром?! Чтобы понять как там эта трембита коммуницирует с внешним миром?
@TimurShemsedinov
2 ай бұрын
Есть опыт работы с трымбитой, но это ещё то ведро с болтами, хотя оно хоть что-то, на чем держится половина ИТ в государстве
Цікавий контент, дякую
Да ну хто ж буде за автоматичне тестування будь яких лівих змін сплачувати ?
@TimurShemsedinov
2 ай бұрын
Безкоштовно на гітхабі для оупенсорса у певному обсязі, що вистачило б
И все это только по верхам стоило нам $16M во время войны
@TimurShemsedinov
2 ай бұрын
Я и так читая код седины добавил себе
@anastasiag7380
2 ай бұрын
Дія не фінансується державними коштами.
@AndreyNikishaev
2 ай бұрын
@@anastasiag7380 1) это не так 2) даже если какието деньги заходят из вне, точно такое же преступление их бездумно тратить во время войны.
@AndreyNikishaev
2 ай бұрын
@@anastasiag7380 вопервых не факт. во вторых 16м во время воцныне лишние
автори Дії ви є тут?)
@TimurShemsedinov
2 ай бұрын
Вони є у групі
@user-kl1cn1sr1b
Ай бұрын
Думаю автори дії самі прекрасно розуміють проблеми які в коді і без ваших "експертних" думок
в чем преимущество перед свитч кейсом? помоему не имеет смысла замечание, и обоснования тоже по нему не увидел kzread.info/dash/bejne/mZt9zZpsd7KYZaw.html
@TimurShemsedinov
2 ай бұрын
switch/case устарел ещё во времена появления c++, это атавизм из середины 20 века, он громоздкий, и его для js просто заменить коллекцией Object или Map, которые проще расширять, проще читается, можно на лету мегять и даже работать быстрее будет, посмотрите лекции мои про антипаттерны и погуглить почему ещё Дэйкстра был против свича
@vugpgu
2 ай бұрын
@@TimurShemsedinovвека "двадцать" нет. Есть век XX
@victorvarenic7610
2 ай бұрын
@@TimurShemsedinov все это очень субективно кроме скорости, за что не уверен
@TimurShemsedinov
2 ай бұрын
@@victorvarenic7610 скорость нафиг никому не нужна, нужна гибкость в рантайме и краткость синтаксиса
Дякую за ревʼю українською.
код гімна, набирали джунів з курсів?
@TimurShemsedinov
2 ай бұрын
З Епаму та академії Поплавського
@vugpgu
2 ай бұрын
Говорят Гоша Дударьь писал на стримах. И Владилен немного писали фронт.
@StarkElessar
2 ай бұрын
@@vugpgu про Гошу и винипуха отдельный лайк))))
Лайк за перехід на рідну мову.
@TimurShemsedinov
2 ай бұрын
Я навіть не володію кримськотатарською 😢
@user-gr3gh7ml1w
2 ай бұрын
Ніколи не пізно згадати - хто ми є насправді@@TimurShemsedinov
@TimurShemsedinov
2 ай бұрын
@@user-gr3gh7ml1w онуки Хубілая та Угідея, правнуки Тімучина? Ні, веду свій родовід від майбутніх поколінь
Жерсть, стидоба, чого я червонію як дивлюсь цей код?...
ужас
Послушал, в общем не удивлен, походу как и всегда наняли джунов а деньги украли
Здравствуйте! Только на украинском теперь?
@TimurShemsedinov
2 ай бұрын
Ну Diia это локальная тема, а так вы можете видеть на канале оба языка есть
Ну да, зачем украинский???
@TimurShemsedinov
2 ай бұрын
Это интересно украинской аудитории, а на курсе по ноде я это разберу подробнее все
@vladiusvostokus
2 ай бұрын
Автосубтитри норм
@vitalyks
2 ай бұрын
@@TimurShemsedinov тогда согласен.👌 Очень хочу подписаться на курс по NodeJS, но пока штудирую react и react native.
Ну зачем эти страдания с украинским? Ну видно же мучаешься…
@ukrainianhulk
2 ай бұрын
Схоже що це ти тут мучишся
@TimurShemsedinov
2 ай бұрын
Вообще жизнь не конфетка, страдай или не живи
@user-fc4iv9xv1j
2 ай бұрын
@@TimurShemsedinov Держись…
@user-vu6hn4ul2i
2 ай бұрын
Зачем? Ответ мы все прекрасно знаем.
@gnom-om
2 ай бұрын
@@user-vu6hn4ul2i а раньше почему слабо было напрягаться? )) пацрыеты ))