シェフのレシピをリファクタリングする必要がありますか？私はdatabagから、Windowsのデスクトップアプリをインストールするレシピを書いた

Question

：

workstation_apps = data_bag_item('winapps','desktop_apps') 

for package in workstation_apps['apps'] do 
     chocolatey_package "#{package}" do 
     action :install 
     end 
end 

はfoodcriticを実行して、私はエラーを取得する：

FC002: Avoid string interpolation where not required: ./recipes/default.rb:23 

あなたが見ることができるように、私は{＃の周りに二重引用符を追加しました変数を展開してコメントとして機能しないようにします。

これを行うより良い方法はありますか？

Answer 1

2つのこと：Rubyで最初にforループを使用すると、eachループが優先されます。

workstation_apps = data_bag_item('winapps','desktop_apps') 
chocolatey_package workstation_apps['apps'] 

正確

Answer 2

"#{package}"をpackage.to_sに置き換えることができます。 package場合は、文字列オブジェクト、単にpackageである：それは明確なラインがブロックに属していることを確認するためにインデントaction ...ライン

workstation_apps = data_bag_item('winapps','desktop_apps') 

for package in workstation_apps['apps'] do 
     chocolatey_package package do 
       action :install 
     end 
end 

。