Max
там где ответственные сотрудники,их там может быть 1,или 50
Zooey
Скорее всего, тут нужно пересмотреть решение. isScrollEnabled использовать не нужно. Попробуй использовать одну таблица с разными секциями
Anonymous
если таблица длинная и занимает много места
та даже непонятно кто из карртинки должен скролится, а кто нет
Евгений З.
про джуна я к слову сказал и не хотел никого обидеть. Просто слишком много текста и ничего по делу. Плюс к тому вчера еще была реклама курсов. Но у меня не сложилось впечатление, что ты на том этапе, когда можешь кого-то учить
я не расстраиваюсь и сам не хочу кому то хамить.А на счет курсов думаю хорошее дело делаю и бесплатно.А у тебя если не сложилось мнение скажи почему? Поправь где я ошибся и я буду благодарен
Евгений З.
мне тут вчера писали что конкурентов выращиваю курсами)
Евгений З.
это же смешно)
Max
Zooey
я не расстраиваюсь и сам не хочу кому то хамить.А на счет курсов думаю хорошее дело делаю и бесплатно.А у тебя если не сложилось мнение скажи почему? Поправь где я ошибся и я буду благодарен
судя по скриншоту с кодом, который ты выложил, можно сказать, что уровень далек от высокого
Евгений З.
Zooey
в этом нет ничего плохого, есть куда стремиться и что изучать
Max
сама вью скролится,а списки нет
Zooey
скинь еще раз скриншот
Евгений З.
а то я так тоже могу то не нравиться и это
Евгений З.
в группе этой же помогают а не говорят ты гавно и досвиданье))
Alex
Индентация блока else
Anonymous
Евгений З.
тем более я видел синьеров и работал с ними мне очень далеко до них но все мы учимся же
Alex
И код ведь повторяется в блоке else
Anonymous
форс анврапы, в guard можно через запятую же
Dmitriy
Оценка компитенции же
Max
UI чуть менее чем странный. Оформи все в тэйблвью, а те ячейки, где может быть много текста, сделай раскрывающимися
мне надо сделать как тут,без раскрывающихся, у меня не так много опыта со свифтом и ios,поэтому пытаюсь сделать что приходит в голову
Zooey
тут, наверное, нет смысла обсуждать, что с отступами проблемы. Я бы рекомендовал просто зайти на swift.org или на гитхаб apple, и просто писать, как пишут там
Anonymous
Строки явно передаются
Евгений З.
давай те так
Евгений З.
принимаю критику такую
Zooey
давайте
Евгений З.
вот здесь я бы изменил так потому что
Евгений З.
код не идеальный согласен
Евгений З.
расскажите ка лучше скажу спасиюо
Anonymous
мне надо сделать как тут,без раскрывающихся, у меня не так много опыта со свифтом и ios,поэтому пытаюсь сделать что приходит в голову
делать скрол внутри скрола - плохая идея (представь себя на месте пользователя). Отображай либо весь текст в ячейке либо продумай вариант с горизонтальным скролом для таких ячеек
Max
делать скрол внутри скрола - плохая идея (представь себя на месте пользователя). Отображай либо весь текст в ячейке либо продумай вариант с горизонтальным скролом для таких ячеек
да там не должно быть скрола внутри скрола,мне просто надо отобразить всех сотрудников,для этого использую таблицу,делаю её такого размера чтобы она отображала все элементы,но она все равно реагирует на прокрутку,а мне как раз этого и не нужно
Max
нужно чтобы скроллвью крутилась
Stanislav
Anonymous
вот здесь я бы изменил так потому что
1) guard для ячейки ты можешь вынести там где ты ее объявляешь заодно сделать безопасную проверку на приведение к твоему типу, а не проверять дважды еще и с форсом
Anonymous
да там не должно быть скрола внутри скрола,мне просто надо отобразить всех сотрудников,для этого использую таблицу,делаю её такого размера чтобы она отображала все элементы,но она все равно реагирует на прокрутку,а мне как раз этого и не нужно
сделай скрин контроллера в IB. Что-то у меня подозрение, что ты не так что-то делаешь
Евгений З.
Alex
Alex
Вот так. if let можно сделать все в одном но мне больше нравится так.
Alex
И isHideRatingButton лучше сделать методом RatingTableViewCell
Евгений З.
Вот так. if let можно сделать все в одном но мне больше нравится так.
вот я и говорю что очень много if очень мношго
Alex
У меня там опечатка в коде в else
Alex
вот я и говорю что очень много if очень мношго
Дело в том, что с твоим форс анврапингом приложение упадет
Alex
Золотое правило: приложение должно падать только если оно сломалось непоправимо. А так если тебе случайно апи выдал говно вместо ответа ты упадешь, а не выведешь алерт.
Евгений З.
Дело в том, что с твоим форс анврапингом приложение упадет
у меня перед форсом 2 проверки
Евгений З.
форс рекомендуют делать только после проверок
Alex
Который сам проверяет
Alex
И не надо делать isEmpty и = nil разными сущностями.
Евгений З.
просто некоторые почему то совсем буквально воспринимают писать мало кода а то что его прочитать невозможно это ладно?
Zooey
https://gist.github.com/super-ultra/43ed765a4718f2872a5c7552452ba91b
Anonymous
Можно еще и без else если сделать что-то типа isHideRatingButton(booles: Competency.competency(by: getCompetency) == true, cell: myCell)
Евгений З.
тут задача не написать все в 2 строки а что б был понятный код
Alex
просто некоторые почему то совсем буквально воспринимают писать мало кода а то что его прочитать невозможно это ладно?
Код должен быть лаконичный и понятный, а тут мне все понятно, это стандартные if let'ы
Zooey
Alex
У меня друг как-то скидывал сниппет:
`
if !s.saved {
blahblah
} else if s.saved {
foobarbaz
}
`
Евгений З.
а много вложеных if это правильно?
Anonymous
Евгений З.
хочу скрины вашего кода посмотреть
Alex
а много вложеных if это правильно?
Да, потому что у тебя много условий. И можно ведь сделать один иф и проверять им все условия сразу, но по мне, это как кому нравится.
Anton
Anonymous
У меня друг как-то скидывал сниппет:
`
if !s.saved {
blahblah
} else if s.saved {
foobarbaz
}
`
Это у дяди Боба такое даже
Alex
У меня друг как-то скидывал сниппет:
`
if !s.saved {
blahblah
} else if s.saved {
foobarbaz
}
`
Понятно - да. Работает - еще бы. Говно - неповторимое.
Евгений З.
Да, потому что у тебя много условий. И можно ведь сделать один иф и проверять им все условия сразу, но по мне, это как кому нравится.
вот я согласен что дело вкуса и не более того
Alex
И это мидол с окладом 140к такое писал.
Zooey
хочу скрины вашего кода посмотреть
я скинул гист, как можно сделать хоть чуть чуть понятнее. Но там проблему нужно решать на уровне выше
Zooey
тут даже не понятно, с чего начать объяснение того, почему код плохой, и с чего нужно начать, чтобы его улучшить
Евгений З.
и вы все синьеры?
Zooey
потому что есть подозрение, что проблемы в каких-то фундаментальных вещах