//for(int i=0;i<=st.countTokens();i++)
here you can't use countTokens as loop condition, because it varies during execution. Once nextToken() executes once, the value of countTokens() decrements by one