псы в рапиде » from archive
@ati, что, на твой взгляд, правильнее: { IN => [ @$hotel_ids ] } или { IN => \@$hotel_ids }
/* кстати, а code review у вас есть? */ ‎- 9000
да, я им занимаюсь прямо сейчас, очевидно ;) ‎- псы в рапиде
Меня не спрашивают, но я бы написал { 'IN' => [ @{$hotel_ids} ] }. Я не люблю как bare words так и опускание {} в разыменовании ссылок. ‎- бородат и духовно хвостат
шутка в том, что оба варианта эквивалентны { IN => $hotel_ids }, только без лишней операции копирования массива :) ‎- псы в рапиде
@squadette: неочевидно: такой вопрос можно задать и о своём коде (а ревьюить, очевидно, имеет смысл чужой). ‎- 9000
@9000, я адресовал этот вопрос именно @ati, потому что он actually knows Perl. ‎- псы в рапиде
@squadette Стоооооп. Нигде не сказано, что его не надо копировать же. Этого из твоего вопроса не очевидно, а я могу сходу придумать 100500 ситуаций когда нужна именно копия, что бы не попортили. Т.е. ты спросил не то, на что ответом является «копирование тут не нужно». ‎- бородат и духовно хвостат
@blacklion ахахаха, Hamming НЕГОДУЭ, между прочим. ‎- псы в рапиде
Я, собственно, и выбираю вариант с [ ] потому что это подчёркивает копирование, тогда как \@ его заметает под ковёр :) ‎- бородат и духовно хвостат
ну да, окей, я забыл, с кем имею дело. На самом деле конечно в этом месте (это обращение к ORM'ке) — список айдишников read-only. ‎- псы в рапиде
Ну тогда, да, ОБА ХУЖЕ, несомненно :) ‎- бородат и духовно хвостат

2015-2016 Mokum.place