0

CombineのPublisherのテストで、以下のような警告が出ています。

Initialization of immutable value 'cancelable' was never used; consider replacing with assignment to '_' or removing it

この警告を(特定の行だけ)抑制する方法はありませんでしょうか?

SwiftLintをはじめとしたLintには行やファイルを指定して特定の警告を無視する手段がありますよね。Xcodeでも同様のものがあればいいなと期待して調べましたが、調べた範囲では見つけられませんでした。

Xcodeのアドバイス通りに_にアサインするとテストが壊れるため(即座に購読がキャンセルされてしまう)cancelableと名前をつけ、テストが終わるまで保持しています。

検討した解決策:

  • テストのインスタンスのプロパティとして保持する方法→不要な情報をテスト後まで保持しておくのは構造が汚いように感じてやめました。
  • LLVMの設定を変えて警告が出ないようにする方法→基本的に警告は出て欲しい(問題ないと確認できたものだけ見えないようにしたい)ので採用できません。
  • var cancelable とする→Initialization of variable 'cancelable' was never usedというほぼ同じエラーが出る。そりゃそうだ。
  • テストの最後でcancelable.cancel()を呼ぶ→変な行が増えるので気持ち悪いが現状これを採用している
2
  • 2
    「ツールは警告するが問題ない」ことを「プログラマの責任で保証する」のであれば逸脱を選択してそのことをコメントで残すだけです。警告を消すために無駄な努力をする=人間がツールに使われている、ってことなので本末転倒。逸脱する手続きが無いってことならその辺の整備が先。
    – 774RR
    Commented 3月28日 2:39
  • @774RR 「警告を消すために無駄な努力をする」状態であるかどうか、という質問です。
    – acevif
    Commented 3月28日 14:08

2 件の回答 2

1

自己解決できましたので共有します。

cancelable_に代入して警告を抑制する方法

let cancelable = foobarbaz.........
_ = cancelable // suppress the 'never used' warning

このworkaroundだと、テスト末尾でcancelable.cancel()を呼ぶ現行の方法よりも定義した場所に近く、また、副作用がないことも自明であるため、可読性が高いと思われます。

訂正:

let cancelable = foobarbaz.........
defer { cancelable.cancel() }

の方が可読性が高くていいかも

(2024-04-22)

警告をコントロールできるSwiftLintに任せる方法

  • SwiftLintを導入しunused_declaration警告を用いるhttps://realm.github.io/SwiftLint/unused_declaration.html
  • 重複するXcodeの警告(GCC_WARN_UNUSED_VARIABLEなど)をNOにする

Objective-Cのコンパイラは(#pragma clang diagnosticのような)細かな警告コントロールが実装されていましたが、
Swiftコンパイラは警告コントロールを(まだ?)実装していません。
SwiftLintを用いるのがより本質的な解決でしょう。

SwiftLintを用いれば、警告を行単位で細かくコントロールし、ソースコード中にプログラマの意図を表明することができます。

SwiftLintはそのうちこのプロジェクトでも必要になると思われることもあり、この際入れてしまおうと思います。

0
0

Cancellableのstore(in:)メソッドを使えばいいのではないでしょうか?

7
  • cancellable.store(in: &disposeBag)という感じでしょうか? cancellable.cancel()とあまり変わらないような...
    – acevif
    Commented 3月30日 8:59
  • Cancellableを変数に入れずにメソッドチェーンでstore(in:)を使うということです。
    – masakih
    Commented 3月31日 12:29
  • ありがとうございます。 (A) disposeBagをテストのメソッド内でローカルに宣言するパターン→行数はcancellable.cancel()よりむしろ増える (B) disposeBagをテストのインスタンスが持つパターン→cancellable.cancel()と行数はほぼ変わらないが、単体テスト同士の間に不要な依存関係ができてしまい、テストの設計としてはむしろ好ましくない。 と思えてしまいますが、勘違いでしょうか?
    – acevif
    Commented 4月3日 20:37
  • 警告を消したかったんじゃないの?
    – masakih
    Commented 4月5日 17:00
  • 1
    あなたが必要と考えている回答の条件を全て質問に追記してほかの方の回答をお待ちください
    – masakih
    Commented 4月11日 12:12

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。