Shopping Cart

[AndreiChenchik]

PR contains

• 4

  • 12

  • 11

  • 10

  • 18

  • 9

  • 8

  • 19

  • 86

Time

• Total estimated 7:10
• Total spent 17:00

Notes

• выбранная командой архитектура и способ верстки: MVVM + Код
• декомпозиция эпика (ссылка на таск-трекер): #4
• декомпозиция эпика (приложенный текстовый документ .md): файл в репе TODO.md https://github.com/BlackBullSquad/iOS-FakeNFT-StarterProject-Public/blob/6cd453a204ef2e508fff0dcde9e750da8691fee4/TODO.md?plain=1#L1-L13
• первоначальная оценка времени выполнения задач: см TODO.md
• реальное время, потраченное на решении каждой задачи: см TODO.md

[maxim-papier]

Похоже vStack надо по вертикали по центру аватарки выравнивать

[AndreiChenchik]

@maxim-papier пофиксил, доделал, смотри аппрувь :)

[maxim-papier]

Мне очень понравилось. Придраться не к чему. Крут!

[AndreiChenchik]

От @ABashkirova

🐛 Что нужно исправить, чтобы двигаться дальше

• [x] UIKit, Foundation не хватило "При сетевых запросах блокируется интерфейс"
• [x] #79 Чистота кода проскочил один нелегальный force unwrap – нужно добавить обработку.
• [x] #79 Архитектура есть серьезные ошибки в реализации: местами View делает больше работы чем надо, иногда работа с моделью происходит напрямую из вью (как хранение, так и взаимодействие). Не все экранны поддерживают архитектуру.
• [x] #81 Работа с сетевыми запросами: местами есть оставленные без обработки ошибки. Нужно решить как с ними поступать продуктово и доработать запросы
• [ ] ~Использование многопоточности для обработки ответа не продемонстрировано. Предполагается здесь наличие бекграудн очереди для сборки запроса/разборки ответа.~
• [x] Работа с Git: здесь по большей части хорошо, но иногда проскальзывают комиты WIP - не понятно что там такое, такие коммиты до публичного репозитория не должны долетать.
• [ ] ~Автотестирование не продемонстрировано.~

🚀 Челенджи и что можно лучше

• [x] #79 Самый большой челендж для тебя это разрбраться с ответственностью у элементов MVVM, корректно собирать зависимости каждого. Ожидается что модель(сервисы, данные) являются зависимостью вьюмодели, а вьюмодель - единственная зависимость у вью.
• [x] #82 Очень будет круто увидеть работу с добавленной ссылкой на вебвью.
• [x] #80 И также доработанные ассеты для темной темы: кажется там не очень много работы, чтобы поддержать эту допфичу.

От @maxim-papier

• [x] #80 Когда открывается экран оплаты, в дизайне нету таббара.

[AndreiChenchik]

@ABashkirova осталось два пункта нужны уточнения:

Использование многопоточности для обработки ответа не продемонстрировано. Предполагается здесь наличие бекграудн очереди для сборки запроса/разборки ответа.

Тут по критериям говорят про гонки, которые при блокировке интерфейса в целом можно +- избежать, а дальше URLSession работает в бэкграунде и пока ты не получил нужные данные просто не надо оттуда выходить в DQMain. Не очень понятно что тут еще делать?

Автотестирование не продемонстрировано.

Вроде это не обязательно, нет?

[IBDesignable]

Не очень понятно что тут еще делать?

Уточни пожалуйста у Наставника, пока у меня информация именно такая: да запрос отправляется на очереди с URLSession, но сам процесс запекания запроса и его парсинга можно положить в свою бекграунд очередь. Выглядит натужно, согласна. Приму аргумент согласованный с Наставником

Автотестирование не продемонстрировано. Вроде это не обязательно, нет?

Все верно, все так.

@AndreiChenchik работу смогу отдать обратно только завтра – технические проблемы с первой сданной работой. Комментарии в ПР появятся сегодня.

[AndreiChenchik]

@IBDesignable спасибо! @x128 ожидаю твой комментарий по поводу потоков, тут еще усложняет тот факт, что парсинг запроса происходит распаковщиком codable, который был предоставлен, можно его конечно поменять, но сомневаюсь что в этом была идея. А прокинуть три Intа в struct на этапе формирования запроса... это прям вообще хз... мне вообще кажется что компилятор это заинлайнит и там просто нечего перемещать между очередями.

[x128]

@x128 ожидаю твой комментарий по поводу потоков, тут еще усложняет тот факт, что парсинг запроса происходит распаковщиком codable, который был предоставлен, можно его конечно поменять, но сомневаюсь что в этом была идея.

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

А прокинуть три Intа в struct на этапе формирования запроса... это прям вообще хз... мне вообще кажется что компилятор это заинлайнит и там просто нечего перемещать между очередями.

При возможности можно сделать для тренировки, т.к. на реальном проекте параметры запроса могут собираться из многих разных источников, или какой-нибудь хедер с асинхронным шифрованием. @IBDesignable о кстати, а это как раз идея на тему, чё можно добавить в ТЗ по-быстрому :)

[AndreiChenchik]

@x128 @ABashkirova @IBDesignable спасибо, я тогда вычеркиваю оба пункта :) Есть что доделать другое, по другим проектам.

[IBDesignable]

Andrei Chenchik – Ревью диплома.pdf

[AndreiChenchik]

Всё порезолвил, ура! @ABashkirova @x128 спасибо! ❤️❤️❤️❤️❤️❤️ Жду команду и мержим.

[AndreiChenchik]

Ура! Все конфликты исправлены. Эта последняя ветка на мёрдж. Дальше будем отправлять на ревью еще раз с видосами!

Спасибо @maxim-papier @IBDesignable @x128 @ABashkirova @Filosuf