alternative à court de ternaire et de sauvetage sous forme de modification

Question

Lorsque le chargement des fichiers, je l'habitude de faire une variante de:

first_file = "#{__dir__}/first_file.txt" 
first_data = File.exist?(first_file) ? File.readlines(first_file) : [] 

second_file = "#{__dir__}/second_file.yaml" 
second_data = File.exist?(second_file) ? YAML.load_file(second_file) : {} 

qui a toujours assis un peu de mal avec moi. Ayant besoin de la vérification pour voir si le fichier existe est quelque chose que a besoin de pour être là, mais ce qui m'importe vraiment est ce qui vient après.

Finalement, j'ai découvert en ligne rescue, et a commencé à le faire comme ceci:

first_file = "#{__dir__}/first_file.txt" 
first_data = File.readlines(firs_file) rescue [] 

second_file = "#{__dir__}/second_file.yaml" 
second_data = YAML.load_file(second_file) rescue {} 

Ce me est beaucoup plus claire - essayer de cette chose que vous voulez vraiment, et si vous ne pouvez pas faire autre chose que est exactement le repli que vous voulez. C'est à la fois plus court et plus clair que le ternaire.

Mais partout où je regarde le même conseil à avoid rescue in modifier form est répété. Je comprends pourquoi: cela peut cacher des erreurs que nous ne prévoyions pas.

Si je pouvais définir Errno::ENOENT comme l'erreur à rescue à partir de la forme en ligne, je le ferais. Mais autant que je peux dire, en ligne rescue ne peut pas spécifier de quelle erreur à sauver.

Alors, quelle est l'alternative? Que puis-je faire qui est aussi court et clair que le rescue en ligne, mais sans l'inconvénient?

Answer 1

Si j'étais vraiment préoccupé par la lisibilité de ces lignes, je voudrais créer quelques méthodes d'aide.

def read_with_fallback(filename, fallback) 
    File.exist?(filename) ? yield(File.open(filename)) : fallback 
end 

def safe_read_lines(filename) 
    read_with_fallback(first_file, []) {|file| file.readlines } 
end 

def safe_read_yaml(filename) 
    read_with_fallback(second_file, {}) {|file| YAML.parse(file.read) } 
end 

first_file = "#{__dir__}/first_file.txt" 
first_data = safe_read_lines(first_file) 

second_file = "#{__dir__}/second_file.yaml" 
second_data = safe_read_yaml(second_file) 

Mais si je devais seulement un ou deux de ceux-ci et ne pas les répéter dans toutes les autres méthodes, je ne voudrais pas déranger. Votre première forme est "assez bonne".