ну код ревью не для этого
ну хоть что то понимаешь
сонар как айрат говорил он присобачил
Sergey
Roman
если такие вещи на ревью проверять — проще уж самому написать
Dr. Friedrich
Бля, и ведь мудакам даже примитивный линтер ИДЕИ подсказывает что тут NPE
Вот не надо тут, в Идее нормальный flow analysis, годный.
Stas
Как человек, который в прод только тесты и писал, осуждаю.
я тоже осуждаю, все про тесты говорят, а я ваще не шарю)
Stas
Вот не надо тут, в Идее нормальный flow analysis, годный.
так Айрат вроде хвалит идею , а ругает мудаков)
Ayrat
ну хоть что то понимаешь
сонар как айрат говорил он присобачил
на этот проект не присобачил. Там помойка
Ayrat
надо разобраться как в этот проект присобачить.
Там просто тысячи баш скриптов, руби, джава, скала. Монорепа по сути
Dr. Friedrich
надо разобраться как в этот проект присобачить.
Там просто тысячи баш скриптов, руби, джава, скала. Монорепа по сути
Сонар норм сбоку пристраивается, и вот в таком коде он бы нашёл проблему
Ilya
Он бы и ещё сотню проблем нашёл бы, подозреваю.
Ayrat
ооо, зацените. Это в одной репе
24536 text files.
19007 unique files.
7385 files ignored.
github.com/AlDanial/cloc v 1.86 T=221.35 s (78.5 files/s, 16124.5 lines/s)
---------------------------------------------------------------------------------------
Language files blank comment code
---------------------------------------------------------------------------------------
JSON 6824 122 0 806124
Go 1586 38696 56892 583517
JavaScript 1363 62343 112223 326616
Python 1667 52116 75004 227664
YAML 55 789 1809 173463
Ruby 969 33003 13429 165317
Java 1334 30560 67291 164971
XML 649 447 187 139563
C 389 12926 17284 66536
CSS 59 3285 432 39418
Scala 133 3890 658 30500
Bourne Shell 597 5556 3101 30013
ERB 272 4413 43 28910
TeX 68 3234 3300 22540
HTML 72 3025 28 19862
R 295 2977 2193 14376
Markdown 176 4152 0 10855
Maven 84 658 147 7629
C/C++ Header 125 3099 7113 7499
XSD 19 272 75 7443
Haml 252 1020 430 7081
SQL 33 368 356 4557
C++ 16 754 691 4099
make 113 1355 1655 4051
Assembly 12 61 86 3658
reStructuredText 13 2411 3279 3624
SVG 6 0 1 2031
Protocol Buffers 22 536 1764 1879
Bourne Again Shell 50 325 241 1142
Sass 8 91 38 1066
Fortran 77 3 338 1351 777
PHP 5 133 473 698
Cython 12 186 236 682
Jupyter Notebook 6 0 11024 621
Lisp 4 115 149 608
Perl 1 87 170 582
Windows Module Definition 28 2 0 581
Handlebars 19 20 8 342
Ruby HTML 6 23 0 207
INI 11 0 0 184
vim script 3 27 69 149
Ayrat
LESS 1 3 11 108
Pascal 1 0 370 98
Mathematica 1 28 0 65
Visualforce Component 5 0 0 49
Fish Shell 1 13 15 47
Dockerfile 2 12 12 40
Thrift 3 11 2 29
C Shell 1 9 7 21
JSX 1 4 0 19
awk 2 0 0 2
MATLAB 1 1 147 1
---------------------------------------------------------------------------------------
SUM: 17378 273496 383794 2911914
---------------------------------------------------------------------------------------
Stas
паскаль?
Ayrat
У нас тут всё!
Ilya
Фортран.
Ayrat
ну короче, это помойка.
Ilya
Господи, два фортрана.
Ayrat
Дата сатанисты выживали как могли!!1
Ayrat
То есть люди готовы писать на чём угодно лишь бы не на продуктах MS почему-то
Ilya
И VB не видно(
Aleksandr
А windows module definition это что простите?
Ayrat
я хз. я зашёл в репу и сделал
cloc .
Ayrat
вы ж не думаете что я знаком со всеми 3М строк кода там?
Ilya
Второй раз за день вижу reStructuredText. После третьего раза погуглю.
Ayrat
я бы 90% сразу сжёг бы, конечно.
Stas
почему там столько YAML ?
Stas
шутки про Senior YAML Developer перестают быть шутками
Ilya
У нас доброволец!
Stas
бгг
Roman
то с чем работал конечно!!11
но про то, с чем работал, ты наверняка знаешь, что оно говно. А про остальное еще есть призрачная надежда
Ayrat
но про то, с чем работал, ты наверняка знаешь, что оно говно. А про остальное еще есть призрачная надежда
статистика показывает что там везде говно
Ayrat
вот прям 10 из 10
Roman
так это
Roman
поджигай
Aleksandr
Просто там походу кодген
Roman
если ты грохнешь эту репу, и потом оповестишь всех об этом — будет инсталляция. А если сначала оповестишь, а потом грохнешь — перформанс
Viacheslav
Aleksandr
Не сразу заметил, но это гошный проект
Ilya
Джсонный проект.
Aleksandr
Да, так точнее
Анна
О, у нас тут завершились дебаты по поводу моих комментов в ревью. Итог: код настолько лапша, что прямощас отрефакторить сложно, так что не будем 🙈
Анна
Vladimir
О, у нас тут завершились дебаты по поводу моих комментов в ревью. Итог: код настолько лапша, что прямощас отрефакторить сложно, так что не будем 🙈
объявляйте банкротство по тех. долгу
Ayrat
хорошие ревью и конкурсы интересные
Анна
жоп сейфти
Анна
не, когда-то я уволилась после того, как полгода учила корейцев декомпозиции блин кода. Но оно опять
Aleksandr
рефакторить нахер, нет ничего хуже лапши
Aleksandr
код свежий?
Aleksandr
и уже сложно?
Анна
А, ещё сегодня перл был. Вкатили хоть немного, но ассерты. Ассерт упал в релизном билде, там где баг раньше маскировался. На дейли чел, который пофиксил баг, рассказывает это историю с явным намёком на то, что это всё ассерты виноваты 🙈
Анна
ААААААА
Анна
типа "раньше нипадало! это ассерты всё!"
Ilya
Ну так надо удалить их из тестов, да и дело с концом.
Roman
О, у нас тут завершились дебаты по поводу моих комментов в ревью. Итог: код настолько лапша, что прямощас отрефакторить сложно, так что не будем 🙈
о, знакомо. На одном проекте как-то обсуждали большую фичу, выявили слабости предложенной архитектуры, но девчонка все равно сделала по-своему. Когда я попытался завернуть ПР, сказали, что "ну да, но щас не будем ничего менять"
Ilya
Слишком радикально. Зачем эти лишние действия? Лучше удалять по необходимости.
Ilya
Анна
так вам еще тестировщиков надо уволить
Ну тот баг на поведение действительно не влиял, по крайней мере до поры до времени
Roman
все равно надо скейлить команду под ожидаемые проблемы заранее
Aleksandr
Roman
Ожидается больше багов, баги находят тестеровщики. Сокращаете тестировщиков — не имеете больше багов
Vladislav