Comments 5
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 для чего в конце?
…
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 для чего в конце?
0
Если вы объясните почему
предпочтительнее, я вам буду очень благодарен.
Спасибо, действительно file.read не нужен в данном случае.
allowed_extensions = ['.doc', '.zip', '.dmg']
if allowed_extensions.include?(File.extname(filename))
предпочтительнее, я вам буду очень благодарен.
Спасибо, действительно file.read не нужен в данном случае.
0
Sign up to leave a comment.
Пишем REST приложение на Sinatra и прикручиваем Redactor. Часть 2