有没有读过head first设计模式的啊?其中的组合模式的一段算法代码有问题。 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
RyanZhu
V2EX    程序员

有没有读过head first设计模式的啊?其中的组合模式的一段算法代码有问题。

  • /li>  
  •   RyanZhu 2013-03-03 20:37:49 +08:00 3917 次点击
    这是一个创建于 4606 天前的主题,其中的信息可能已经有所发展或是发生改变。
    今天在看head first设计模式,发现在组合模式中,有段代码有问题,是在书的第369页,我利用书中的CompositeIterator来遍历MenuComponent时候,apple pie会打印两次,个人认为是在hasNext()实现的时候,漏pop了一个迭代器。我不知道是哪里错了,为何在有嵌套的对象中,嵌套对像会被打印两次呢?

    @Override
    public boolean hasNext() {
    // TODO Auto-generated method stub
    if (stack.empty()) {
    return false;
    } else {
    Iterator<MenuComponent> iter = stack.peek();
    if (!iter.hasNext()) {
    this.stack.pop();
    return hasNext();
    } else {
    return true;
    }
    }
    }

    @Override
    public MenuComponent next() {
    // TODO Auto-generated method stub
    if (hasNext()) {
    Iterator<MenuComponent> iter = stack.peek();
    MenuComponent compOnent= iter.next();
    if(component instanceof MenuItem&&component.getName().equals("Apple pie")){
    System.out.println("hello");
    }
    if (component instanceof Menu) {
    stack.push(component.createIterator());
    }
    return component;
    } else {
    return null;
    }
    }


    public static void main(String[] args) throws Exception {
    // TODO Auto-generated method stub
    Menu menu1=new Menu("Pancake house menu", "Breakfase");
    Menu menu2=new Menu("Diner menu","Lunch");
    Menu menu3=new Menu("Cafe menu", "Dinner");
    Menu menu4=new Menu("Dessert menu", "dessert of course");
    Menu allMenus=new Menu("All menus", "All menu combined");

    allMenus.add(menu1);
    allMenus.add(menu2);
    allMenus.add(menu3);

    menu2.add(new MenuItem("Pasta", "blarblarblarblar", true, 3.89));
    menu2.add(menu4);
    menu4.add(new MenuItem("Apple pie", "blarblarblarblar", true, 1.59));

    Waitress w=new Waitress(allMenus);
    w.printVegetarianMenu();
    }

    public class Waitress {
    MenuComponent allMenus;

    public Waitress(MenuComponent mc){
    this.allMenus=mc;
    }

    public void printMenu(){
    allMenus.print();
    }

    public void printVegetarianMenu() throws Exception{
    Iterator<MenuComponent> iter=allMenus.createIterator();
    int i=0;
    while(iter.hasNext()){
    if(i==2){
    i++;
    }
    MenuComponent mc=iter.next();
    try{
    if(mc.isVegetarian()){
    i++;
    mc.print();
    }
    }catch(UnsupportedOperationException ex){}
    }
    }
    }

    menu 4里面的apple pie会被打印两次,这是为什么呢?
    2 条回复    1970-01-01 08:00:00 +08:00
    cikelengfeng
        1
    cikelengfeng  
       2013-03-05 13:09:18 +08:00
    用断点调试很容易发现这类错误,试试吧
    RyanZhu
        2
    RyanZhu  
    OP
       2013-03-07 19:31:02 +08:00
    @cikelengfeng 调试不方便的。。递归算法没法watch。。。不过基本确定就是,栈没清干净。。。虽然还是没做出来。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     5400 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 30ms UTC 07:42 PVG 15:42 LAX 00:42 JFK 03:42
    Do have faith in what you're doing.
    ubao snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86