Exception in thread "main" java.util.NoSuchElementException in if statem

Question

if (charIte.next()=='{' || charIte.next()=='}'
                || charIte.next()=='[' || charIte.next()==']' 
                || charIte.next()=='(' || charIte.next()==')'
                || charIte.next()=='*' || charIte.next()=='"' 
                || charIte.next()=='/'){
}

The program returns:

Exception in thread "main" java.util.NoSuchElementException at line
|| charIte.next()=='(' || charIte.next()==')'

What is the problem?

The problem is you're calling `next()` too many times. You need to store it in a variable. — shmosel, Oct 10 '17 at 03:10

score 2 · Answer 1 · answered Oct 10 '17 at 03:11

Each invocation of next() consumes one token. Call it once, and save and then compare with the result. Like,

char ch = charIte.next();
if (ch == '{' || ch == '}' || ch == '[' || ch == ']' || ch == '(' 
            || ch == ')' || ch == '*' || ch == '"' || ch == '/') {
    // ...
}

score 0 · Answer 2 · answered Oct 10 '17 at 03:19

Each time you do charIte.next() you're asking to read the next token. I think what you should do to ischar ite = charIte.next.chartAt(0) and then use ite in your if statement

if (ite next()=='{' || next()=='}' 
|| ite.next()=='[' || ite.next()==']' 
            || ite.next()=='(' || ite.next()==')'
            || ite.next()=='*' || ite.next()=='"' 
            || ite.next()=='/'){

Info on Scanner

Exception in thread "main" java.util.NoSuchElementException in if statem

2 Answers2