что то есть еще?

isHideRatingButton - хреновое название метода, который что-то меняет

что то есть еще?

выше писали и не только я. Лень же рефакторить тем более я не за маком

Вот так. if let можно сделать все в одном но мне больше нравится так.

И isHideRatingButton лучше сделать методом RatingTableViewCell

Вот так. if let можно сделать все в одном но мне больше нравится так.

вот я и говорю что очень много if очень мношго

У меня там опечатка в коде в else

вот я и говорю что очень много if очень мношго

Дело в том, что с твоим форс анврапингом приложение упадет

У меня там опечатка в коде в else

если потом то то добавлять то это будет капец

Золотое правило: приложение должно падать только если оно сломалось непоправимо. А так если тебе случайно апи выдал говно вместо ответа ты упадешь, а не выведешь алерт.

Дело в том, что с твоим форс анврапингом приложение упадет

у меня перед форсом 2 проверки

форс рекомендуют делать только после проверок

у меня перед форсом 2 проверки

Так а зачем проверять во много строк если можно if let

Который сам проверяет

форс рекомендуют делать только после проверок

где?

И не надо делать isEmpty и = nil разными сущностями.

где?

Гарды по ходу.

просто некоторые почему то совсем буквально воспринимают писать мало кода а то что его прочитать невозможно это ладно?

https://gist.github.com/super-ultra/43ed765a4718f2872a5c7552452ba91b

Можно еще и без else если сделать что-то типа isHideRatingButton(booles: Competency.competency(by: getCompetency) == true, cell: myCell)

тут задача не написать все в 2 строки а что б был понятный код

просто некоторые почему то совсем буквально воспринимают писать мало кода а то что его прочитать невозможно это ладно?

Код должен быть лаконичный и понятный, а тут мне все понятно, это стандартные if let'ы

просто некоторые почему то совсем буквально воспринимают писать мало кода а то что его прочитать невозможно это ладно?

твой код очень тежело понять

У меня друг как-то скидывал сниппет: ` if !s.saved { blahblah } else if s.saved { foobarbaz } `

а много вложеных if это правильно?

твой код очень тежело понять

+

У тебя внешнее скролвью куда ты пытаешься запихнуть таблицу?

хочу скрины вашего кода посмотреть

а много вложеных if это правильно?

Да, потому что у тебя много условий. И можно ведь сделать один иф и проверять им все условия сразу, но по мне, это как кому нравится.

У меня друг как-то скидывал сниппет: ` if !s.saved { blahblah } else if s.saved { foobarbaz } `

Проиграл

У меня друг как-то скидывал сниппет: ` if !s.saved { blahblah } else if s.saved { foobarbaz } `

Это у дяди Боба такое даже

У меня друг как-то скидывал сниппет: ` if !s.saved { blahblah } else if s.saved { foobarbaz } `

Понятно - да. Работает - еще бы. Говно - неповторимое.

Да, потому что у тебя много условий. И можно ведь сделать один иф и проверять им все условия сразу, но по мне, это как кому нравится.

вот я согласен что дело вкуса и не более того

И это мидол с окладом 140к такое писал.

а много вложеных if это правильно?

сделай guard. Проблема же не в том

хочу скрины вашего кода посмотреть

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

тут даже не понятно, с чего начать объяснение того, почему код плохой, и с чего нужно начать, чтобы его улучшить

и вы все синьеры?

потому что есть подозрение, что проблемы в каких-то фундаментальных вещах

и вы все синьеры?

вряд ли

Нет конечно

и вы все синьеры?

Не обязательно быть синьором чтобы увидеть косяки в выложенном тобой коде

Все условия проверяются по порядку и по логической связи

[weak self = self] это зачем?

У меня друг как-то скидывал сниппет: ` if !s.saved { blahblah } else if s.saved { foobarbaz } `

if s.saved == true { } else if s.saved != true { } Я заимпрувил еще круче

чем отличается от [weak self]?

Все условия проверяются по порядку и по логической связи

что то сложнее есть?

чем отличается от [weak self]?

Не знаю, я где-то увидел weak self = self и так и писал

что то сложнее есть?

чего ты хочешь этим добиться? это как-то поможет твоему коду?

чего ты хочешь этим добиться? это как-то поможет твоему коду?

хочу посмотреть как другие пишут(не тролить)

Не знаю, я где-то увидел weak self = self и так и писал

вроде в старых версиях свифта нельзя было писать [weak self]

Как правильно делать запросы на поиск когда печатается текст в uitextfield?

что то сложнее есть?

здесь должно быть посложнее https://github.com/kickstarter/ios-oss

хочу посмотреть как другие пишут(не тролить)

есть же эпловый гитхаб, они там и статьи выкладывают

есть же эпловый гитхаб, они там и статьи выкладывают

так я же с вами общаюсь

Но я не сениор( Увы

так я же с вами общаюсь

я бы начал не с кода в твоем случае

я бы начал не с кода в твоем случае

))

что то сложнее есть?

Это одна из самых сложных функций в моем коде. Вся функциональность должна быть поделена на маленькие понятные кусочки. Изоляция модулей в слои, абстракции, чтобы можно было поменять метод не меняя пол-проекта.

ПАТТЕРНЫ ПРОЕКТИРОВАНИЯ ОБЪЕКТО ОРИЕНТИРОВАННОГО СОФТА.

и это как раз не связано с языком программирования

ПАТТЕРНЫ ПРОЕКТИРОВАНИЯ ОБЪЕКТО ОРИЕНТИРОВАННОГО СОФТА.

сколько лет пишешь на ios

и на какой позиции работаешь?

ERROR: S client not available

сколько лет пишешь на ios

Ээээ... три месяца. Фрилансер.

Как правильно делать запросы на поиск когда печатается текст в uitextfield?

рребята

[weak self = self] это зачем?

По-моему тут даже "зачем вообще weak" https://stackoverflow.com/questions/34931172/should-self-be-captured-as-strong-in-a-uialertactions-handler

рребята

Ща гляну.

По-моему тут даже "зачем вообще weak" https://stackoverflow.com/questions/34931172/should-self-be-captured-as-strong-in-a-uialertactions-handler

Я его пихаю везде – так тупо проще не заморачиваться с тем где могут произойти стронг референсы.

и это как раз не связано с языком программирования

а у тебя опыт какой ?

и на какой позиции работаешь?

сначала просишь помочь, а потом докапываешься, какой у кого опыт

Я его пихаю везде – так тупо проще не заморачиваться с тем где могут произойти стронг референсы.

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

сначала просишь помочь, а потом докапываешься, какой у кого опыт

да почему везде докапываюсь?

а у тебя опыт какой ?

Но я начинал на джаве :) Это забило в меня ООП навсегда, лол.

Хотя перестраховка тоже хорошо

сначала просишь помочь, а потом докапываешься, какой у кого опыт

как будто это поможет твоим знаниям в программировании и твоей безграмотности

Я таки в упор не понимаю, в чем принципиальная ценность лычки "senior", например.

да почему везде докапываюсь?

Потому что это так выглядит?

Потому что это так выглядит?

вот я и говорю что не так

мы же просто общаемся

А как разрулить это?

а когда опыт спрашиваю думаете оскорбляю

а когда опыт спрашиваю думаете оскорбляю

это просто не по делу

это просто не по делу

ну некоторые же отвечают значит все норм

советы раздаете хочу понимать

я же спокойно общаюсь

У тебя внешнее скролвью куда ты пытаешься запихнуть таблицу?

да

да

ну так это и есть скролвью внутри скролвью

я же спокойно общаюсь

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

ну так это и есть скролвью внутри скролвью

StackView вместо таблицы будет сносным решением?

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

И познай паттерны дизайна ООП, они реально пригодятся. Работай с объектами, а не со статическими функциями.

StackView вместо таблицы будет сносным решением?

скрола же не будет. Просто длинный список всех.

StackView вместо таблицы будет сносным решением?

Тебе не вместо таблицы его надо, тебе надо таблицу, а в ее ячейках - StackView

скрола же не будет. Просто длинный список всех.

так и нижно

И познай паттерны дизайна ООП, они реально пригодятся. Работай с объектами, а не со статическими функциями.

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