Модифицированный запрос на угадывание книги Rust

Я изменил код из Учебника по игре в угадайку из Rust Book чтобы сделать его немного короче; для слайда. Увы, я допустил ошибку, и она больше не выполняется корректно: первый ввод работает, как ожидалось, но последующие вводы теперь не дают обратной связи.

Каков наилучший способ защиты от этой ситуации?

use std::io;
use std::cmp::Ordering;
use rand::Rng;

fn main() {
    println!("Guess the number!");

    let secret_number = rand::thread_rng().gen_range(1, 101);

    let mut guess = String::new();

    loop {
        io::stdin().read_line(&mut guess)
            .expect("Failed to read line");

        let guess: u32 = match guess.trim().parse() {
            Ok(num) => num,
            Err(_) => continue,
        };

        println!("You guessed: {}", guess);

        match guess.cmp(&secret_number) {
            Ordering::Less => println!("Too small!"),
            Ordering::Greater => println!("Too big!"),
            Ordering::Equal => {
                println!("You win!");
                break;
            }
        }
    }
}

person user2023370    schedule 13.12.2019    source источник
comment
Каков наилучший способ защиты от этой ситуации? какая ситуация?   -  person Stargateur    schedule 13.12.2019
comment
Вопрос обновлен.   -  person user2023370    schedule 13.12.2019


Ответы (1)


read_line добавит строку в буфер, поэтому ваш guess будет накапливать все входные данные, включая символы новой строки! Перемещение let mut guess = String::new(); внутри цикла решает проблему:

fn main() {
    ...

    loop {
        let mut guess = String::new();

        ...
    }
}
person edwardw    schedule 13.12.2019
comment
в этом случае явно лучше повторно использовать ту же память, используя clear() - person Stargateur; 13.12.2019
comment
@Stargateur Конечно. Но OP ищет более короткий код, и он также больше соответствует коду в книге Rust. Я бы сказал, что это более педагогически последовательно. - person edwardw; 13.12.2019