Expected identifier before numeric constant

Projects Programming
1

Hi. My code has been working until I break down the classes. It kept prompte me Expected identifier before numeric constant

#include <Adafruit_NeoPixel.h>
#ifdef __AVR__
 #include <avr/power.h> // Required for 16 MHz Adafruit Trinket
#endif
#define LED_PIN_1    13
#define LED_PIN_2    12
#define LED_PIN_3    14
#define LED_PIN_4    27
#define LED_PIN_5    26
#define LED_PIN_6    25
#define LED_PIN_7    33

 
#define LED_COUNT     90
#define LED_COUNT_7   30
#define BRIGHTNESS   50

class Lighting
{
  public:
  const short int width= 30;
  const short int len= 25;
  const short int led_strip= 64;
  const short int shape_size=40;
  Adafruit_NeoPixel led_1(LED_COUNT, LED_PIN_1, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_2(LED_COUNT, LED_PIN_2, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_3(LED_COUNT, LED_PIN_3, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_4(LED_COUNT, LED_PIN_4, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_5(LED_COUNT, LED_PIN_5, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_6(LED_COUNT, LED_PIN_6, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_7(LED_COUNT, LED_PIN_7, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel LEDS[pins];

  Lighting();
  void fill_shape();
  void print_randshape();
  void set_shape(short int coord[10][2]);
  void chang_randShape();
  void get_accel (short int intake[2]);
  void print_img();
  void print_shape();
  void move_shape(short int acceleration[2]);
  void light_up();
}
void setup() {
  // put your setup code here, to run once:

}

void loop() {
  // put your main code here, to run repeatedly:

}

Can some one help me with my error ?

2

The above is just a portion of my code. However, I wasn't able to get it wokring

3

void set_shape(short int coord[10][2]);

_____________ REMOVED OLD EYES ARE OLD EYES _______________

4

Romonaga:
void set_shape(short int coord[10][2]);

You have several function calls like this

Remove the short int from the calls to the function. You also are addressing what appears to be arrays. Yet I did not see arrays defined. On my phone might have missed it

My phone must be on the blink - I don't see any function calls.

5

AWOL:
My phone must be on the blink - I don't see any function calls.

Nope it was me on my phone. I see this is part of a class definition. Old eyes are old eyes.

6

The section of the code I extracted, are attempting to initialize 7 Adafruit_NeoPixel. However, you are doing this is your header, and your header is expecting you to have function prototypes. These variables should be delacred in the header and initialized in your class implementation.

You also have declared all the functions in your header, but I see no implementation yet.

Also, at the end of your class declaration needs to end in }; You are missing the ;

  Adafruit_NeoPixel led_1(LED_COUNT, LED_PIN_1, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_2(LED_COUNT, LED_PIN_2, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_3(LED_COUNT, LED_PIN_3, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_4(LED_COUNT, LED_PIN_4, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_5(LED_COUNT, LED_PIN_5, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_6(LED_COUNT, LED_PIN_6, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel led_7(LED_COUNT, LED_PIN_7, NEO_RGB + NEO_KHZ400);
  Adafruit_NeoPixel LEDS[pins];
7

Apparently, C++ won't let you declare an object that way inside a class definition. The bizarre compiler errors certainly don't make that clear though. Here's an untested version that compiles at least:

#include <Adafruit_NeoPixel.h>
#ifdef __AVR__
#include <avr/power.h> // Required for 16 MHz Adafruit Trinket
#endif
#define LED_PIN_1    13
#define LED_PIN_2    12
#define LED_PIN_3    14
#define LED_PIN_4    27
#define LED_PIN_5    26
#define LED_PIN_6    25
#define LED_PIN_7    33


#define LED_COUNT     90
#define LED_COUNT_7   30
#define BRIGHTNESS   50

class Lighting
{
  public:
    const short int width = 30;
    const short int len = 25;
    const short int led_strip = 64;
    const short int shape_size = 40;
    Adafruit_NeoPixel led_1;
    Adafruit_NeoPixel led_2;
/*    Adafruit_NeoPixel led_3(LED_COUNT, LED_PIN_3, NEO_RGB + NEO_KHZ400);
    Adafruit_NeoPixel led_4(LED_COUNT, LED_PIN_4, NEO_RGB + NEO_KHZ400);
    Adafruit_NeoPixel led_5(LED_COUNT, LED_PIN_5, NEO_RGB + NEO_KHZ400);
    Adafruit_NeoPixel led_6(LED_COUNT, LED_PIN_6, NEO_RGB + NEO_KHZ400);
    Adafruit_NeoPixel led_7(LED_COUNT, LED_PIN_7, NEO_RGB + NEO_KHZ400);
    Adafruit_NeoPixel LEDS[pins];*/

    Lighting(): led_1(LED_COUNT, LED_PIN_1, NEO_RGB + NEO_KHZ400), led_2(LED_COUNT, LED_PIN_1, NEO_RGB + NEO_KHZ400)
    {
      
    }
    void fill_shape();
    void print_randshape();
    void set_shape(short int coord[10][2]);
    void chang_randShape();
    void get_accel (short int intake[2]);
    void print_img();
    void print_shape();
    void move_shape(short int acceleration[2]);
    void light_up();
};
void setup()
{
  // put your setup code here, to run once:
}

void loop()
{
  // put your main code here, to run repeatedly:
}