Error ')' or ',' expected

An error ')' or ',' expected error seems straight-forward but has stumped me since last evening.

Trying all known fixes, I think perhaps a second or third set of eyes might see the error.

Enclosed is a code segment with the line 15  (or a close line) throwing the error:

   statusLabel = new JLabel("Move the mouse over this panel.");                            middleImageLabel.add(statusLabel);                            middleImageLabel.addMouseMotionListener(new MouseMotionListener() {                                @Override                                public void mouseDragged(MouseEvent e) {                                    System.out.println("Mouse dragged at: " + e.getX() + ", " + e.getY());                                }                                @Override                                public void mouseMoved(MouseEvent e) {                                    System.out.println("Mouse moved at: " + e.getX() + ", " + e.getY());                                }                  } if (randomNumber1 > 26 && randomNumber2 < 50) {                                leftImageLabel4.setIcon(image16);//blue_blank                                middleImageLabel4.setIcon(image11);//arrow_straight                                rightImageLabel4.setIcon(image16);//blue_blank                            }

Thanks

How are you compiling the code?  If you use the javac command it gives this error message with ^ showing where the error is:
TestCode28.java:340: error: ')' expected      int x = Integer.parseInt("12"                                   ^

Mat Dif wrote:. . . Trying all known fixes, I think perhaps a second or third set of eyes . . .

You might have seen it yourself if you had indented your code correctly. The long lines make it very difficult to read. Look in the old Sun style guide, which tells you about line lengths. Keep your lines not more than 80 characters, in monospaced type. I have corrected your code to shorten the lines and remove unnecessary whitespace and got the following:-statusLabel = new JLabel("Move the mouse over this panel."); middleImageLabel.add(statusLabel); middleImageLabel.          addMouseMotionListener(new MouseMotionListener() {     @Override     public void mouseDragged(MouseEvent e) {          System.out.println("Mouse dragged at: " +                  e.getX() + ", " + e.getY());     }     @Override     public void mouseMoved(MouseEvent e) {           System.out.println("Mouse moved at: " +                  e.getX() + ", " + e.getY());     } } if (randomNumber1 > 26 && randomNumber2 < 50) {     leftImageLabel4.setIcon(image16);//blue_blank     middleImageLabel4.setIcon(image11);//arrow_straight     rightImageLabel4.setIcon(image16);//blue_blank }You appear to be using five spaces per indentation level, which is too many; stick to three or four. And get your text editor to convert tabs to spaces automatically. I haven't corrected that problem. At least you aren't using tabs.
Look at line 4. It has a ( in its middle. There isn't a corresponding ) to finish your method call. The anonymous class should finish with a }, followed by ); to complete the method call. The if, now line 17, appears to the javac tool to be part of the anonymous class, and the syntax doesn't make sense.
Search my posts for writing backwards. What you do is start the method callabc.addXYZListener(); // won't compileThen add the listener:-abc.addXYZListener(new XYZListener() { }); // won't compileThen fill in the methods:-abc.addXYZListener(new XYZListener() {   @Override   public void foo(XYZEvent e)   {      something();   }   @Override   public void bar(XYZEvent e)   {      somethingElse();   } }); // will compile :)
I don't like the indentation type with } starting the line, then if, then { to finish the line. I would prefer to see the if as a line by itself.
Are you aware of classes called XYZAdapter? Each XYZListener with more than one method has a corresponding class called XYZAdapter. You only need to implement the methods you actually need, but be sure to check for spelling errrors by using @Override.

Please decide where you want the if. It might never be called where it is. If you want it called when something happens to the mouse, put the if into another method in the anonymous class and call that method from another method.

Back when I worked developing code for the Macintosh, co-workers would exploit the fact that modern monitors could display lots of text to produce code with up to 100 characters or mode in a line.

I hated this because, for one thing, if you printed that sort of stuff and you didn´t have a tiny enough font applied, the code would run off the edge of the paper.

My own preference for line length is 65 characters. It's actually shaped from the ancient days when I had to code on punched cards. Although standard IBM cards ran to 80 columns, most programming languages reserved up to 8 characters for a sequence number. If you dropped the deck, a quick run through a sorting machine would put them back in the right order. Some languages carved out additional zones, hence 65 instead of 72.

This carried over to when we got terminals with full-screen editors. The standard for programming terminals was for a long time 24 rows x 80 columns. The text editors here would generally prefix or suffix the edit text with a zone where you typed in editing commands.

The days are mostly behind us now, but there are still good reasons to keep line lengths from running off into infinity.

Norm Radder wrote:How are you compiling the code?  If you use the javac command it gives this error message with ^ showing where the error is:
TestCode28.java:340: error: ')' expected      int x = Integer.parseInt("12"                                   ^

NetBeans/Maven

Thanks.

Campbell Ritchie wrote:Please decide where you want the if. It might never be called where it is. If you want it called when something happens to the mouse, put the if into another method in the anonymous class and call that method from another method.

Thank you for your reply. I'm going to begin looking into this tonight and will post back here should I have questions.

-Matthew

Norm Radder wrote:. . .      int x = Integer.parseInt("12"

Where did that code come from? I can't find it.

I made it as an example for the compiler to give a missing ) message.
The OP had said:

with the line 15  (or a close line) throwing the error:

I wondered if he could locate the error more closely than that.

The } that completes the anonymous class is at the beginning of OP's line 15. That requires ); following to complete the method call. The remainder of that line (if ...) now seems to be dangling somewhere and, assuming it compiles in the first place, is probably impossible to call from anywhere.

In the situation where you create an anonymous class (what you do with "addMouseMouseMotionListener(...)" is to align all your parantheses and braces. So like:
middleImageLabel.addMouseMotionListener(new MouseMotionListener()    {        @Override        public void mouseDragged(MouseEvent e) {            System.out.println("Mouse dragged at: " + e.getX() + ", " + e.getY());        }        @Override        public void mouseMoved(MouseEvent e) {            System.out.println("Mouse moved at: " + e.getX() + ", " + e.getY());        }    } ); if (randomNumber1 > 26 && randomNumber2 < 50) { ...}
then you see at once what Campbell refers to.