1. Почему строковые константы, а не enum-ы? 2. stop_labels можно сделать

Question

1. Почему строковые константы, а не enum-ы? 2. stop_labels можно сделать

HashSet<Label> вместо Vec и вообще засунуть в once_cell. Можно просто слайсом оставить: static STOP_LABELS: &'static [&'static str] = &["availability", ...]
3. rec.value != String::new() && rec.value.len() > 2 -- зачем первое условие, если второе и так проверит, что строка не пустая?
4. subset.iter().for_each(|rec| println!("{}", rec.value)); -- я бы обычный for loop посоветовал.
5. Переменная l в 69 строке -- зачем? Можно же rec.label = "time_detected".to_owned()

Для мелкой утилиты вообще не принципиально + половина это вопрос вкуса.

#backend #embedded #programming #russian #rust

0

22.12.2019