Між іншим був сертифікованим спеціалістом по бухгалтерській підсистемі для вісімки :)
ага :)
На жаль я так ще і не побачив Ваших шедеврів програмування, але то таке.
[[NSArray alloc] init];
чим саме поганий такий спосіб?
Якщо Вам так зручніше, то хай буде так :)
А чому Вам соромно себе назвати?
Встидаєтесь власного імені?
Ціную.
Дякую!
Але після цього Ви вчора писали, що буде таки від Вас ревью.
Не переймайтесь.
І головне, що це перший коментар :)
Дякую містер Annonymous Annonymous .
Особливо за наведені приклади солюшинів, які Ви так смачно рознесли.
Чувак, ты понимаешь, что после этого треда найти работу станет в 100 раз сложнее? Разве что в «Рога и Копыта» куда программистов берут на вес? Доу читает куча народу и, поверь, этот тред сейчас активно обсуждается в куче закрытых чатиков и каналов (откуда я о нём и узнал). Ты теперь звезда украинской iOS разработки, поздравляю.
Якщо потім працювати у такого як Ви — то краще бути «реджектнутим» одразу.
Последняя попытка быть серьёзным. Тебе максимально корректно указали на кучу просто элементарных ошибок в коде, которые недопустимы даже для джуна. Дело совсем не в форматировании, а в том, что базовые вещи сделаны неправильно.
Нарешті Ваше виховання дозволило перейти до конкретніших речей.
Самое простое из того, что сразу бросается в глаза:
— код написан на Obj-C который был актуален в каменном веке, спасибо что хоть нет ручного управления памятью
Вже втомився повторювати, це було узгоджено з рекрутером. Я питався на чому писати, мені сказали Обджектів Сі теж треба знати :)
У мене є товариш у Нідерландах, дядькові 60 років, ще пише на Коболі.
— не используются реюзебл целс (это умеют джуны с2-3 месяцами опыта)
Це використовуються, вже відповідав іншому товаришу
github.com/...reenViewController.m#L153
— синглтон это плохо само по себе, это code smells, но в коде определён синглтон, который каждый раз создаётся явно, а это явная ошибка
50/50
У CoreData є такий прикол як запізнення на
Як варіант подолання цього «глюку» роботу з БД переніс в сінгтон — в результаті всі данні завжди актуальні.
Інші класи треба було не полінуватись і винести в окрему бібліотеку
— проверка наличия сети через запрос гугла (это смешно)
Були такі причини:
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
Ви неуважно дивились в код :(
Вчора хіба не Ви обіцяти на всю гілку свій код ревью показати?
Чи вже передумали?
мне за Вас тестовое сделать? найдется ли у Вас время, его покритиковать?
Дякую, не треба.
Не на часі.
Гарно з"їхав.
Спочатку загадково написав зауваження, потів розповів про своє виховання, а тепер радить прислухатись до критики.
Мені багато розповідали про лідів таких яв Ви:
може тільки критикувати, і не показати як привильно.
Я думав, що брешуть.
А ні, виявилось таки є такі екземпляри.
Є над чим подумати.
Такої «критики» тут ціла гілка.
Всі знають, що все погано, але ніхто не може показати як правильно.
Пічалька :(
Так загадково написали «конструктивну критику» в навіть не показали як правильно.
Тьфу! Размазня!
Він у нас такий :)
Осмелюсь предположить, что ход Ваших и моих (а также части участников данной темы) не совпадает.
Це дуже прикро.
Де для Вас очевидно мені треба писати в документації.
А там де мені очевидно, там Вам не очевидно.
Парадокс?
дивно що розробник не «здогадався» © додати таку фічу :(
Нічого не нагадує?
Це ж очевидно!
Такі як Ви теж мають вплив на тон дискусії.