Есть ли какой-нибудь эквивалент JavaScript forEach в Rust? - PullRequest
0 голосов
/ 24 февраля 2019

Как мне улучшить эту функцию:

use std::{env, process::exit};

fn get_grid() -> [[u8; 9]; 9] {
    let mut grid: [[u8; 9]; 9] = Default::default();
    let mut args: Vec<String> = env::args().collect();
    if args.len() != 10 {
        eprintln!("This program need 9 strings of 9 numbers between 0 and 9");
        exit(1);
    }
    args.remove(0);
    let _: Vec<_> = args
        .iter()
        .enumerate()
        .map(|(i, arg)| {
            let _line: Vec<_> = arg
                .split(' ')
                .enumerate()
                .map(|(j, value)| match value.parse() {
                    Ok(x) => {
                        grid[i][j] = x;
                        x
                    }
                    Err(e) => {
                        eprintln!("Value {} is not a valid integer [{}]", value, e);
                        exit(1);
                    }
                })
                .collect();
        })
        .collect();
    return grid;
}

Насколько я понимаю, .map() при сборе создаст новую итерацию (здесь Vec) и вернет ее.Мне не нужно иметь эту итерацию, я просто хочу изменить внешний массив и не создавать ничего из этой итерации.

В JavaScript есть .map, но также .forEach, которыйповторяется на map и ничего не возвращает.Есть ли какой-нибудь эквивалент в Rust?

Я мог бы просто использовать for (index, value) in args.iter().enumerate(), но я ищу способ избежать явного цикла, если он есть.

1 Ответ

0 голосов
/ 24 февраля 2019

Для изменения существующей структуры данных использование явного цикла является наиболее идиоматичным способом сделать это:

for (i, arg) in args.iter().enumerate() {
    for (j, value) in arg.split(' ').enumerate() {
        match value.parse() {
            Ok(x) => {
                grid[i][j] = x;
            }
            Err(e) => {
                eprintln!("Value {} is not a valid integer [{}]", value, e);
                exit(1);
            }
        }
    }
}

Вы можете написать это с помощью Iterator::for_each, но это вряд ли будет рассматриваться "лучше "большинством разработчиков Rust:

args.iter().enumerate().for_each(|(i, arg)| {
    arg.split(' ')
        .enumerate()
        .for_each(|(j, value)| match value.parse() {
            Ok(x) => {
                grid[i][j] = x;
            }
            Err(e) => {
                eprintln!("Value {} is not a valid integer [{}]", value, e);
                exit(1);
            }
        })
});

Независимо от того, что вы используете, вы определенно не должны собирать все те Vec s, которые затем выбрасываете.

...