ArrayLists freeze up

Programming Questions

dlillard5

in Programming Questions • 1 year ago

This code runs but for some reason when the mouseReleased condition is met it freezes. Just wondering if anybody knows why this happens

ArrayList cx;
ArrayList cy;
void setup(){
size(500,500);
background(50);
smooth();
cx=new ArrayList();
cy=new ArrayList();
}
void draw(){
for(int i=0;i<cx.size()-1;i++){
for(int j=0;i<cy.size()-1;j++){
background(255,0,0);
float ccx= (Float) cx.get(i);
float ccy= (Float) cy.get(i);
stroke(255);
strokeWeight(10);
point(ccx,ccy);
}
}
println(frameCount);
}
void mouseReleased(){
cx.add( new Float(mouseX));
cy.add( new Float(mouseY));
}

Replies(2)

jbum

Re: ArrayLists freeze up

1 year ago

You have some bugs due to cut & paste code.

for(int i=0;i<cx.size()-1;i++){
for(int j=0;i<cy.size()-1;j++){

In your inner loop, you are comparing i to cy.size(), I think you meant

for(int i=0; i<cx.size()-1; i++){
for(int j=0; j<cy.size()-1; j++){

Also, the -1s in those lines (cx.size()-1, etc.) are unnecessary. In fact, I think the whole inner loop is unnecessary.

Another cut & paste bug:

float ccy= (Float) cy.get(j);

You probably meant to use j instead of i.

I'm pretty sure you don't want to be clearing the background color as well... Are you trying to make something like a paint program? If so, I would respond to mouseDragged rather than mouseReleased.

dlillard5

I'm not trying to make a paint program I'm just trying to set up a system where I can mark locations as in use or not... The background was just an attempt to see if anything was happening. Thanks for being patient with me here.

So when this code is run, you get points at all the x's and all the y's. If I wanted to give each coordinate an id, how would you suggest to do that? a class? another ArrayList?

Top Reply