float xpos;
float ypos;
float rectW;
float rectH;
float cl;
float R;
float G;
float B;


import promidi.*;

MidiIO midiIO;

void setup(){
 size(640, 480);
 smooth();
 background(0);
 
 //get an instance of MidiIO
 midiIO = MidiIO.getInstance(this);
 println("printPorts of midiIO");
 
 //print a list of all available devices
 midiIO.printDevices();
 
 //open the first midi channel of the first device
 midiIO.openInput(0,0);
 midiIO.openInput(1,0);
 frameRate(30);

 

}

void draw(){
 moveBox();
}

void noteOn(Note note, int device, int channel){
 
 if (note.getPitch() ==0) {
   R = random(255);
   G = random(255);
   B = random(255);
 
 }
        println(note.getPitch());

}

void moveBox() {
  color cl = color(R, G, B);
 fill(cl);

 if (rectW < 20){
  rectW = 20;
 }else if (rectH <20){
  rectH=20;
 }
 
 
rect(xpos, ypos, rectW, rectH);
}

void controllerIn(Controller controller, int device, int channel){
 if (controller.getNumber() == 4){
  xpos = controller.getValue() * 5;
 } else if (controller.getNumber() == 5){
  ypos = controller.getValue() *4;
 }
 else if(controller.getNumber() ==6){
  rectW = controller.getValue() *2;
 }else if (controller.getNumber() ==7){
 rectH = controller.getValue() *2;
 }
 
 
 println(controller.getValue());
}




Everytime i change a value it draws a new square on top of the old one.  I just want a single square to move around the screen.  How would i go about doing this?  Help please. thanks.

thanks! Also when i hit a note to change color it changes the color but when i release a note the color changes again.  How would i stop that?