User avatar
» from archive

@ati, что, на твой взгляд, правильнее: { IN => [ @$hotel_ids ] } или { IN => \@$hotel_ids }

Comment

/* кстати, а code review у вас есть? */

 ‎· 9000
Comment

да, я им занимаюсь прямо сейчас, очевидно ;)

 ‎· псы в рапиде
Comment

Меня не спрашивают, но я бы написал { 'IN' => [ @{$hotel_ids} ] }. Я не люблю как bare words так и опускание {} в разыменовании ссылок.

 ‎· бородат и духовно хвостат
Comment

шутка в том, что оба варианта эквивалентны { IN => $hotel_ids }, только без лишней операции копирования массива :)

 ‎· псы в рапиде
Comment

@squadette: неочевидно: такой вопрос можно задать и о своём коде (а ревьюить, очевидно, имеет смысл чужой).

 ‎· 9000
Comment

@9000, я адресовал этот вопрос именно @ati, потому что он actually knows Perl.

 ‎· псы в рапиде
Comment

@squadette Стоооооп. Нигде не сказано, что его не надо копировать же. Этого из твоего вопроса не очевидно, а я могу сходу придумать 100500 ситуаций когда нужна именно копия, что бы не попортили. Т.е. ты спросил не то, на что ответом является «копирование тут не нужно».

 ‎· бородат и духовно хвостат
Comment

@blacklion ахахаха, Hamming НЕГОДУЭ, между прочим.

 ‎· псы в рапиде
Comment

Я, собственно, и выбираю вариант с [ ] потому что это подчёркивает копирование, тогда как \@ его заметает под ковёр :)

 ‎· бородат и духовно хвостат
Comment

ну да, окей, я забыл, с кем имею дело. На самом деле конечно в этом месте (это обращение к ORM'ке) — список айдишников read-only.

 ‎· псы в рапиде
Comment

Ну тогда, да, ОБА ХУЖЕ, несомненно :)

 ‎· бородат и духовно хвостат

1 2 3 4 5 6 7 8 9 10