Help with joining two pieces of code.

The code could be structured in the following way:

Loop {
  If (digitalread( pushpin, HIGH) {
     mode != mode // toggles mode on push
   }


   If (mode==1) {
     // code from loop in first example
   } else {
     // code from loop in second example
   }
}

This should give you an idea... Cannot not guarantee accuracy of code. on the road ATM.

Paul

Thanks Paul & Chris,

Ok so I fused both your suggestions together but now it seems there are syntax errors. I tried moving curly braces and parenthesis around but didnt seem to have much luck.

here's what I have right now, how is it looking to you?

//===============================================================

// Global Variables & Constants

//===============================================================

//Init the Pins used for PWM 

const int redPin = 9; // LED1 ANODE

const int greenPin = 10; // LED2 ANODE

const int bluePin = 11; // LED3 ANODE

// Init the Pins used for 10K pots

const int redPotPin = 0;

const int greenPotPin = 1;

const int bluePotPin = 2;

// Init our Vars

int currentColorValueRed;

int currentColorValueGreen;

int currentColorValueBlue;

//Init the Switch

const int modePin = 13; // Active HIGH, held low by 4.7K

int mode = 0; // Selector State (Initial state = ALL OFF)

int val = 0; // Pin 7 HIGH/LOW Status

int butState = 0; // Last Button State

int modeState = 0; // Last Mode State

boolean debug = 1; // 1 = Print Serial Enabled / 0 = disabled

//===============================================================

// SETUP

//===============================================================

void setup () {

pinMode(redPin, OUTPUT);

pinMode(greenPin, OUTPUT);

pinMode(bluePin, OUTPUT);

pinMode(modePin, INPUT);

  // Start off with the LED off.

  setColourRgb(0,0,0);

if (debug){

Serial.begin(9600);

Serial.print("Initial Mode: ");

Serial.println(mode);

Serial.print("Setup Complete\n");

}

}

void setColourRgb(unsigned int red, unsigned int green, unsigned int blue) {

  analogWrite(redPin, red);

  analogWrite(greenPin, green);

  analogWrite(bluePin, blue);

}

void loop(){

  If (digitalRead( modePin, HIGH) { 

     mode != mode // toggles mode on push 

   }

   If (mode==1) { 

     // code from loop in first example 

      // Read the voltage on each analog pin then scale down to 0-255 and inverting the value for common anode

  currentColorValueRed = (255 - map( analogRead(redPotPin), 0, 1024, 0, 255 ) );

  currentColorValueBlue = (255 - map( analogRead(bluePotPin), 0, 1024, 0, 255 ) );

  currentColorValueGreen = (255 - map( analogRead(greenPotPin), 0, 1024, 0, 255 ) );

// Write the color to each pin using PWM and the value gathered above

  analogWrite(redPin, currentColorValueRed);

  analogWrite(bluePin, currentColorValueBlue);

  analogWrite(greenPin, currentColorValueGreen);

   }

   else { 

     // code from loop in second example 

unsigned int rgbColour[3];

  // Start off with red.

  rgbColour[0] = 255;

  rgbColour[1] = 0;

  rgbColour[2] = 0;

  // Choose the colours to increment and decrement.

  for (int decColour = 0; decColour < 3; decColour += 1) {

    int incColour = decColour == 2 ? 0 : decColour + 1;

    // cross-fade the two colours.

    for(int i = 0; i < 255; i += 1) {

      rgbColour[decColour] -= 1;

      rgbColour[incColour] += 1;

      setColourRgb(rgbColour[0], rgbColour[1], rgbColour[2]);

      delay(5);

    }

  }

}

} 

oh and how are you getting code preview on here?

I tried [code] [/code] didnt work;

SimpleOneButton.ino: In function 'void loop()':

SimpleOneButton:59: error: 'If' was not declared in this scope

SimpleOneButton:59: error: expected `;' before 'digitalRead'

SimpleOneButton:63: error: expected `;' before '{' token

SimpleOneButton:77: error: 'else' without a previous 'if'

Code is added in the correct format by clicking the >> button on the toolbar.

WWhen you get an error referring to an expected ';' have a look at the line above, more often than not it will be missing its closing semi-colon.

FOr example

mode != mode

should be

mode != mode;

Guess I'm getting there.. lol

Added a semicolon and made sure they were there where they should be (I think)

more errors now. haha

impleOneButton.ino: In function 'void loop()':

SimpleOneButton:48: error: 'If' was not declared in this scope

SimpleOneButton:48: error: expected `;' before 'digitalRead'

SimpleOneButton:53: error: expected `;' before '{' token

SimpleOneButton:66: error: 'else' without a previous 'if'

//===============================================================
// Global Variables & Constants
//===============================================================

//Init the Pins used for PWM  
const int redPin = 9; // LED1 ANODE
const int greenPin = 10; // LED2 ANODE
const int bluePin = 11; // LED3 ANODE


// Init the Pins used for 10K pots
const int redPotPin = 0;
const int greenPotPin = 1;
const int bluePotPin = 2;


// Init our Vars
int currentColorValueRed;
int currentColorValueGreen;
int currentColorValueBlue;


//Init the Switch 
const int modePin = 13; // Active HIGH, held low by 4.7K

int mode = 0; // Selector State (Initial state = ALL OFF)
int val = 0; // Pin 7 HIGH/LOW Status
int butState = 0; // Last Button State
int modeState = 0; // Last Mode State
boolean debug = 1; // 1 = Print Serial Enabled / 0 = disabled

//===============================================================
// SETUP
//===============================================================


void setup () {
 pinMode(redPin, OUTPUT);
 pinMode(greenPin, OUTPUT);
 pinMode(bluePin, OUTPUT);
 pinMode(modePin, INPUT);
 setColourRgb(0,0,0);
 }


 void setColourRgb(unsigned int red, unsigned int green, unsigned int blue) {
  analogWrite(redPin, red);
  analogWrite(greenPin, green);
  analogWrite(bluePin, blue);
}

void loop(){
If digitalRead( modePin, HIGH) {  
  // toggles mode on push   
  mode != mode;   
   } 
    
   If (mode==1) {  
 // Here we hve chosen our first loop AKA 'Manual Control'  
 // Read the voltage on each analog pin then scale down to 0-255 and inverting the value for common anode
  currentColorValueRed = (255 - map( analogRead(redPotPin), 0, 1024, 0, 255 ) );
  currentColorValueBlue = (255 - map( analogRead(bluePotPin), 0, 1024, 0, 255 ) );
  currentColorValueGreen = (255 - map( analogRead(greenPotPin), 0, 1024, 0, 255 ) );
 // Write the color to each pin using PWM and the value gathered above
  analogWrite(redPin, currentColorValueRed);
  analogWrite(bluePin, currentColorValueBlue);
  analogWrite(greenPin, currentColorValueGreen);
       
   }
  
   else {  
// Here we have chosen out 'Rainbow Cycle'  
      unsigned int rgbColour[3];
// Start off with red.
  rgbColour[0] = 255;
  rgbColour[1] = 0;
  rgbColour[2] = 0;
// Choose the colours to increment and decrement.
  for (int decColour = 0; decColour < 3; decColour += 1) {
    int incColour = decColour == 2 ? 0 : decColour + 1;
// cross-fade the two colours.
    for(int i = 0; i < 255; i += 1) {
      rgbColour[decColour] -= 1;
      rgbColour[incColour] += 1;
      setColourRgb(rgbColour[0], rgbColour[1], rgbColour[2]);
      delay(5);
    }
  }
}
     
   
} 

Ok... A couple of small things, Your 'if' statements need to be in lowercase. You should find that if you replace the capital 'I' with a lowercase one, the syntax colouring will turn orange (a good sign).

Your if conditions should also be wrapped in a pair of brackets, e.g:

if (x==y) {

also you need to change your digitalread command to this format:

digitalRead(inPin)

You don't need to specify wether the pin is HIGH or LOW because you are reading the value not setting it.

Hopefully this will help.

Thanks so much for the tip on capital letters Paul, I see that there were a few of those and I corrected them.

The code now compiles! yay!

however, on boot it just goes straight into the rainbow loop and stays there, pressing the switch doesnt change anything..

Here's how it looks now...

//===============================================================
// Global Variables & Constants
//===============================================================

//Init the Pins used for PWM  
const int redPin = 9; // LED1 ANODE
const int greenPin = 10; // LED2 ANODE
const int bluePin = 11; // LED3 ANODE


// Init the Pins used for 10K pots
const int redPotPin = 0;
const int greenPotPin = 1;
const int bluePotPin = 2;


// Init our Vars
int currentColorValueRed;
int currentColorValueGreen;
int currentColorValueBlue;


//Init the Switch 
const int modePin = 13; // Active HIGH, held low by 4.7K
 int mode = 0; // Selector State (Initial state = ALL OFF)

//===============================================================
// SETUP
//===============================================================


void setup () {
 pinMode(redPin, OUTPUT);
 pinMode(greenPin, OUTPUT);
 pinMode(bluePin, OUTPUT);
 pinMode(modePin, INPUT);
 setColourRgb(0,0,0);
 }


 void setColourRgb(unsigned int red, unsigned int green, unsigned int blue) {
  analogWrite(redPin, red);
  analogWrite(greenPin, green);
  analogWrite(bluePin, blue);
}

void loop(){
if (digitalRead(modePin)) {  
  // toggles mode on push   
  mode != mode;   
   } 
    
   if (mode==1) {  
 // Here we hve chosen our first loop AKA 'Manual Control'  
 // Read the voltage on each analog pin then scale down to 0-255 and inverting the value for common anode
  currentColorValueRed = (255 - map( analogRead(redPotPin), 0, 1024, 0, 255 ) );
  currentColorValueBlue = (255 - map( analogRead(bluePotPin), 0, 1024, 0, 255 ) );
  currentColorValueGreen = (255 - map( analogRead(greenPotPin), 0, 1024, 0, 255 ) );
 // Write the color to each pin using PWM and the value gathered above
  analogWrite(redPin, currentColorValueRed);
  analogWrite(bluePin, currentColorValueBlue);
  analogWrite(greenPin, currentColorValueGreen);
       
   }
  
   else {  
// Here we have chosen out 'Rainbow Cycle'  
   unsigned int rgbColour[3];
// Start off with red.
  rgbColour[0] = 255;
  rgbColour[1] = 0;
  rgbColour[2] = 0;
// Choose the colours to increment and decrement.
  for (int decColour = 0; decColour < 3; decColour += 1) {
    int incColour = decColour == 2 ? 0 : decColour + 1;
// cross-fade the two colours.
    for(int i = 0; i < 255; i += 1) {
      rgbColour[decColour] -= 1;
      rgbColour[incColour] += 1;
      setColourRgb(rgbColour[0], rgbColour[1], rgbColour[2]);
      delay(5);
    }
  }
}
     
   
} 

Try adding a Serial.println("Pressed"); I underneath the mode != mode; line.

This will enable you to see wether the button press is recognised in the serial monitor.

AH! ok.

After doing this, it thinks the button has been pressed. it outputs 'pressed' in the monitor on its own seemingly at the end of the rainbow loop before it starts again.

So it looks like the if statement isnt being read as an if statement?

Or it just thinks its permanantly pressed?

I changed the input pin to number 8 (digital pin) to see if it made any difference, it did not.

Totally stumped. lol

//===============================================================
// Global Variables & Constants
//===============================================================

//Init the Pins used for PWM  
const int redPin = 9; // LED1 ANODE
const int greenPin = 10; // LED2 ANODE
const int bluePin = 11; // LED3 ANODE


// Init the Pins used for 10K pots
const int redPotPin = 0;
const int greenPotPin = 1;
const int bluePotPin = 2;


// Init our Vars
int currentColorValueRed;
int currentColorValueGreen;
int currentColorValueBlue;


//Init the Switch 
const int modePin = 8; // Active HIGH, held low by 4.7K
int mode = 0; // Selector State (Initial state = ALL OFF)

//===============================================================
// SETUP
//===============================================================


void setup () {
 pinMode(redPin, OUTPUT);
 pinMode(greenPin, OUTPUT);
 pinMode(bluePin, OUTPUT);
 pinMode(modePin, INPUT);
 setColourRgb(0,0,0);
 Serial.begin(9600);
 }


 void setColourRgb(unsigned int red, unsigned int green, unsigned int blue) {
  analogWrite(redPin, red);
  analogWrite(greenPin, green);
  analogWrite(bluePin, blue);
}

void loop(){
if (digitalRead(modePin)) {  
  // toggles mode on push   
  mode != mode;  
 Serial.println("Pressed");  
   } 
    
   if (mode==1) {  
 // Here we hve chosen our first loop AKA 'Manual Control'  
 // Read the voltage on each analog pin then scale down to 0-255 and inverting the value for common anode
  currentColorValueRed = (255 - map( analogRead(redPotPin), 0, 1024, 0, 255 ) );
  currentColorValueBlue = (255 - map( analogRead(bluePotPin), 0, 1024, 0, 255 ) );
  currentColorValueGreen = (255 - map( analogRead(greenPotPin), 0, 1024, 0, 255 ) );
 // Write the color to each pin using PWM and the value gathered above
  analogWrite(redPin, currentColorValueRed);
  analogWrite(bluePin, currentColorValueBlue);
  analogWrite(greenPin, currentColorValueGreen);
       
   }
  
   else {  
// Here we have chosen out 'Rainbow Cycle'  
   unsigned int rgbColour[3];
// Start off with red.
  rgbColour[0] = 255;
  rgbColour[1] = 0;
  rgbColour[2] = 0;
// Choose the colours to increment and decrement.
  for (int decColour = 0; decColour < 3; decColour += 1) {
    int incColour = decColour == 2 ? 0 : decColour + 1;
// cross-fade the two colours.
    for(int i = 0; i < 255; i += 1) {
      rgbColour[decColour] -= 1;
      rgbColour[incColour] += 1;
      setColourRgb(rgbColour[0], rgbColour[1], rgbColour[2]);
      delay(5);
    }
  }
}
     
   
}