リモート/ローカルサーバーのファイル操作メソッドをRubyでより効率的に書く方法

Question

私は、サーバ上に存在するファイルに対していくつかのファイル操作を行うメソッドを記述しようとしています。たとえば、ソースからターゲットへのコピー、ファイルのシンボリックリンク、ファイルの読み込みなどです。ここでは、2つのシナリオ、つまりサーバーをローカルまたはリモートにする必要があります。

リモートサーバーのシナリオでは、（net/sshを使用して）サーバーに接続してから、execメソッドを使用してシェルコマンドを実行しています。ローカルサーバーでは、私はRuby FileUtilsメソッドを使用している接続を確立する必要はないので。あなたの参照のためにコードの部分を貼り付けました。

誰かがこの方法をより効率的に書くよう提案できるかどうかを確認したいと考えました。

def create_link 
begin 
    if self.particular_file_exists? 
    if !@ssh.local 
     @ssh.exec!("ln -s file1 file1-dump") 
    else 
     FileUtils.ln_sf("file1", "file1-dump") 
    end 
    end 
    rescue => e 
    $LOG.log(2, "Error occurred") 
    end 
end 

Answer 1

だけで簡単に編集：

def create_link 
    return unless self.particular_file_exists? 
    if @ssh.local 
    FileUtils.ln_sf "file1", "file1-dump" 
    else 
    @ssh.exec! "ln -s file1 file1-dump" 
    end 
rescue 
    $LOG.warn "Error occurred" 
end 

深いアイデアのためにあなたは何このコードについてhttps://codereview.stackexchange.com/

Answer 2

私は、パフォーマンスの最適化について言うが、ここで私は、拡張/より保守コードと思われるものですができません。あなたは、すべての罪深い方法でそれらのローカル/リモート支店のすべてを必要としません。代わりに、環境固有の動作を独自のオブジェクトに抽出し、それらに作業を委任します。このような何か：

class FileManipulator 
    attr_reader :handler 

    def initialize(ssh) 
    @handler = ssh? RemoteHandler.new(ssh) : LocalHandler.new 
    end 

    def create_link 
    handler.create_link if handler.particular_file_exists? 
    rescue => e 
    $LOG.log(2, "Error occurred") 
    end 

    RemoteHandler = Struct.new(:ssh) do 
    def create_link 
     ssh.exec!("ln -s file1 file1-dump") 
    end 
    end 

    LocalHandler = Struct.new do 
    def create_link 
     FileUtils.ln_sf("file1", "file1-dump") 
    end 
    end 
end