Adding new object to vector in existing session

Question

Hi,
I'm currently developing a website. In it, the user can click a button(name=option,value=item) and an item gets added to a list. At the end, the list is outputted. I am having difficulties though with my session.
My session("Item") holds a collection,basically a vector. Every time the button is pressed, I want to add a new object (Item) to this vector(ItemCollection) but when I do it simply overrides it so the list is only ever outputting one item. Here is the code. I have traced through it with 'System.out.print' and it only gets to 'System.out.println("session");
The getItem(), gets the item from the database and returns a object of type Item.
This may seem like only a basic thing but I just cant see where it is going wrong.
Thanks
else if(option.equals("item")){
                   System.out.println("ItemPressed");
                   String itemIdStr=request.getParameter("id");
               int itemId=Integer.parseInt(itemIdStr);
                  ItemCollection list=new ItemCollection();
                   list=(ItemCollection)session.getAttribute("Item");
                   Item item=new Item();
                   Item itS=new Item();
                   System.out.println("getItem()");
                   itS=item.getItem(itemId);
                   System.out.println(itS.getItemName());
                   System.out.println("session");
                   list.addItem(itS);
                   session.setAttribute("Item",list);     
                System.out.println("addedToSession");
              }

Answer 1

else if(option.equals("item")){
                System.out.println("ItemPressed");
                String itemIdStr=request.getParameter("id");
               int itemId=Integer.parseInt(itemIdStr);
               ItemCollection list=new ItemCollection();
                list=(ItemCollection)session.getAttribute("Item");
                Item item=new Item();
                Item itS=new Item();
                System.out.println("getItem()");
                itS=item.getItem(itemId);
                System.out.println(itS.getItemName());
                System.out.println("session");
                list.addItem(itS);
                session.setAttribute("Item",list);     
                System.out.println("addedToSession");
           }Maybe it's too early in the morning here, but it looks to me like you have all of your variables declared in your execution path....
... if that is the case, then uh... ya, they are going to be reinitalized each pass and only have at most the last item in them from your work.