Try for flow controlを使用することをお勧めしますか？

Question

関数は、このようなシグネチャを持つ私たちのコードベースにあります：

def hasPermission(...): Try[Unit] 

ユーザーが特定のアイテムに特定のアクションを実行する権限を持っている場合、それは基本的にチェックします。ユーザーに許可がある場合は空のSuccessを返し、そうでない場合は特定の例外タイプのFailureを返します。この機能は、多くの場合、このような内包内で使用されています

for { 
    _ <- hasPermission(...) 
    res <- doSomething() 
} yield res 

これは悪い習慣のように思えるが、私はそのように感じ、なぜ私はかなり明確にすることはできません。私には、hasPermissionのように見えるのは、単にBooleanです。

これはTryの適切な使用ですか？

編集：より具体的であるため、私の質問はリンクされたものとは違うと思います。その人はTry [Unit]を返すという一般的な質問をしていますが、私はそれが受け入れられると信じています。

Answer 1

方法がhasPermissionと表示される場合は、BooleanまたはTry[Boolean]を返します。 Try[Unit]はTry[Boolean]のようにはっきりしていません。呼び出し元は、例外を調べて、アクセス権がないかどうか、またはアクセス許可情報の取得に失敗したかどうかを判断する必要があります。

これは一般的にはhasPermissionと呼ばれ、その結果に応じて演技することで競争条件が発生する可能性があります（例：hasPermissionが呼び出された後で許可が取り消された場合）。したがって、多くの場合、def doSomething(...): Try[Unit]を実行し、次にa NoPermissionException。

Answer 2

Generally, the use of exceptions for control flow is an anti-pattern

Tryトライ（しゃれが意図していない）、そのフロー制御をカプセル化するために、しかし、あなたが実際には例外を使用するを必要としない場合は、理由はありません。 Scalaの2.12のEither実装は、おそらく欲しいものに近いようだ：

どちらかがRightが上で動作するデフォルトのケースを想定していることを意味し、右バイアスされています。あなたがウェブサーバを実装していると仮定しましょう、そしてこのロジックは、特定のパスを制御している

：

type Request = ... 
type Response = String 
type ResponseOr[+T] = Either[Response, T] 

def checkPermission: ResponseOr[Unit] = 
    if(hasPermission) Right(()) 
    else Left("insufficient permissions") 

def doSomething(req: Request): ResponseOr[Something] = 
    if(argumentsAreBad(req)) Left("arguments are bad!") 
    else Right(new Something) 

def makeResponse(result: Something): Response = ??? 

def handleIt(req: Request): Response = { 
    val result = for { 
    _ <- checkPermission 
    result <- doSomething 
    } yield makeResponse(result) 
    result.merge // special method for `Either[T, T]` that gives a `T` 
} 

それはLeftであれば、mapとflatMapのような操作はそのままLeft値を返しますSuccess対Failureと同様の動作が表示されます - LeftをFailureに類似していると思います。フラットマップ/マップステップのいずれかがLeftを返す場合は、それが最終結果ですstはスキップされます。例外は必要ありません。