search

rambler-ios / team

sweet home
177 stars 40 forks source link

Анализируй это! #101

Closed Brain89 closed 7 years ago

Brain89 commented 7 years ago

Доклад посвящён наиболее популярным статическим анализаторам кода для iOS. Будет рассказано, как ими пользоваться, какие проблемы они решают, как внедрить их в ваш привычный цикл разработки, как писать для них свои правила. Плюс к этому поделимся своим опытом интеграции статического анализа и CI. После доклада слушатели, ещё не успевшие плотно поработать со статическими анализаторами кода, смогут быстро к этому приступить.

etolstoy commented 7 years ago

наиболее популярным статическим анализаторам

про SwiftLint будет?

serkrapiv commented 7 years ago

нет

2017-05-11 18:43 GMT+03:00 Egor Tolstoy notifications@github.com:

наиболее популярным статическим анализаторам

про SwiftLint будет?

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/rambler-ios/team/issues/101#issuecomment-300830706, or mute the thread https://github.com/notifications/unsubscribe-auth/ABCUBWd2SCwNuVO1Y-_1ZM9Cm7_a_KS7ks5r4yybgaJpZM4NYEMT .

aknew commented 7 years ago

Добрый день. Подскажите, будет ли в докладе рассмотрен вопрос взаимодействия анализаторов (конкретно пункта analyze он же clang-analyzer, но и про другие интересно) c Typhoon и DI в целом? Похоже, что clang-analyzer попросту не находит связи между классами из-за DI и у него не получается их анализировать, в целом он так же ведет себя если вообще не присваивать объекты и только дергать у него методы/свойства. С уважением. ПС хотел накидать простенький пример для иллюстрации, но столкнулся с тем что Xcode перестал ходить внутрь других фаилов при анализе вообще, как с этим бороться не знаю пока, а без этого пример не очень-то информативен.

serkrapiv commented 7 years ago

Интересно, пока рассматривать не планировали. Было бы здорово посмотреть на пример если будет возможность

On 16 May 2017, at 16:35, Alexander Kazansky notifications@github.com wrote:

Добрый день. Подскажите, будет ли в докладе рассмотрен вопрос взаимодействия анализаторов (конкретно пункта analyze он же clang-analyzer, но и про другие интересно) c Typhoon и DI в целом? Похоже, что clang-analyzer попросту не находит связи между классами из-за DI и у него не получается их анализировать, в целом он так же ведет себя если вообще не присваивать объекты и только дергать у него методы/свойства. С уважением. ПС хотел накидать простенький пример для иллюстрации, но столкнулся с тем что Xcode перестал ходить внутрь других фаилов при анализе вообще, как с этим бороться не знаю пока, а без этого пример не очень-то информативен.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub, or mute the thread.

aknew commented 7 years ago

Жаль что этот момент не рассматривается Пример выложил на гуглдрайв , там готовый проект с подключенным тайфуном и куча вариантов вызова одной и той же функции с по-разному задекламированным/присвоенным свойством (как именно задается через #define CASE N в хедере) чтоб посмотреть поймает ли анализатор деление на нуль, что я получаю в том или ином случае там тоже есть (и это меня смущает - я практически уверен что полгода назад в подобных ситуациях поведение было другим)

serkrapiv commented 7 years ago

Спасибо, покопался сам ещё. Получается даже при ручном задании зависимостей снаружи анализатор не видит ошибки. Нужно либо на месте классы создавать, либо определения классов держать в том же файле. И так и так тлен какой-то.

Brain89 commented 7 years ago

Слайды - https://www.slideshare.net/Rambler-iOS/ss-76457847 Видео - https://youtu.be/v_0M0fUUrGw