aylo May 25 2012 at 12:35

Пишем REST приложение на Sinatra и прикручиваем Redactor. Часть 2

5 min

2.8K

Comments 5

AlexeyK May 25 2012 at 16:49

ext = File.extname(filename)
…
if ext == ".doc" || ext == ".zip" || ext == ".dmg"

лучше заменить на что-то вроде

allowed_extensions = ['.doc', '.zip', '.dmg']
if allowed_extensions.include?(File.extname(filename))

…

end

и я не совсем понял, вот это

File.open(File.join(settings.files, filename), 'wb') {|f| f.write file.read } #загрузили — сохранили

file.read для чего в конце?

akzhan May 25 2012 at 18:40

вот так вроде лаконичнее

if ext =~ /\A\.(?:doc|zip|dmg)\z/

Хотя для этого не обязательно вытаскивать расширение, можно сравнить и так:

if filename =~ /\.(?:doc|zip|dmg)\z/

akzhan May 25 2012 at 18:43

Кстати, желательно при получении файла его расширение преобразовать в нижний регистр, как мне кажется.

aylo May 25 2012 at 18:08

Если вы объясните почему

allowed_extensions = ['.doc', '.zip', '.dmg']
if allowed_extensions.include?(File.extname(filename))

предпочтительнее, я вам буду очень благодарен.

Спасибо, действительно file.read не нужен в данном случае.

akzhan May 25 2012 at 18:45

маска для сравнения (регулярка ли, массив ли) действительно должна быть вынесена вверх, на уровень класса, причем в виде замороженной константы. Так удобнее соблюдать DRY и т.д.

Но это все уже — стиль кодирования, не более того.

Show the best of all time