`else return`または` return`を使うべきでしょうか？

Question

ブール値を返す関数を作成するときは、if/elseのすべての文を返す必要がありますか？

たとえば、あなたは、ファイルが迅速に存在していることを確認したい場合、あなたは

func fileExists() -> Bool 
{ 
    if let dir = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first 
    { 
     let path = dir.appendingPathComponent("\(self.name).\(self.type)") 
     if(FileManager.default.fileExists(atPath: path.path)) 
     { 
      return true 
     } 
    } 
    return false 
} 

を行うだろう疑問が示すように、条件が満たされた場合、我々は関数の最後にreturn falseを追加する必要があり、あります上記、または私たちが私に

func fileExists() -> Bool 
{ 
    if let dir = FileManager.default.urls(for: .documentDirectory, in: .userDomainMask).first 
    { 
     let path = dir.appendingPathComponent("\(self.name).\(self.type)") 
     if(FileManager.default.fileExists(atPath: path.path)) 
     { 
      return true 
     } else { 
      return false 
     } 
    } else { 
     return false 
    } 
} 

（下図参照）、各ifなステートメントにreturn falseを追加する必要があり、最初の例はクリーナーに見えます、しかし、私はこれは良い習慣であることを確認していません。それはよりきれいに見えますが、私は代わりに最高の/最も安全な方法を使用したいので、この質問は最もよく見えるものに関する意見ではありません。

Answer 1

この質問は純粋に意見に基づくもののように見えます。しかし、私はあなたの質問に対する答えを含む一般的な機能パターンを知っています。

機能がなければならない：

• そのパラメータを検証します。
• 次に、は、エラー処理を処理し、成功できない場合に救済するタスクを行います。
• は、その結果を計算して返します。

結果は、非常にリニアな制御フローであり、必要以上のネストはありません。このガイドラインをコードに適用する：

func fileExists() -> Bool 
{ 
    //If path exists, the variable dir will have a value. If not, bail out 
    guard let dir = FileManager.default.urls(
     for: .documentDirectory, 
     in: .userDomainMask 
    ).first else { 
     return false 
    } 

    let path = dir.appendingPathComponent("\(self.name).\(self.type)") 

    // if(...) { return true; } else { return false; } pattern factored out 
    return FileManager.default.fileExists(atPath: path.path) 
} 

Answer 2

両方ともここで問題なく動作しますが、より良い練習のためには、他のものを使用する必要があります。その複数の第一のものより安全な、それは両方の例は、条件

if(FileManager.default.fileExists(atPath: path.path)) 

が満たされた場合にのみ「true」を返すも

Answer 3

コードのあなたの将来の範囲を増加します。

これが「真」を返す唯一の方法であるため、わかりやすくするために、最初の例を取り上げます。 上記の条件が満たされない場合、関数は必然的に "false"を返すため、 "else"ブロックは不要です。