【Effective-Rustlings-jp】Day 19：Clippyに耳を傾けよう

# はじめに
こんにちは。細々とプログラミングをしている[sotanengel](https://x.com/sota_eng_prog)です。
この記事は以下の記事の連載です。

https://qiita.com/sotanengel/private/78103dcfe2fa34f8815e


<details>
<summary><strong>他の連載記事</strong> (<em>詳細</em>)</summary>

- [Day 1：型システムを使ってデータ構造を再現しよう](https://qiita.com/sotanengel/private/588a7f7cc03c36fc1adc)
- [Day 2：型システムを用いて共通の挙動を表現しよう](https://qiita.com/sotanengel/private/2a8fc62b6d9f8e916021)
- [Day 3：OptionとResultに対してはmatchを用いずに変換しよう](https://qiita.com/sotanengel/items/b54990dc82d877b9d70d)
- [Day 4：標準のErrorを使おう](https://qiita.com/sotanengel/private/79f683c3d75d09244567)
- [Day 5：型変換を理解しよう](https://qiita.com/sotanengel/private/8cc28c8b6108e5998e23)
- [Day 6：newtypeパターンを活用しよう](https://qiita.com/sotanengel/private/38814f8b5a88aad9eccc)
- [Day 7：複雑な型にはビルダを使おう](https://qiita.com/sotanengel/private/80bc555e95ed56b72f14)
- [Day 8：明示的なループの代わりにイテレータ変換を使用することを検討しよう](https://qiita.com/sotanengel/private/f7f4fc03f0b2dc445bf2)
- [Day 9：標準トレイトに習熟しよう](https://qiita.com/sotanengel/private/24a8ad443fc23e779ccd)
- [Day 10：RIIパターンにはDropトレイトを実装しよう](https://qiita.com/sotanengel/private/a3d1b04f98ea80b8989d)
- [Day 11：ジェネリクスとトレイトオブジェクトのトレードオフを理解しよう](https://qiita.com/sotanengel/private/8e2e88d5619b434c3ada)
- [Day 12：デフォルト実装を用いて、実装しなければならないトレイトメソッドを最小限にしよう](https://qiita.com/sotanengel/private/16b02d32732727eaf66f)
- [Day 13：Don't panic](https://qiita.com/sotanengel/private/aa833ce99c8e8b8eecf2)
- [Day 14：リフレクションを避けよう](https://qiita.com/sotanengel/private/6108b22f6644d4fde997)
- [Day 15：可視範囲を最小化しよう](https://qiita.com/sotanengel/private/4075925e05de50dfac96)
- [Day 16：ワイルドカードインポートを避けよう](https://qiita.com/sotanengel/private/555314f77c8318bb1c45)
- [Day 17：パブリックインターフェースのドキュメントを書こう](https://qiita.com/sotanengel/private/b4e8a11522b253ebb62e)
- [Day 18：分別を持ってマクロを使おう](https://qiita.com/sotanengel/private/a33b5ab66a82a26b4adc)
- [Day 19：Clippyに耳を傾けよう](https://qiita.com/sotanengel/private/b4c161bb87fc9e7e813e)
- [Day 20：ユニットテスト以上のものを書こう](https://qiita.com/sotanengel/private/58fcafaee5418b2b2b16)

</details>

また本記事は[Effective Rust(David Drysdale (著), 中田 秀基 (翻訳))](https://www.oreilly.com/library/view/effective-rust/9781098151393/)を参考に作成されております。とてもいい書籍ですので興味を持った方は、ぜひ読んでみてください！
<div style="text-align: center;">
  <figure>
    <a href="https://www.amazon.co.jp/Effective-Rust-%E2%80%95Rust%E3%82%B3%E3%83%BC%E3%83%89%E3%82%92%E6%94%B9%E5%96%84%E3%81%97%E3%80%81%E3%82%A8%E3%82%B3%E3%82%B7%E3%82%B9%E3%83%86%E3%83%A0%E3%82%92%E6%9C%80%E5%A4%A7%E9%99%90%E3%81%AB%E6%B4%BB%E7%94%A8%E3%81%99%E3%82%8B%E3%81%9F%E3%82%81%E3%81%AE35%E9%A0%85%E7%9B%AE-David-Drysdale/dp/4814400942"  target="_blank">
      <img src="https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/1758960/cb1245be-02d6-0c74-8b79-e46288f2ea73.png" alt="商品画像" width="50%">
    </a>
  </figure>
</div>

# 今日の内容
## 概要
Rustには`Clippy`というCargoの追加コンポーネントが存在します。
このコンポーネントを使うことで安全で可読性の高いコード開発が可能となります。

https://github.com/rust-lang/rust-clippy

https://qiita.com/msakuta/items/db0ad407ff39b649a9a6

## 使わない変数名
### 問([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/exercises/29_tooling_Listen_to_clippy/1_unused_variables.rs))
使わない変数は命名によってはClippyで警告が出ます。
修正してみましょう。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    // TODO: 使わない変数の記載方法を確認して、以下のコードを修正してください。
    let unused_var = 42;
    println!("Hello, world!");
}
```

</details>

### 解答([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/solutions/29_tooling_Listen_to_clippy/1_unused_variables.rs))
コード参照。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    let _unused_var = 42; // 変数を使わない場合はアンダースコアで始める
    println!("Hello, world!");
}
```

</details>

## 無駄なクローン
### 問([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/exercises/29_tooling_Listen_to_clippy/2_clone_on_copy.rs))
無駄なクローンはClippyで警告が出ます。
修正してみましょう。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    let x = 5;
    // TODO: 以下は不要なクローンです。修正してください。
    let y = x.clone();
    println!("{}", y);
}
```

</details>

### 解答([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/solutions/29_tooling_Listen_to_clippy/2_clone_on_copy.rs))
コード参照。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    let x = 5;
    let y = x; // クローン不要
    println!("{}", y);
}
```

</details>

## 非効率な結合
### 問([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/exercises/29_tooling_Listen_to_clippy/3_inefficient_to_string.rs))
文字列の非効率な結合はClippyで警告が出ます。
修正してみましょう。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    // TODO: 非効率的な文字列結合を修正してください。
    let s = "Hello".to_string() + " " + &"world!".to_string();
    println!("{}", s);
}
```

</details>

### 解答([リンク](https://github.com/sotanengel/effective-rustlings-jp/blob/20250101/solutions/29_tooling_Listen_to_clippy/3_inefficient_to_string.rs))
コード参照。

<details>
<summary><strong>コード</strong> (<em>詳細</em>)</summary>

```rust
fn main() {
    let s = format!("{} {}", "Hello", "world!"); // `format!`を使用して効率的に結合
    println!("{}", s);
}
```

</details>

# さいごに

もしも本リポジトリで不備などあれば、リポジトリのissueやPRなどでご指摘いただければと思います。

https://github.com/sotanengel/effective-rustlings-jp/issues