Control Statement Violation: `if`, `for`, `guard`, `switch`, `while`, and `catch` statements shouldn't unnecessarily wrap their conditionals or arguments in parentheses. (control_statement)

公開日:2019-07-09
最終更新:2019-07-09

環境

macOS : Majove 10.14.5
XCode : 10.2
Swift : 5
Deployment Target : 12.2

起こったこと

XCode でプロジェクトをビルドしたら、下記の Warning が表示された。

Control Statement Violation: `if`, `for`, `guard`, `switch`, `while`, and `catch` statements shouldn't unnecessarily wrap their conditionals or arguments in parentheses. (control_statement)  

コード (修正前)

xxx.filter { /* 省略 */ }  
            .map { (event) -> ( /* 省略 */ ) in  
                switch ( event ) {    // ここで Build Warning 発生  
                case .error( /* 省略 */ ):  
                    /* 省略 */  
                case .completed:  
                    /* 省略 */  
                }  
            }  
            .subscribe( /* 省略 */ )  
            .disposed(by: disposeBag)  

Warning の原因

switch ( event ) {の、条件文を囲んでいる() が不要だった。
SwiftLint が出してくれる Warning の一つで、control_statementという種類。

コード (修正後)

xxx.filter { /* 省略 */ }  
            .map { (event) -> ( /* 省略 */ ) in  
                switch  event  {  
                case .error( /* 省略 */ ):  
                    /* 省略 */  
                case .completed:  
                    /* 省略 */  
                }  
            }  
            .subscribe( /* 省略 */ )  
            .disposed(by: disposeBag)  

参考リンク

記事が少しでもいいなと思ったらクラップを送ってみよう!
0
+1
できることがふえるといいなぁ

よく一緒に読まれている記事

0件のコメント

ブログ開設 or ログイン してコメントを送ってみよう
目次をみる

技術ブログをはじめよう

Qrunch(クランチ)は、ITエンジニアリングに携わる全ての人のための技術ブログプラットフォームです。

技術ブログを開設する

Qrunchでアウトプットをはじめよう

Qrunch(クランチ)は、ITエンジニアリングに携わる全ての人のための技術ブログプラットフォームです。

Markdownで書ける

ログ機能でアウトプットを加速

デザインのカスタマイズが可能

技術ブログ開設

ここから先はアカウント(ブログ)開設が必要です

英数字4文字以上
.qrunch.io
英数字6文字以上
ログインする