blog.bouzuya.net

2024-03-29 clippy::format_collect

署名付き URL の続きを進める過程で、 clippy の警告を受けたのでメモ。

signature
    .into_iter()
    .map(|b| format!("{:02x}", b))
    .collect::<String>()

と書いたところ↓のように警告を受けた。

use of `format!` to build up a string from an iterator
this can be written more efficiently by appending to a `String` directly
for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#format_collect
`#[warn(clippy::format_collect)]` on by default
clippyClick for full compiler diagnostic
lib.rs(168, 14): call `fold` instead
lib.rs(168, 22): ... and use the `write!` macro here

clippy::format_collect に怒られた。効率が悪いのはまあわかる。説明どおり https://rust-lang.github.io/rust-clippy/master/index.html#format_collect を確認する。それによると。

fn hex_encode(bytes: &[u8]) -> String {
    bytes.iter().map(|b| format!("{b:02X}")).collect()
}

のかわりに

use std::fmt::Write;
fn hex_encode(bytes: &[u8]) -> String {
    bytes.iter().fold(String::new(), |mut output, b| {
        let _ = write!(output, "{b:02X}");
        output
    })
}

を使えとのこと。わかるけど面倒だ。

crates:hex があるのでそちらを使うのも良さそうだ。


お腹の調子が悪い。食べ過ぎか……。


今日のコミット。