fn card_states_init(
    mut cmds: Commands,
    query_card: Query<(Entity, Option<&Children>), Added<Card>>,
    query_card_none: Query<(Entity, Option<&CardNone>)>,
    query_card_hover: Query<(Entity, Option<&CardHover>)>,
    query_card_action: Query<(Entity, Option<&CardAction>)>,
) {
    for (card, children) in query_card.iter() {
        if let None | Some((_, None)) = children
            .map(|c| c.iter().find_map(|c| query_card_none.get(*c).ok()))
            .flatten()
        {
            let child = cmds.spawn(CardNone).id();
            cmds.entity(card).add_child(child);
        }
        if let None | Some((_, None)) = children
            .map(|c| c.iter().find_map(|c| query_card_hover.get(*c).ok()))
            .flatten()
        {
            let child = cmds.spawn(CardHover).id();
            cmds.entity(card).add_child(child);
        }
        if let None | Some((_, None)) = children
            .map(|c| c.iter().find_map(|c| query_card_action.get(*c).ok()))
            .flatten()
        {
            let child = cmds.spawn(CardAction).id();
            cmds.entity(card).add_child(child);
        }
    }
}
我在写上面这行代码段的时候在思考, 我是否应该将这重复的四段代码修改为下面这样:
fn card_states_init(
    mut cmds: Commands,
    query_card: Query<(Entity, Option<&Children>), Added<Card>>,
    query_card_none: Query<(Entity, Option<&CardNone>)>,
    query_card_hover: Query<(Entity, Option<&CardHover>)>,
    query_card_action: Query<(Entity, Option<&CardAction>)>,
) {
    for (card, children) in query_card.iter() {
        card_state_init(&mut cmds, children, card, &query_card_none);
        card_state_init(&mut cmds, children, card, &query_card_hover);
        card_state_init(&mut cmds, children, card, &query_card_action);
    }
}
fn card_state_init<T: Component>(
    cmds: &mut Commands,
    children: Option<&Children>,
    card: Entity,
    query_card_state: &Query<(Entity, Option<&T>)>,
) {
    if let None | Some((_, None)) = children
        .map(|c| c.iter().find_map(|c| query_card_state.get(*c).ok()))
        .flatten()
    {
        let child = cmds.spawn(CardNone).id();
        cmds.entity(card).add_child(child);
    }
}
这样修改乍一看确实简化了重复, 但是却引入了泛型, 如何后面这几个 state 在初始化的时候要单独处理就要么退回来重新用原本的方案再修改, 要么再写一种 init, 然后修改. 感觉不如一开始就不做这样的 "过早优化"
这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。
V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。
V2EX is a community of developers, designers and creative people.