псы в рапиде » 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. ‎· псы в рапиде
Ну тогда, да, ОБА ХУЖЕ, несомненно :) ‎· бородат и духовно хвостат