在Struct中实例化2d Vec?

时间:2015-01-26 06:45:31

标签: rust

在使用构造函数返回新的struct对象时,我无法实例化vec。我尝试过的语法(可能不正确地使用collect())会导致大量的编译器错误。

fn main() {
    let level = Level::new();
}

struct Level {
    tiles: Vec<Vec<Tile>>
}

struct Tile {
    idx: i32
}

impl Level {
    fn new() -> Level {
        Level {
            tiles: {
            let mut t = Vec::new();
            let mut t2 = Vec::new();
            for x in range(0, 80) {
                for y in range(0, 24) {
                    t2.push(Tile::new(x, y));
                }
                t.push(t2);
            }
            t
        }
    }
}

impl Tile {
    fn new(x: i32, y: i32) -> Tile {
        Tile { pos: Point { x: x, y: y } }
    }
}

struct Point {
    x: i32,
    y: i32
}

我收到这些错误:

src/game/dungeon/level/mod.rs:47:25: 47:27 error: use of moved value: `t2`
src/game/dungeon/level/mod.rs:47                 t2.push(Tile::new(x, y));
                                                     ^~
src/game/dungeon/level/mod.rs:49:28: 49:30 note: `t2` moved here because it has type `collections::vec::Vec<game::dungeon::level::Tile>`, which is non-copyable
src/game/dungeon/level/mod.rs:49                     t.push(t2);
                                                        ^~
src/game/dungeon/level/mod.rs:49:28: 49:30 error: use of moved value: `t2`
src/game/dungeon/level/mod.rs:49                     t.push(t2);
                                                        ^~
src/game/dungeon/level/mod.rs:49:28: 49:30 note: `t2` moved here because it has type `collections::vec::Vec<game::dungeon::level::Tile>`, which is non-copyable
src/game/dungeon/level/mod.rs:49                     t.push(t2);
                                                        ^~

2 个答案:

答案 0 :(得分:4)

是的,你做错了。类似的代码在C / C ++,BTW中也是不正确的。

        let mut t = Vec::new();
        let mut t2 = Vec::new();
        for x in range(0, 80) {
            for y in range(0, 24) {
                t2.push(Tile::new());
            }
            t.push(t2);
        }

问题是,你总是在内循环中推进相同的t2,然后你总是将同一t2推入t。后者违反了所有权语义,因此Rust编译器正确地告诉您使用移动值。

惯用法是使用迭代器,它看起来像这样:

(0..80).map(|_| (0..24).map(|_| Tile::new()).collect()).collect()

如果您需要访问索引,可以使用map()闭包参数:

(0..80).map(|x| (0..24).map(|y| Tile::new(x, y)).collect()).collect()

编译器应自动推导出所需的collect()结果类型。

答案 1 :(得分:3)

弗拉基米尔的回答非常好,不过我觉得功能风格可能会隐藏错误。

你实际上离解决方案不远;问题很简单,就是你不能在外循环的每次迭代中重用相同的t2。因此,最简单的转换是在外部循环中创建t2

impl Level {
    fn new() -> Level {
        Level {
            tiles: {
            let mut t = Vec::new();
            for x in range(0, 80) {
                let mut t2 = Vec::new(); // Moved!
                for y in range(0, 24) {
                    t2.push(Tile::new(x, y));
                }
                t.push(t2);
            }
            t
        }
    }
}