Кодревью опенсорс проекта или как выглядят 10 лет тех.долга
Vložit
- čas přidán 2. 06. 2024
- Вчера в телеграм канале я спросил подписчиков какое паблик видео они хотят чтоб я сделал и мне прислали ссылку на опенсорс проект и пожелание покодревьюить. И вот что из этого вышло.
ссылка на проект
github.com/HabitRPG/habitica-...
мой гитхаб аккаунт
github.com/johnnysc
Ссылка на телеграм канал
t.me/easyCodeRu
Крутецкое видео хоть и не всё пока осилил, почаще бы такие видосы в паблик, хоть и хейт льётся на такое от "одарённых". Узнал про поворот экрана в любой прилаге через сплит мод, за это отдельное спасибо, единственный момент, хотелось бы конечно, чтоб ты билданул проект, шоб попроваливаться прям в каждый класс)
Ну так в проекте 75 тыщ линий кода
Если в каждый класс вчитаться то видео будет на 10 часов
Спасибо за видео!
Такого говнокода я ещё не видел в своей жизни ))) Мне хватило увидеть что творится в MainActivity и частично в ViewModel'e, и я понял, что нет смысла тратить больше времени и смотреть полностью видео до конца как ты разносишь В Пух и прах этот говнокод ))) Откуда у тебя столько терпения? :D
да в принципе ты прав. Мне сегодня достаточно посмотреть на 1 вьюмодельку чтоб понять что все плохо в коде. А терпение: не знаю, я же работаю с людьми, без него никуда
Спасибо! Коммент в продвижение.
мега удачный проект под руку попался, писали 100 человек и каждый мыслит по своему, 90% которых индусы, а остальные случайно попали
Ну и на протяжении 10 лет сменялись подходы и стиль
И вуаля. И хмл и компоуз
И мввм и отсутствие архитектуры вообще
Спасибо за ревью моего любимого раньше проекта!
Пользовалась им ещё в универе, очень люблю геймификацию в обучении, а в хабитике было много людей со всякими тематическими чатами в том числе. Тоже слышала о нем, как об архитектурном примере, но как-то даже подумать не могла что там такое под капотом.
Прилож прикольный на самом деле, если не быть сильно агрессивным в тестировании)0 Отсутствие локализации на ру - классика, так что этому уже даже удивляться лень...
Сайт (когда я активно юзала хабитику) был несравнимо лучше чем андроид прилож, но to-do list в вебе открывать - совсем извращение.
P.s. подписка, кстати, продается за 3к на 12 месяцев, а в проект контрибьютить можно без комментариев - удобно 😅
Локализация частичная. То есть они начали делать и не закончили
"Активити должна заниматься двумя вещами: - разруливать все что относится к активити"
- например? что относится к активити?
1. Вьюхи активити
2. Навигация фрагментов
3. startActivityForResult и пермишены например
41:16 можешь показать или ссылку дать какую livaDataсейчас используешь которая не ломается, я знал твою обёртку communication над liveData, но не знал есть ещё более крутое решение
Попробуй посмотреть в практис тдд репе. Там много обсервер патерн задач
Мне кажется или easyCodeRu во всех своих видео нарушает принципы KISS и YAGNI?
Да
@@easyCodeRu Ну тогда это смело)
Да
Контент полезный, но братух, окей? от этих слов паразитов да желание слушать дальше мгновенно пропадает, окей?
Так и хочется сказать нет б… не окей, окей? Да?
Есть у меня такая штука как интенсив где на протяжении 5 часов я объясняю как писать код. И чтобы понимать что ребята понимают меня я время от времени спрашиваю типа : все ясно? Вот тут вот так, ок?
Так что, братух, не держи зла, окей?