Такі як Ви теж мають вплив на тон дискусії.

Між іншим був сертифікованим спеціалістом по бухгалтерській підсистемі для вісімки :)

ага :)

На жаль я так ще і не побачив Ваших шедеврів програмування, але то таке.

[[NSArray alloc] init];

чим саме поганий такий спосіб?

Якщо Вам так зручніше, то хай буде так :)

А чому Вам соромно себе назвати?

Встидаєтесь власного імені?

Ціную.
Дякую!

Але після цього Ви вчора писали, що буде таки від Вас ревью.

Не переймайтесь.

І головне, що це перший коментар :)

Дякую містер Annonymous Annonymous .
Особливо за наведені приклади солюшинів, які Ви так смачно рознесли.

Чувак, ты понимаешь, что после этого треда найти работу станет в 100 раз сложнее? Разве что в «Рога и Копыта» куда программистов берут на вес? Доу читает куча народу и, поверь, этот тред сейчас активно обсуждается в куче закрытых чатиков и каналов (откуда я о нём и узнал). Ты теперь звезда украинской iOS разработки, поздравляю.

Якщо потім працювати у такого як Ви — то краще бути «реджектнутим» одразу.

Последняя попытка быть серьёзным. Тебе максимально корректно указали на кучу просто элементарных ошибок в коде, которые недопустимы даже для джуна. Дело совсем не в форматировании, а в том, что базовые вещи сделаны неправильно.

Нарешті Ваше виховання дозволило перейти до конкретніших речей.

Самое простое из того, что сразу бросается в глаза:
— код написан на Obj-C который был актуален в каменном веке, спасибо что хоть нет ручного управления памятью

Вже втомився повторювати, це було узгоджено з рекрутером. Я питався на чому писати, мені сказали Обджектів Сі теж треба знати :)

У мене є товариш у Нідерландах, дядькові 60 років, ще пише на Коболі.

— не используются реюзебл целс (это умеют джуны с 2-3 месяцами опыта)

Це використовуються, вже відповідав іншому товаришу
github.com/...​reenViewController.m#L153

— синглтон это плохо само по себе, это code smells, но в коде определён синглтон, который каждый раз создаётся явно, а это явная ошибка

50/50
У CoreData є такий прикол як запізнення на 1-2 секунди з оновленням даних в таблицях.
Як варіант подолання цього «глюку» роботу з БД переніс в сінгтон — в результаті всі данні завжди актуальні.

Інші класи треба було не полінуватись і винести в окрему бібліотеку

— проверка наличия сети через запрос гугла (это смешно)

Були такі причини:
1) Перевіряти наявність мережі перед запитом таки треба — певен погодитесь
2) Сам хост з АПІ ендпоінтом видавав щось незрорзуміле, тому я прописав Гугл
3) Використовувати pod (неодноразово бачив в чужому коді) для такої елеметарної задачі — соромно

— код в стиле [viewControllers objectAtIndex:self.navigationController.viewControllers.count — 2] — это пипец и вон из профессии

а тут будь ласка детальніше.
Ви вже другий хто на це звертає увагу.

Буду щиро вдячний за приклад правильного, на Ваш погляд солюшину повернення змінної на попередній екран.

Ну и банальное не умение читать ТЗ. Не сделан пункт 2), зато масса усилий на опциональный пункт 3) (который тоже сделан неправильно). Вот для меня это даже не звоночек, а колокол, что в работе будет так же. Делаем не то что надо, а то что умеем или хочется.

Після прочитання «ТЗ» у мене склалося враження, що людина яка його писала прийшла з веброзробки. Що і було підтверджено такою цитатоб з відповіді do it
«— проблему синхронізації можна легко вирішити використовуючи простий isSynchronized, після появи інтернет-з’єднання можна взяти всі об’єкти з бази даних, використовуючи фільтр isSynchronized і по черзі завантажити кожен об’єкт на сервер, змінюючи цей прапор на true після відповіді про успішну операцію від сервера.»
"
jobs.dou.ua/...​tware/reviews/38594/#5389

Отже, той хто ставив задачу не розуміє, що девайс може перезавантажитись або додаток буде вивантажено з пам"яті.

Тому для збереження даних я вирішив використати CoreData.

Далі.
Якщо Ви звернули увагу, то в моїй дата моделі є більше властивостей у об«екта Task

Вони були додані для викішення задачі синхронізації та видалення даних як локально так і на сервері.

Я допускав варіант коли у одного користувача може бути кілька пристроїв.

Використання даних з JSON, отриманого з серверу має суттево іншу логіку ніж робота з CoreData з наступною синхронізіцією даних <- це і є тією самою причиною, чому я спочатку зробив локальну БД, а інтеграцію вебсервісів залишив на потім.

Короче, «учиться, учиться, учиться» как говорил тот, чьё имя протзносить нельзя )

А тут робота така :)

Навіщо?

Зараз починається саме цікаве:
тролі видохлись.

Підтягнулися «критики», правда не можуть обгрунтувати, але це вже прогрес :)

Ви промахнулись :(

Так загадково написали «конструктивну критику» в навіть не показали як правильно.

Тьфу! Размазня!

було про.

Если бы я был ревьювером, то ревью закончилось бы на реализации ф-ции
-(BOOL)isConnected{..}
с фидбеком «лучше бы ему заняться физическим трудом на свежем воздухе». Ну про синглтон, который не синглон вообще молчу.

Если это уровень лидов сейчас, то ой.

А про повторне використання ячейок я знаю.

У мене таке використовується:
MyTaskTableViewCell *myTaskTableViewCell = [tblTasks dequeueReusableCellWithIdentifier:@"MyTaskTableViewCell"];

github.com/...​reenViewController.m#L153

Ви неуважно дивились в код :(

Вчора хіба не Ви обіцяти на всю гілку свій код ревью показати?
Чи вже передумали?

мне за Вас тестовое сделать? найдется ли у Вас время, его покритиковать?

Дякую, не треба.
Не на часі.

Гарно з"їхав.

Спочатку загадково написав зауваження, потів розповів про своє виховання, а тепер радить прислухатись до критики.

Мені багато розповідали про лідів таких яв Ви:
може тільки критикувати, і не показати як привильно.
Я думав, що брешуть.

А ні, виявилось таки є такі екземпляри.

Є над чим подумати.

Такої «критики» тут ціла гілка.

Всі знають, що все погано, але ніхто не може показати як правильно.

Пічалька :(

Так загадково написали «конструктивну критику» в навіть не показали як правильно.

Тьфу! Размазня!

Він у нас такий :)

Осмелюсь предположить, что ход Ваших и моих (а также части участников данной темы) не совпадает.

Це дуже прикро.
Де для Вас очевидно мені треба писати в документації.

А там де мені очевидно, там Вам не очевидно.

Парадокс?

дивно що розробник не «здогадався» © додати таку фічу :(

Нічого не нагадує?

Це ж очевидно!