Head First Java - Chapter 11 - MiniMusicCmdLine Error

[Sudheer Kolanu]

Hello,

I was trying to execute the program on Pg 346 from Head First Java, MiniMusicCmdLine but the below error message displayed. I have gone through the progream multiple times but could not located what is causing this error.

C:\Library\Headache\Pro\Player>javac MiniMusicCmdLine.java
MiniMusicCmdLine.java:45: class, interface, or enum expected
}→
^
1 error

import javax.sound.midi.*;
public class MiniMusicCmdLine{
public static void main(String[] args){
MiniMusicCmdLine mini = new MiniMusicCmdLine();
if(args.length < 2){
System.out.println("Don't forget the instrument and note args");
}else{
int instrument = Integer.ParseInt(args[0]);
int note = Integer.parseInt(args[1]);
mini.play(instrument,note);
}
}

public void play(int instrument, int note){
try{
Sequencer player = MidiSystem.getSequencer();
player.open();
Sequence seq = new Sequence(Sequence.PPQ, 4);
Track track = seq.CreateTrack();
MidiEvent event = null;

ShortMessage first = new ShortMessage();
first.setMessage(192, 1, instrument, 0);
MidiEvent changeInstrument = new MidiEvent(first, 1);
track.add(changeInstrument);

ShortMessage a = new ShortMessage();
a.setMessage(144,1,note, 100);
MidiEvent noteOn = new MidiEvent(a,1);
track.add(noteOn);

ShortMessage b = new ShortMessage();
b.setMessage(128,1,note, 100);
MidiEvent noteOff = new MidiEvent(b,16);
track.add(noteOff);
player.setSequence(seq);
player.start();
}
catch(Exception ex) 
{
ex.printStackTrace();
}
}
}

[Christophe Verré]

Apart from the fact that there some unknown methods in your cord (e.g. ParseInt), it seems that there is a strange character at the end of your file. Maybe you copy/pasted the code and it left some strange characters ? Remove the last line, and retype "}".

[Sudheer Kolanu]

I have typed in everything. I just noticed that there was an extrac closing curly brace at the last which was not posted in this thread which I removed now but getting differnt error now.

C:\Library\Headache\Pro\Player>javac MiniMusicCmdLine.java
MiniMusicCmdLine.java:8: cannot find symbol
symbol : method ParseInt(java.lang.String)
location: class java.lang.Integer
int instrument = Integer.ParseInt(args[0]);
^
1 error

[Christophe Verré]

Sudheer Kolanu wrote:I have typed in everything. I just noticed that there was an extrac closing curly brace at the last which was not posted in this thread

Thanks for that...

but getting differnt error now.

That's the error I was talking about. Check the name of your method (tip: Java is case-sensitive)

[Sudheer Kolanu]

Thank you, it worked.

[Campbell Ritchie]

Sudheer Kolanu wrote:I have typed in everything. . . .

You shouldn't type anything. You should copy-and-paste, then you are sure you are posting real code.