Не? ;)

Это сообщение отредактировал(а) spyswamp - 21.7.2011, 11:12

Не-а... С точки зрения компилятора это одно и то же. А с точки зрения читабельности кода - это не айс. То же самое и относится к анонимным классам и проч. усложняющим чтение кода вещам.

Короткий "if" хорошо использовать в простых конструкциях типа:


В других случаях это ни к чему хорошему не ведет   

Это сообщение отредактировал(а) ivanovpv - 21.7.2011, 16:28

Простите, конечно, но читабельность ухудшает лишь ваше бездумное использование final'а. ;) А так же int index = 0; lri.size() - 1; а вот менять сообщение после указаний на недочеты - это уже плохой тон.  Атата! В остальном опытному глазу гораздо проще увидеть суть тернарного оператора (а этот оператор называется именно так, а не "короткий if", еще один камень вслед за final), нежели if со вложенностью, скобки, очевидные комментарии и дополнительный return в конце.

И прошу не относиться к маленькому ревью как к наезду. Просто указал на недочеты.

оффтоп: В качестве отличного примера кодирования рекомендую изучать исходники Half Life перед сном. 

Да ладно ужо наехал... В качестве оправдания (а придется оправдываться) - скажу что исходник выдран из довольно большого класса. В методе еще есть туча параметров, которые убирал когда постировал - так что мог и ошибитьс выкашивая ненужные куски кода... Если бы я выложил весь код который раза в 4 длиннее - то ничего не было бы понятно. А наезжать все не дураки и слова правильные тоже знаем - тока вот думается форум не для того, чтобы выпячивать себя - а чтобы обменивать опытом. В StackOverflow вас давно бы уже изничтожили.

Ваша метода в общем то ясна - нас**ть на ближнего... Я хотел помочь, в итоге только получил по шарам. Ну да ладно - попишите парочку прожектов под сто тыщь строк, потом может меня и поймете. Так шта давай - с флагом и барабаном вперед - пральной дарогой идете.

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

Добавлено через 4 минуты и 47 секунд
ivanovpv, а если мало "спасибо" выше, то плюсану потом, когда 100 сообщений наберу.   

Это сообщение отредактировал(а) spyswamp - 22.7.2011, 10:03

Какой продакшн? О чем вы говорите уважаемый? Вы что клиент у меня принимать работу или босс? Код был выложен не на предмет обсасывания на оптимизацию и проч. схоластику, а просто чтобы указать на то как можно улучшить (и довольно просто) ваш же и код.  Повторяю код выдранный из контекста не прогнанный ни через компилятор или упаси боже профайлер, добавлены были комментарии - чтобы вам же и было удобно.

Далее начался откровенный   - собсно говоря вот и все. Ваши комментарии касаемо ста тыщь строк кода также как и про "новичковость", умничания по тернарные операции и проч. сами же вас и характеризуют, что я и выразил своей оценкой. Я в отличие от вас никого не поучаю, хотя наверное с опытом Java разработок с 95-го года и с количеством подчиненных в 100 с гаком наверное имею на это право   

За сим кланяюсь - тема закрыта. Все щасливы и разбегаются по своим углам "разрабатывать крупные корпоративные прожекты на мульон строк".

ivanovpv, по-моему, ты тролль, а не начальник и джавист с 95 года. Ты даже элементарных вещей не знаешь.

Добавлено через 1 минуту и 44 секунды
Признайся, что лоханулся на элементарщине, не знаешь где правильно применять final и пытаешься лечить.  Ну а минус я тебе верну, не переживай.

spyswamp, зачем тут цирк устраиваете. Я, например, вообще Джавы не знаю, но это не мешает мне писать проги под моего планшета и мобилу. Нахожу исходники и правлю под себя или в книжке примеры смотрю... Что не понимаю, спрашиваю на форуме. И всё нормально работает...   

Это сообщение отредактировал(а) CaptainNemo - 23.7.2011, 22:21

Не знаю чего ты взъелся то? Применение final оправданно в данной ситуации поскольку возвращается значение типа ResolveInfo то есть информация об интенте обрабатывающем данный тип фильтра. Очевидно, что у автора нет желания изменять его значение о чем он и сообщает компилятору применением модификатора final.  При этом достигаются 2 цели: первых в дальнейшем если ResolveInfo случайно где-то будет модифицироваться то компилятор ругнется, во вторых компилятор информацию может использовать для своих оптимизаций. 

Так что ты неправ.

Bearness, RTFM.