randombits
randombits

Reputation: 48490

Java: Identifier expected

What's the issue here?

class UserInput {
  public void name() {
    System.out.println("This is a test.");
  }
}

public class MyClass {
  UserInput input = new UserInput();
  input.name();
}

This complains:

<identifier> expected
   input.name();

Upvotes: 52

Views: 573218

Answers (5)

Eliezer Miron
Eliezer Miron

Reputation: 408

I saw this error with code that WAS in a method; However, it was in a try-with-resources block.

The following code is illegal:

    try (testResource r = getTestResource(); 
         System.out.println("Hello!"); 
         resource2 = getResource2(r)) { ...

The print statement is what makes this illegal. The 2 lines before and after the print statement are part of the resource initialization section, so they are fine. But no other code can be inside of those parentheses. Read more about "try-with-resources" here: https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html

Upvotes: 0

Jonathan Payne
Jonathan Payne

Reputation: 2223

Try it like this instead, move your myclass items inside a main method:

    class UserInput {
      public void name() {
        System.out.println("This is a test.");
      }
    }

    public class MyClass {

        public static void main( String args[] )
        {
            UserInput input = new UserInput();
            input.name();
        }

    }

Upvotes: 6

Bohemian
Bohemian

Reputation: 425308

Put your code in a method.

Try this:

public class MyClass {
    public static void main(String[] args) {
        UserInput input = new UserInput();
        input.name();
    }
}

Then "run" the class from your IDE

Upvotes: 57

geekosaur
geekosaur

Reputation: 61459

input.name() needs to be inside a function; classes contain declarations, not random code.

Upvotes: 8

Tudor
Tudor

Reputation: 62459

You can't call methods outside a method. Code like this cannot float around in the class.

You need something like:

public class MyClass {

  UserInput input = new UserInput();

  public void foo() {
      input.name();
  }
}

or inside a constructor:

public class MyClass {

  UserInput input = new UserInput();

  public MyClass() {
      input.name();
  }
}

Upvotes: 18

Related Questions