HELP: led strip problem

Using Arduino LEDs and Multiplexing
1

Hello. I have a problem with my led strip code. I want it to be controlled with bluetooth. But I cannot figure out that whats wrong with my code. I enclose the code. Thank you.

#include "FastLED.h"

#define DATA_PIN 10
#define CHIPSET WS2812
#define NUM_LEDS 30
struct CRGB  leds[NUM_LEDS];
int myleds_white1 [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15};
int myleds_black1 [] = {16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30};
int myleds_red [] = {16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30};
int myleds_black2 [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15};
int myleds_full [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15,
                    16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30
                   };
char data;

void setup() {
FastLED.addLeds<NEOPIXEL, DATA_PIN>(leds, NUM_LEDS);
FastLED.setBrightness(100);
FastLED.clear();
Serial.begin(9600);
}


void loop() {
data = read_bluetooth();


Serial.println(data);
if (data != '@') {

  switch (data) {

    case '7':
      ledstrip_on_full();  //all leds on
      break;

  case '8':
      ledstrip_off();  //all leds off
      break;

 case '9':
      ledstrip_on_white();   // white
      break;
   
 case 'a':
      ledstrip_on_red();    //red
      break;

case 'b':
      ledstrip_off(); //off
      break;
  }
}
void ledstrip_on_full() { //all leds on


  for (int i = 0; i < 30; i++)
  {
    leds[myleds_full[i] ] = CRGB::White //all 30 leds on ligthing white


  }
  FastLED.show();
  delay (15);
}

if (read_bluetooth() == 'b') {
  ledstrip_off();
  break;
}

void ledstrip_on_red() {  //red on
  for (int i = 0; i = 30; i++) {
    leds[myleds_black2[i] ] = CRGB::Black;   //0-15 leds off
    leds[ myleds_red[i] ] = CRGB::Red;      //16-30leds on
    FastLED.show();
    delay(150);
 }

  if (read_bluetooth() == 'b') {
    ledstrip_off();
    break;
  }
}
void  ledstrip_on_white //white on
for (int i = 0; i <= 30; i++) {
  leds[myleds_black1[i] ] = CRGB::Black; //first 15 black
  leds[ myleds_white1[i] ] = CRGB::White; //16-30 leds turn off white
  FastLED.show();
  delay(150);
}

ledstrip_off();
}

void ledstrip_off() {
for (int i = 0; i <= 30; i++) {
  leds[i] = CRGB::Black;   //all leds OFF
  FastLED.show();
}
}


char read_bluetooth() {
char msg;
if (Serial.available()) {


  msg = Serial.read();
} else msg = '@';
return msg;
}
2

It appears OK, it is not documented, no explanation of your problem, With that information it must be OK!

3

It seems that you have the Bluetooth module on Serial and want to print to the serial monitor, too. Using one serial port for the two devices can be problematic. I use a software serial port (on Uno, there are several libraries available) or spare hardware port (Mega) for the Bluetooth and keep Serial (USB) for upload, program output and debug prints and monitoring variable values.

char read_bluetooth() 
{
  char msg;
  if (Serial.available()) 
  {
     msg = Serial.read();
  } 
  else 
  {
     msg = '@';
  }
  return msg;
}

If you Serial print the value of msg what do you get?

4

Use tools -> autoformat in the IDE. After that check the indentations; functions should start at the beginning of a line. E.g. below does not start at the beginning of a line after an autoformat.

  void ledstrip_on_full() { //all leds on

This indicates a mismatch in { and } (you seem to have a few of them) in the part before that. Fix that and your code will at least compile.

5

Hello sorry. I wrote documentation in the programme. And my problem is that i cannot switch between the different modes. and i getting this error
arduino_leds--:96:19: error: 'i' was not declared in this scope
exit status 1
'ledstrip_on_full' was not declared in this scope

6

Use the IDE autoformat tool (ctrl-t or Tools, Auto Format) and you can see where there are several missing and out of place curly brackets. Also if you put the cursor to the right of a bracket or parenthesis the IDE will mark the matching bracket or parenthesis. Plus many more syntax errors and many warnings.

7

This compiles, but still has many warnings. I don't have time to chase them all down. If you have not already done so, enable compile warnings in the IDE, File, Preferences menu to see the warnings when you compile.
See the comments marked with *******.

#include "FastLED.h"

#define DATA_PIN 10
#define CHIPSET WS2812
#define NUM_LEDS 30
struct CRGB  leds[NUM_LEDS];
int myleds_white1 [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15};
int myleds_black1 [] = {16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30};
int myleds_red [] = {16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30};
int myleds_black2 [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15};
int myleds_full [] = {0, 1, 2, 3, 45, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15,
                      16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30
                     };
char data;

void setup()
{
   FastLED.addLeds<NEOPIXEL, DATA_PIN>(leds, NUM_LEDS);
   FastLED.setBrightness(100);
   FastLED.clear();
   Serial.begin(9600);
}


void loop()
{
   data = read_bluetooth();


   Serial.println(data);
   if (data != '@')
   {

      switch (data)
      {

         case '7':
            ledstrip_on_full();  //all leds on
            break;

         case '8':
            ledstrip_off();  //all leds off
            break;

         case '9':
            ledstrip_on_white();   // white
            break;

         case 'a':
            ledstrip_on_red();    //red
            break;

         case 'b':
            ledstrip_off(); //off
            break;
      }
   }
}  // ****************** added to close loop()

void ledstrip_on_full()   //all leds on
{
   for (int i = 0; i < 30; i++)
   {
      //leds[myleds_full[i] ] = CRGB::White //all 30 leds on ligthing white
      leds[myleds_full[i]] = CRGB::White; //******* fixed syntax mistake missing ;
   }
   FastLED.show();
   delay (15);
   //} *************  this } is out of place

   if (read_bluetooth() == 'b')
   {
      ledstrip_off();
      // break; ************* break statement not within loop or switch
   }
} // ************** added to close ledstrip_on_full()

void ledstrip_on_red()    //red on
{
   for (int i = 0; i == 30; i++)  // ********** = for assignment == for comparison
   {
      leds[myleds_black2[i] ] = CRGB::Black;   //0-15 leds off
      leds[ myleds_red[i] ] = CRGB::Red;      //16-30leds on
      FastLED.show();
      delay(150);
   }

   if (read_bluetooth() == 'b')
   {
      ledstrip_off();
      //break;  ************** break statement not within loop or switch
   }
}

void  ledstrip_on_white() //white on  ******* added ()
{ // ***************** added {
   for (int i = 0; i <= 30; i++)
   {
      leds[myleds_black1[i] ] = CRGB::Black; //first 15 black
      leds[ myleds_white1[i] ] = CRGB::White; //16-30 leds turn off white
      FastLED.show();
      delay(150);
   }
   ledstrip_off();
}

void ledstrip_off()
{
   for (int i = 0; i <= 30; i++)
   {
      leds[i] = CRGB::Black;   //all leds OFF
      FastLED.show();
   }
}


char read_bluetooth()
{
   char msg;
   if (Serial.available())
   {
      msg = Serial.read();
   }
   else msg = '@';
   return msg;
}