VladimirCreator / Packages

A collection of packages where I practice git, GitHub, TypeScript, Swift and so on.
https://vladimircreator.github.io/Packages/
Apache License 2.0
1 stars 0 forks source link

Add "Тестовое задание №5" #62

Closed VladimirCreator closed 10 months ago

VladimirCreator commented 1 year ago

UPD: 3:58 PM Thu 21 Sep 2023

Полезные ссылки:

  1. Interfacing with UIKit
  2. Meme Creator
VladimirCreator commented 12 months ago

Опубликовать видео в последнем коммите.

Видео опубликовано в коммите 66a81d406496c9a916c9baf86e6f260bed1ab8be.

VladimirCreator commented 11 months ago

Issue №1

Создание отдельного репозитория под тестовое задание.

Fix

Зачем?

VladimirCreator commented 11 months ago

Issue №2

Нет необходимости указывать internal.

Fix

Ладно…

VladimirCreator commented 11 months ago

Issue №3

Ненужные комментарии.

Fix

Эммм… Ладно.

VladimirCreator commented 11 months ago

Issue №4

Всё должно быть в camelCase.

Fix

Согласен, но здесь существенной роли не играет.

VladimirCreator commented 11 months ago

Issue №5

Стоит поработать над именованиями.

Fix

Согласен, но сложно найти баланс между переиспользованием и назначением.

VladimirCreator commented 11 months ago

Issue №6

Свойства View в extension.

Fix

Не понял о чём речь.

VladimirCreator commented 11 months ago

Issue №7

Свойства нужно вынести из View в Model.

Fix

Не понял о чём речь.

VladimirCreator commented 11 months ago

Issue №8

Отсутствие или лишние пробелы снижают читаемость.

Fix

Что?

VladimirCreator commented 11 months ago

Issue №9

Большая вложенность ForEach снижает отрисовку View.

Fix

Не знаю, как это замерили, но предположим.

VladimirCreator commented 11 months ago

Issue №10

Много одинаковых структур с разными параметрами.

Fix

???

VladimirCreator commented 11 months ago

Issue №11

Следует использовать + I для выравнивания.

Fix

🙃

VladimirCreator commented 11 months ago

Issue №12

Куча минусов за View -> Нейминги, DRY и т.д.

Fix

???

VladimirCreator commented 11 months ago

Issue №13

Закоментированный код.

Fix

Эммм… Ладно.

VladimirCreator commented 11 months ago

Issue №14

Какой-то пустой файл.

Fix

NOWAY

5:20 PM Fri 22 Sep 2023

Речь шла о NavigationViewModel.swift? Был удалён в f59e7b2.

VladimirCreator commented 11 months ago

Issue №15

Карусель неправильно работает.

Fix

Что значит неправильно? Но я понимаю о чём идёт речь, поэтому засчитывается.

VladimirCreator commented 11 months ago

Issue №16

Вырывание цены и описания не соответствует требованиям.

Fix

Верно подмечено и исправлено в 635d372.

VladimirCreator commented 11 months ago

Issue №17

FlexibleLayout не реализован.

Fix

Не хватает контекста.

VladimirCreator commented 11 months ago

Issue №18

Шрифты не соответствуют требованиям.

Fix

Разве?

VladimirCreator commented 11 months ago

Issue №19

Блок с кнопками отличается от тех, что представлены в требованиях.

Fix

Действительно так.

VladimirCreator commented 11 months ago

Issue №20

Приложение лагает.

Fix

VladimirCreator commented 11 months ago

Issue №21

Баг с картинками (видно placeholder).

Fix

Я просто не придумал нормальную заглушку, а это значит, что это не неисправность.

VladimirCreator commented 11 months ago

Issue №22

Валидация не работает.

Fix

Верно подмечено.

VladimirCreator commented 11 months ago

Issue №23

Кривой ввод номера.

Fix

Верно подмечено.

VladimirCreator commented 11 months ago

Issue №24

Баг с bottomTabBar (появляется отступ)

Fix

Не понял о чём речь.