Код либо сломан, либо setAttribute()
бесполезен.
Если вы получаете изменяемый контейнер (например, список или вектор) из сеанса, то нет необходимости помещать его снова в сеанс. Это эквивалент этого кода:
session.setAttribute("Register", buylist);
session.setAttribute("Register", buylist);
session.setAttribute("Register", buylist);
Нет, но первая строка имеет эффект.
Другая возможность состоит в том, что в коде есть ошибка, и на самом деле это означало следующее:
Vector buylist = session.getAttribute("Register");
if( buylist == null ) {
buylist = new Vector();
session.setAttribute("Register", buylist);
}
т.е. создайте новый вектор, если он еще не существует.