Line won't keep still in the draw () method!!!!

Programming Questions

beth45

Line won't keep still in the draw () method!!!!

in Programming Questions • 1 years ago

I created four patterns in a test sketch that did not display them in the draw() method. When I brought them into my main program to display them in the draw method, one pattern (which is just a still wavy line) became an animated pattern and it won't keep still!!! (even when I tell it too). Can someone help me with this?

RandomColorPalette myRandomColorPalette;
Stitches myCrossStitches;
Stitches myFlatStitches;
Stitches myDottedStitches;
Stitches myWavyStitches;

void setup() { // call the setup function to define initial enviroment properties

size(900, 700);
background(230);

myRandomColorPalette=new RandomColorPalette (color(255, 255, 255), 100, 0, 0);

myCrossStitches=new Stitches (color(227, 101, 221), 100, 600, 0, 10);
myFlatStitches=new Stitches (color(250, 100, 90), 100, 400, 0, 10, 6.6);
myDottedStitches=new Stitches (color(227, 208, 140), 100, 200, 0, 10, 4);
myWavyStitches=new Stitches (color(125, 225, 30), 100, 800, 0, 60.0, 40.0, 3.0);

} // end the setup function

void draw () { // call the draw function

background(230);

myRandomColorPalette.display();
myCrossStitches.display(10);
myFlatStitches.display(10, 6.6);
myDottedStitches.display(10, 4);
myWavyStitches.display(60.0, 40.0, 40.0);

} // end the draw function

abstract class Grid {

float sizeOfSquares;
color gridClr;
float boxPosX;
float boxPosY;

Grid(color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY) {

    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
} // end of the constructor

void display () {

    // initialization of quilt patterns (from Shiffman page 261)

    for (int i=0;i<width;i+=sizeOfSquares) {
      for (int j=0;j<height;j+=sizeOfSquares) {
        fill(gridClr);
        rect(i, j, sizeOfSquares, sizeOfSquares);
      }
    }
} // end of the display function

boolean mouseClickOverBox () {

    if (mousePressed && ((mouseX>boxPosX) && (mouseX<boxPosX+sizeOfSquares) && (mouseY>boxPosY) && (mouseY<boxPosY+sizeOfSquares))) {
      return true;
    }
    else {
      return false;
    }
}
}

class RandomColorPalette extends Grid {

RandomColorPalette(color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
}

void display () {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);

    if (mouseClickOverBox()) {
      gridClr=(color(random(1, 255), random(1, 255), random(1, 255)));

      fill (gridClr);
      rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    }
}
}

class Stitches extends RandomColorPalette {

float clrLine;
float strokeClr;
int stitchLength;
float spacing;
int dotSize;
float stitchOffset;
float stitchSpeed;
float stitchScalar;
float stitchAngle;
float stitchX;

// Constructor overloading for four different stitch patterns

// constructor for cross stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempStitchLength) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempStitchLength= constrain (tempStitchLength, 8, 12);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;

    stitchLength=tempStitchLength;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for flat stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempStitchLength, float tempSpacing) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);
    tempStitchLength= constrain (tempStitchLength, 8, 12);
    tempSpacing= constrain (tempSpacing, tempStitchLength/5, tempStitchLength*4/5);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    spacing=tempSpacing;
    stitchLength=tempStitchLength;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for dotted stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempSpacing, int tempDotSize) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempSpacing= constrain (tempSpacing, 8, 12);
    tempDotSize=constrain(tempDotSize, 1, 4);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    dotSize=tempDotSize;
    spacing=tempSpacing;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for the wavy stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, float tempStitchOffset, float tempStitchSpeed, float tempStitchScalar) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempStitchOffset=constrain (tempStitchOffset, 8, 12);
    tempStitchSpeed=constrain(tempStitchSpeed, 1, 4);
    tempStitchScalar=constrain(tempStitchScalar, 1, 4);
    stitchAngle=90;
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    stitchOffset=tempStitchOffset;
    stitchSpeed=tempStitchSpeed;
    stitchScalar= tempStitchScalar;
    stitchX=boxPosX+sizeOfSquares/2;
}

// Method overloading for display function of four different stitch patterns

// display () cross stitch

void display (int stitchLength) {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);

    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {
      stroke(0);
      line (stitchX, i, stitchX+stitchLength, i+stitchLength);
      line (stitchX, i+stitchLength, stitchX+stitchLength, i);
    }
} // end the display () function

// display () flat stitch

void display (int stitchLength, float spacing) {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);

    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {

      stroke(0);
      line (stitchX, i, stitchX, sizeOfSquares);
      stroke (gridClr);
      line (stitchX, i+spacing, stitchX, sizeOfSquares);
    }
} // end the display () function

// display () dotted stitch

void display (int stitchLength, int dotSize) {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {       // loop that will use a counter that will draw a shape every 8 pixels for the whole
      // height of the canvas
      stroke (strokeClr);       // determine the color of the stroke of the ellipses that form the dotted line as purple
      fill (clrLine);       // fill with purple the ellipses that form the dotted lines
      ellipse (stitchX, i, dotSize, dotSize);       // draw little ellipses every 8 pixels from top to bottom on the left of
      // the canvas to make up the dotted line
    }       // end the for loop
} // end the display () function

// display () wavy stitch

void display (float StitchOffset, float StitchSpeed, float StitchScalar) {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);

    for (float i=boxPosY; i<sizeOfSquares; i+=stitchScalar) {
      float waveStitch=stitchOffset+cos(radians(stitchAngle))*stitchScalar;
      stitchAngle+=stitchSpeed;
      stroke(0);
      strokeWeight (3);
      fill (0);
      point (stitchX+waveStitch, i);
    }       // end the for loop
   } // end the display () function
} // end Stitches class

Replies(11)

PhiLho

Re: Line won't keep still in the draw () method!!!!

1 years ago

stitchAngle+=stitchSpeed;
This is likely to be the moving part...

Note: your design of having various overloaded constructors and display methods is... original but brittle. What if you need another pattern needing the same kind of parameters in one of these functions?
Beside, it forces the Stitches class to hold fields than are not used in some variants.

It is better to make a common, perhaps abstract class to these classes, holding the fields (variables) they have in common, perhaps common methods (functions), and to extend it, making four distinct classes, each with its own, specific fields/methods.

Beside, I will question the hierarchy of classes you make.
Grid seems misnamed, it is really a simple box, the name introduce confusion.
And Stitches (why the plural?) are not RandomColorPalette. The latter introduces no new variable (over Grid) nor new methods used by Stitches, so beside Grid, they have nothing in common (since Stitches' display() don't call super()).
So Stitches should extends Grid instead.

beth45

would appreciate any input!!! Re: Re: Line won't keep still in the draw () method!!!!

1 years ago

Hi,

I changed the names of my variables like you suggested and played around with the code but I still can't figure it out. I am not a coder. I am an artist learning how to code. I don't think like a coder so I am quite lost. I would appreciate any input you could give me. Re: The Stitches. I agree that they should be in different classes but in this case I may have to do it this way because I have been instructed to use method and constructor overloading in at least one class so I may leave them that way.

beth45

Re: Line won't keep still in the draw () method!!!!

1 years ago

The reason Stitches extends Random ColorPalette is because, if you click on the stitch box, it will generate a random color which I will ...attempt!... to have the user pick as a stitch color. This is a feature I added after I posted.

Grid may not be the best name for the abstract class indeed. Maybe Boxes would have been better or grid Squares or something like that but basically it lays out the idea that everything is a function of the size and placement of these squares in the program.

As for making an abstract stitches class, I did not think about it but you are right. I am not sure i will have time to change that though... Here is what I have with this half of my program. If you click on any of the boxes, you will see it change color. Ideally, I would want this feature with my pattern boxes too butv I don't know if I'll be able to pull that off.

RandomColorPalette myRandomColorPalette;
Stitches myFlatStitches;
Stitches myDoubleFlatStitches;
Stitches myCrossStitches;
Stitches myDottedStitches;
/*Stitches myWavyStitches;*/

void setup() { // call the setup function to define initial enviroment properties

size(900, 700); // set the canvas size at 800 pixels by 800 pixels
background(230); // set background color to light gray

myRandomColorPalette=new RandomColorPalette (color(255, 255, 255), 100, 0, 0);

myFlatStitches=new Stitches (color(250, 100, 90), 100, 200, 0, 10, 6.6);
myDoubleFlatStitches=new Stitches (color(250, 100, 90), 100, 400, 0, 10, 5.0, 6.6);
myDottedStitches=new Stitches (color(227, 208, 140), 100, 600, 0, 10, 4);
myCrossStitches=new Stitches (color(227, 101, 221), 100, 800, 0, 10);
/* myWavyStitches=new Stitches (color(125, 225, 30), 100, 800, 0, 40.0, 3.0);*/
} // end the setup function

void draw () { // call the draw function

myRandomColorPalette.display();

myFlatStitches.display();
myDoubleFlatStitches.display ();
myCrossStitches.display();
myDottedStitches.display();
/*myWavyStitches.display();*/

myFlatStitches.showFlatStitch(10, 6.6);
myDoubleFlatStitches.showDoubleFlatStitch (10, 5.0, 6.6);
myCrossStitches.showCrossStitch(10);
myDottedStitches.showDottedStitch(10, 4);
/*myWavyStitches.display(40.0, 3.0);*/

myFlatStitches.drawFlatStitch(10, 6.6);
myDoubleFlatStitches.drawDoubleFlatStitch (10, 5.0, 6.6);
myCrossStitches.drawCrossStitch (10);
myDottedStitches.drawDottedStitch(10, 4);
   /*myWavyStitches.drawWavyStitch(40.0, 3.0);*/

} // end the draw function

abstract class Grid {

float sizeOfSquares;
color gridClr;
float boxPosX;
float boxPosY;

Grid(color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY) {

    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
} // end of the constructor

void display () {

    // initialization of quilt patterns (from Shiffman page 261)
    for (int i=0;i<width;i+=sizeOfSquares) {
      for (int j=0;j<height;j+=sizeOfSquares) {
        fill(gridClr);
        rect(i, j, sizeOfSquares, sizeOfSquares);
      }
    }
} // end of the display function

boolean mouseClickOverBox () {
    if (mousePressed && ((mouseX>boxPosX) && (mouseX<boxPosX+sizeOfSquares) && (mouseY>boxPosY) && (mouseY<boxPosY+sizeOfSquares))) {
      return true;
    }
    else {
      return false;
    }
}
}

class RandomColorPalette extends Grid {

RandomColorPalette(color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
}

void display () {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    if (mouseClickOverBox()) {
      gridClr=(color(random(125, 255), random(125, 255), random(125, 255)));
      fill (gridClr);
      rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    }
}
}

class Stitches extends RandomColorPalette {

float clrLine;
float strokeClr;
int stitchLength;
float halfStitchLength;
float spacing;
int dotSize;
float stitchOffset;
float stitchSpeed;
float stitchScalar;
float stitchAngle;
float stitchX;

// Constructor overloading for different stitch patterns

// constructor for flat stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempStitchLength, float tempSpacing) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempStitchLength= constrain (tempStitchLength, 8, 12);
    tempSpacing= constrain (tempSpacing, tempStitchLength/5, tempStitchLength*4/5);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    spacing=tempSpacing;
    stitchLength=tempStitchLength;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for double flat stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempStitchLength, float tempHalfStitchLength, float tempSpacing) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempStitchLength= constrain (tempStitchLength, 8, 12);
    tempSpacing= constrain (tempSpacing, tempStitchLength/5, tempStitchLength*4/5);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    spacing=tempSpacing;
    stitchLength=tempStitchLength;
    halfStitchLength=tempHalfStitchLength;
    // halfStitchLength=stitchLength/2;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for cross stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempStitchLength) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempStitchLength= constrain (tempStitchLength, 8, 12);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    stitchLength=tempStitchLength;
    stitchX=boxPosX+sizeOfSquares/2;
}

// constructor for dotted stitch

Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, int tempSpacing, int tempDotSize) {

    super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    tempSpacing= constrain (tempSpacing, 8, 12);
    tempDotSize=constrain(tempDotSize, 1, 4);
    gridClr=tempgridClr;
    sizeOfSquares=tempsizeOfSquares;
    boxPosX=tempBoxPosX;
    boxPosY=tempBoxPosY;
    dotSize=tempDotSize;
    spacing=tempSpacing;
    stitchX=boxPosX+sizeOfSquares/2;
}

/* // constructor for the wavy stitch

   Stitches (color tempgridClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, float tempStitchSpeed, float tempStitchScalar) {

   super (tempgridClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

   tempStitchSpeed=constrain(tempStitchSpeed, 20, 60);
   tempStitchScalar=constrain(tempStitchScalar, 3, 6);
   stitchAngle=0.0;
   gridClr=tempgridClr;
   sizeOfSquares=tempsizeOfSquares;
   boxPosX=tempBoxPosX;
   boxPosY=tempBoxPosY;
   stitchSpeed=tempStitchSpeed;
   stitchScalar= tempStitchScalar;
   stitchX=boxPosX+sizeOfSquares/2;
   } */

// Method overloading for display function of different stitch patterns

// display () flat stitch

void display () {
    stroke(0);
    fill(gridClr);
    rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    if (mouseClickOverBox()) {
      gridClr=(color(random(1, 255), random(1, 255), random(1, 255)));
      fill (gridClr);
      rect (boxPosX, boxPosY, sizeOfSquares, sizeOfSquares);
    }
}   // end the display () function

// showFlatStitch ()

void showFlatStitch (int stitchLength, float spacing) {
    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {
      stroke(0);
      line (stitchX, i, stitchX, sizeOfSquares);

      stroke (gridClr);
      line (stitchX, i+spacing, stitchX, sizeOfSquares);
    }
} // end the showFlatStitch () function

// showDoubleFlatStitch ()

void showDoubleFlatStitch (int stitchLength, float halfStitchLength, float spacing) {
    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {
      stroke(0);
      line (stitchX, i, stitchX, sizeOfSquares);
      line (stitchX+stitchLength/2, i, stitchX+stitchLength/2, sizeOfSquares);
      stroke (gridClr);
      line (stitchX, i+spacing, stitchX, sizeOfSquares);
      line (stitchX+stitchLength/2, i+spacing, stitchX+stitchLength/2, sizeOfSquares);
    }
} // end the showDoubleFlatStitch () function

// showDottedStitch ()

void showDottedStitch (int stitchLength, int dotSize) {
    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {
      stroke (strokeClr);
      fill (clrLine);
      ellipse (stitchX, i, dotSize, dotSize);
    }
}

// showCrossStitch ()

void showCrossStitch (int stitchLength) {
    for (float i=boxPosY; i<sizeOfSquares; i+=stitchLength) {
      stroke(0);
      line (stitchX, i, stitchX+stitchLength, i+stitchLength);
      line (stitchX, i+stitchLength, stitchX+stitchLength, i);
    }
}

/* // drawWavyStitch ()

   void drawWavyStitch (float stitchSpeed, float stitchScalar) {
   for (float i=boxPosY; i<sizeOfSquares; i+=stitchScalar) {
   float waveStitch=cos(radians(stitchAngle))*stitchScalar;
   stitchAngle+=stitchSpeed;
   stroke(0);
   strokeWeight (3);
   fill (0);
   point (stitchX+waveStitch, i);
   }       // end the for loop
   } // end the drawWavyStitch () function   */

// Method overloading for drawing tool of different stitch patterns

// drawFlatStitch ()

void drawFlatStitch (int stitchLength, float spacing) {
    if ((keyPressed) && (key == 'a' || key == 'A') && mousePressed && (dist(pmouseX, pmouseY, mouseX, mouseY) >= stitchLength)) {
      stroke(0);

      stroke(0);
      line (pmouseX, pmouseY, mouseX, mouseY);

      stroke (230);
      line (pmouseX+spacing/3, pmouseY+spacing/3, mouseX, mouseY);
    }
} // end the drawFlatStitch () function

// drawDoubleFlatStitch ()

void drawDoubleFlatStitch (int stitchLength, float halfStitchLength, float spacing) {
    if ((keyPressed) && (key == 'b' || key == 'B') && mousePressed && (dist(pmouseX, pmouseY, mouseX, mouseY) >= stitchLength)) {
      stroke(0);
      line (pmouseX, pmouseY, mouseX+spacing, mouseY+spacing);
      stroke (230);
      line (pmouseX+spacing, pmouseY+spacing, mouseX, mouseY);
    }
} // end the doubleFlatStitch () function

// drawCrossStitch ()

void drawCrossStitch (int stitchLength) {
    if ((keyPressed) && (key == 'c' || key == 'C') && mousePressed && (dist(pmouseX, pmouseY, mouseX, mouseY) >= stitchLength)) {
      stroke (0);       // determine the color of the stroke of the ellipses that form the dotted line as purple
      line (mouseX, mouseY, mouseX+stitchLength*3/4, mouseY+stitchLength*3/4);
      line (mouseX, mouseY+stitchLength*3/4, mouseX+stitchLength*3/4, mouseY);
    }       // end the if condition block of code
} // end the drawDottedStitch () function

// drawDottedStitch ()

void drawDottedStitch (int stitchLength, int dotSize) {
    if ((keyPressed) && (key == 'd' || key == 'D')&& mousePressed && (dist(pmouseX, pmouseY, mouseX, mouseY) >= stitchLength)) {
      stroke (0);       // determine the color of the stroke of the ellipses that form the dotted line as purple
      fill (0);       // fill with purple the ellipses that form the dotted lines
      ellipse (mouseX, mouseY, dotSize, dotSize);       // draw little ellipses every n pixels
    }       // end the if condition block of code
} // end the drawDottedStitch () function
} // end Stitches class

PhiLho

Re: Line won't keep still in the draw () method!!!!

1 years ago

If the current structure is part of your assignment, that's OK, I suppose. I just hope the teacher will show better alternatives...

" I still can't figure it out"
Well, I saw no questions in the last message, so I felt no answer was needed.
Beside, you made multiple topics around the same code, it is hard to keep track of progress/needs...

beth45

Re: Re: Line won't keep still in the draw () method!!!!

1 years ago

K. I actually reworked the code and found a way. This is how I organized my Pattern class. Is this a logical and efficient way to do so (in terms of how I integrate the PImage)?

Currently when I click on the Pattern tiles (there are four), the colors change on all of the (but different colors) to a random color only when I click on the top one. I can't figure out why? My objective is to to have each one of them individually change color when I click on it (and nothing happens to the others).

class Pattern extends RandomColorPalette {

PImage quiltPattern;

Pattern(color tempboxClr, float tempsizeOfSquares, float tempBoxPosX, float tempBoxPosY, PImage tempQuiltPattern) {

    super (tempboxClr, tempsizeOfSquares, tempBoxPosX, tempBoxPosY);

    boxClr=tempboxClr;
    sizeOfSquares=tempsizeOfSquares;
    quiltPattern=tempQuiltPattern;
} // end of the constructor

void showPatternTiles() {

    for (int q = 0; q < hawaiianPattern.length; q++) {
      image (hawaiianPattern[q], 0, ((q+1)*(sizeOfSquares+50)), sizeOfSquares, sizeOfSquares);
    if (mousePressed && ((mouseX>boxPosX) && (mouseX<boxPosX+sizeOfSquares) && (mouseY>boxPosY) && (mouseY<boxPosY+sizeOfSquares))) {
        color patternTint=(color(random(125, 255), random(125, 255), random(125, 255)));
        tint (patternTint);
        image (hawaiianPattern[q], 0, ((q+1)*(sizeOfSquares+50)), sizeOfSquares, sizeOfSquares);
      }
     }
} // end of display () mode
} // end of Pattern class

PhiLho

Re: Line won't keep still in the draw () method!!!!

1 years ago

Looks OK, although I don't see how you use the quiltPattern field. No need to force it into the class if you really are just using the global array. My remarks are quite generic and can lack insight into your plans and intents.

beth45

Re: Line won't keep still in the draw () method!!!!

1 years ago

What I don't understand is why all my patterned tiles are tinted when I click on any of them (I want this to happen one at a time).

This is is draw():

for (int q = 0; q < myPatternTile.length; q++) { // create an array that loads the array of quilt patterns as numbered files
    myPatternTile[q].showPatternTiles();
}

This is in my Pattern class:

void showPatternTiles() {
image (quiltPattern, boxPosX, boxPosY);
if (mouseClickOverBox()) {
      tint (color(random(175, 255), random(175, 255), random(175, 255)));
    }
    image (quiltPattern, boxPosX, boxPosY);
} // end of showPatternTiles () mode

PhiLho

Re: Line won't keep still in the draw () method!!!!

1 years ago

if (mouseClickOverBox()) {
tint (color(random(175, 255), random(175, 255), random(175, 255)));
} else {
noTint();
}
The tint() acts on all subsequent drawings.

beth45

Re: Re: Line won't keep still in the draw () method!!!!

1 years ago

Tried it and although only the box I click gets a tint on it, that tint only stays during the time mouse is Pressed.

PhiLho

Re: Line won't keep still in the draw () method!!!!

1 years ago

Then, you need a special field (variable at the class level), for example one to hold the color to use in tint(). Let's name it tintColor, for example.

void showPatternTiles() {
image (quiltPattern, boxPosX, boxPosY);
if (tintColor != 0) {
    tint(tintColor);
} else {
    noTint();
}
image (quiltPattern, boxPosX, boxPosY);
} // end of showPatternTiles() method

But then, you have to do the test at a more global level, because you want a switch behavior: when clicking on a box, the others loose their tint.

for (int q = 0; q < myPatternTile.length; q++) {
if (myPatternTile[q].mouseClickOverBox()) {
    myPatternTile[q].tintColor = color(random(175, 255), random(175, 255), random(175, 255));
    for (int r = 0; r < myPatternTile.length; r++) {
      if (r != q) myPatternTile[q].tintColor = 0; // Reset the other tint colors
    }
}
}
for (int q = 0; q < myPatternTile.length; q++) { // create an array that loads the array of quilt patterns as numbered files
myPatternTile[q].showPatternTiles();
}

(untested)

beth45

Re: Re: Line won't keep still in the draw () method!!!!

1 years ago

I will try this but since this project has to be done by tomorrow night, I will have to solve my other problems with it first.

Because I have only been coding since Sept, I just created the most disorganized and badly structured program ever made. It's sort of spaghetti code but it is working just enough for me to try to finish it for tomorrow night (it's due then).

I am going to make a new post with a couple of questions. They are probably easy to solve but the syntax and logic I need to use are such a mystery to me that I keep hitting my head against the wall.

I know I should be reorganize my code in a more logical structure but given my delay, I have to work with it as it is for now.

I'll post this new post in a few minutes. Thanks for the tint (pun on thanks for the tip). I will use it when I reorganize my code for a larger project.

Top Reply