Pull to refresh

Comments 2

Вы конечно извините, но это ужасно. Ладно что 3-я версия языка, не так много поменялось, но:
— полный хаос в написании — в каждом файле/классе по-разному;
— говорил в комментариях к прошлой статье что force optional unwrapping не доглядели, но тут используете его вовсю, при чем и где не надо тоже;
— lazy var (да еще и в паре с предыдущим пунктом) это просто что-то с чем-то;
— почему все еще не DataTask для простейших запросов не понятно;

Пожалуйста, в следующий раз познакомьтесь с языком ближе, перед тем как писать статью.
Прощу прощения, но по моему UIViewController должен отвечать за работу с View(отображением, это подразумевает весь UIKit) и форматирование данных. Он не должен по хорошему ничего знать про то что есть сеть и как с ней работать.
Создайте какой-то класс, например VoteUtility, который уже позволяет поднять или опустить и он же знает куда данные передавать и какие.

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

Так же считаю что комментарии к UI, в статье о работе с сетью (исходя из заголовка) являются излишними.

И коль пошла такая пьянка — почему не создать бы расширение на URLSessionDownloadTask и не мучаться с еще одним классом, если не хочется писать какой-то враппер?
Sign up to leave a comment.

Articles