How to reset buffer

I am using the colorduino library(8x8 matirx)

I have this in the loop:

void loop()
{
  setTest();
  
  
  if (Serial.available() > 0) 
  {
     unsigned char inByte = Serial.read();  
     // Serial.println (inByte);
     // Serial.flush();
  
    switch(inByte)
    {
      Serial.flush();
    case 'a':  
    {
    
    
      for( unsigned char i = 0; i < 10 ; i++)
      {       
 
      setHeart();
      delay(100);
      setHeart2();
      delay(100);     
      break;      
     }//End for
    } 

  default:  
  {  
     for( unsigned char i = 0; i< 1 ; i++)
     {
        setTest();      
       delay(1000);      
       break;
      }
  }
 //End for    
 
  
    }//End switch
   
    }
 
 
 //  Serial.flush();
 
 
 
  
  //*******************************END TEST**************************************************
  /*
 //****************************************BEGIN INPUT KEYBOARD**********************************
  //Serial.flush();
  
  if (Serial.available() > 0) {
    // get incoming byte:
    inByte = Serial.read();  
    switch(inByte)
    {
    case 'a':   
    
 //if(millis() > stateEnd)
 // { 
    Serial.println(" \na is pressed ");
   // stateStart = millis();
   // stateEnd = stateStart + 1000;
    //Serial.flush();
    //initializeColorduino();
    //setHeart(); 
    //setSmiley();
    // delay(1000);
    setTest();
   // setSmiley2();
    //delay(1000);
    //setSmiley();  
      break;
  
    case 'b': 
    
    //Serial.flush();
     // plasma_morph2();
    // Serial.flush();
     Serial.println(" \nb is pressed ");
    // initializeColorduino();
     setSmiley();//Showing the heart
      break;  
  
     case 'c':   
     
    // Serial.flush();
    Serial.println("\nc is pressed") ;
    setSmiley2();
    break;
    
    case 'd':  
     Serial.println("\nd is pressed");
     setAngry();
     break;
  
  case 'e':  
     Serial.println("\nd is pressed");
     setSmiley2();
    break;
      
    default:
    {
      setAngry();
      Serial.println(" \nWrong character, IDIOT!! ");   // send back the value received.
      break;
    }
    }//End switch    
    Serial.flush();
    delay(1000);    
    Serial.print(inByte);   // send back the value received.     
  }
  //***************************END INPUT KEYBOARD*******************************************
  */ 
 
}//End Loop

but is doesnt do the pattern for 10 times

Your ‘a’ case falls straight into your default case - is that what you intended?

You have “Serial.flush” inside the switch but not associated with any case (that’s not your problem here, but it looks odd)

HI awol,

thx for ur quick replay.

I want to reset the buffer, because otherwise the images overlap eatch other.

what do u mean with: that is not my probleme here?

My intention is just that u press 'a' - for now it is 1 but I have more annimations that u will see 10 times the heart annimation.

THX anyway

You break out of your "for" loop on the first time through - if you only want one iteration, you don't need a "for" loop.

I want it 10 times. I mean I have now only 'a' but normally I have 'b', 'c' etc.

But it must 10 times.

It goes now only 2 times. not 10 times.

and it has to do 10 times - like in the code

But I have some problems with the buffer.

Can u give me an example.

thx

See reply #3, in particular the second word.

??

did u test it.

Then u can see it by urself. that it doesnt reset the buffer.

void loop()
{
  setTest();
  
  
  if (Serial.available() > 0) 
  {
     unsigned char inByte = Serial.read();  
     // Serial.println (inByte);
     // Serial.flush();
  
    switch(inByte)
    {
       Serial.flush();
      
    case 'a':  
    {
   
    
      for( unsigned char i = 0; i < 10 ; i++)
      {       
 
      setHeart();
      delay(100);
      setHeart2();
      delay(100);     
      break;      
     }//End for
    }
   
   case 'b':
  {
    for(unsigned char i = 0; i < 10; i++)
    {
      setSmiley();
      delay(500);
      setSmiley2();
      delay(500);
      break;
    }
  } 

  default:  
  {      
       setTest();      
       delay(1000);      
       break;      
  }
 //End for    
 
  
    }//End switch
   
    }
    }//End Loop
 
 
 //  Serial.flush();

if u test it u see it by urself that is doesnt work correct.

the images are overlapping

(Can you please stop typing in txt - it is really hard to read.)

I can’t test it, because I don’t have your hardware.
However, you do, so you tell me what this bit of code does:

    for(unsigned char i = 0; i < 10; i++)
    {
      setSmiley();
      delay(500);
      setSmiley2();
      delay(500);
      break;
    }

HI Awol,

sorry then.

But u have to get the hardware it is the only way to test it.

then u see it by urself.

it has to do it with the buffer.

The images are overlapping eatch other.

I won't be getting any hardware any time soon, but will you be looking at that "break"?

The break statement has a number of uses. Using break inside a for loop or a while loop causes the loop to stop looping, regardless of the continue condition.

A break statement is also required at the end of each case statement in a switch statement, to break out of the switch statement.

Move the break statement out of the for loop. Put it after the for loop, at the end of the case section.

And, if you don't start spelling out words, and using proper capitalization and punctuation, you will not be getting more help here.

Is this line correct for what you want?

 for( unsigned char i = 0; i < 10 ; i++)

Or should it be?

 for( unsigned char  i < 10 ; i++)

“modified my obvious error out.”

Or should it be?

Code:
for( unsigned char i == 0 & i < 10 ; i++)

:astonished:

AWOL:

Or should it be?

Code:
for( unsigned char i == 0 & i < 10 ; i++)

:astonished:

Doh! I did it again. I need to be more awake before I post impossible for loops!

Mak