Click here to Skip to main content
15,877,915 members
Please Sign up or sign in to vote.
0.00/5 (No votes)
See more:
i've been trying to figure out what's wrong with it and I can't wrap my head around there, any guidance would be much appreciated! I'm pretty much a noob when it comes to coding. i tried to do an ultrasound sensor that changes the led rgb light once you get close, Here's the code.

#include <Adafruit_NeoPixel.h>

const int EchoPin = 5;
const int TriggerPin = 6;
const int dist[] = {100}, {200}, {300};
const String colors[] = {"blau"}, {"vermell"}, {"verd"};
const int delay = 3000;

int PIN=6; //Pin donde está conectada la tira de leds
int NUMPIXELS=6; //Número de leds conectados

Adafruit_NeoPixel pixels = Adafruit_NeoPixel(NUMPIXELS, PIN, NEO_GRB + NEO_KHZ800);


void setup() {
   Serial.begin(9600);
   pinMode(TriggerPin, OUTPUT);
   pinMode(EchoPin, INPUT);
}
void loop() {
   int cm = ping(TriggerPin, EchoPin);
   Serial.print("Distancia: ");
   Serial.println(cm);
   startSequence (cm)
   delay(1000);
}
int ping(int TriggerPin, int EchoPin) {
   long duration, distanceCm;
   sendPing 
   
    duration = pulseIn(EchoPin, HIGH);  //medir temps ms
   
   distanceCm = duration * 10 / (292 * 2);  //calcular a cm
   return distanceCm;
}
void sendPing () {

   digitalWrite(TriggerPin, LOW);  //generar puls 
   delayMicroseconds(4);
   digitalWrite(TriggerPin, HIGH);  //Trigger
   delayMicroseconds(10);
   digitalWrite(TriggerPin, LOW);
}

void startSequence (int cm) {

  String color;

  Switch (cm) {
    case dist [0]:
      color = colors [0];
      break; 

    case dist [1]:
      color = colors [1];
      break;

    case dist [2]:
      color = colors [2];
      break;  

  }
  switchOnLights (color);

}


void switchOnLights (String color) {

  // encendre leds (buscar llibreria - demanar joan)

  delay (delay)

  // apagar leds 



}

void setup() {
      pixels.begin(); // Inicialización
      pixels.show(); // Inicialitza tots els pixels apagats
}

void loop() {
  pixels.setPixelColor(0,0,0,255);
   pixels.show();
 delay (1000);
  pixels.setPixelColor(0,0,0,0);
  pixels.show();

  pixels.setPixelColor(1,0,0,255);
   pixels.show();
 delay (1000);
  pixels.setPixelColor(1,0,0,0);
  pixels.show();

  pixels.setPixelColor(2,0,0,255);
   pixels.show();
 delay (1000);
  pixels.setPixelColor(2,0,0,0);
  pixels.show();

}


What I have tried:

i dont know if im doint it wrong but it should work
Posted
Updated 4-Jan-23 8:38am
Comments
CHill60 4-Jan-23 12:28pm    
Save us some time - which line throws the error

1 solution

Try taking out some of the curly brackets:
C++
const int dist[] = {100}, {200}, {300};
const String colors[] = {"blau"}, {"vermell"}, {"verd"};
Should be like this:
C++
const int dist[] = {100, 200, 300};
const String colors[] = {"blau", "vermell", "verd"};
But "String" isn't a type, try "string" instead.

You should expect to get syntax errors every day, probably many times a day while you are coding - we all do regardless of how much experience we have! Sometimes, we misspell a variable, or a keyword; sometimes we forget to close a string or a code block. Sometimes the cat walks over your keyboard and types something really weird. Sometimes we just forget how many parameters a method call needs.

We all make mistakes.

And because we all do it, we all have to fix syntax errors - and it's a lot quicker to learn how and fix them yourself than to wait for someone else to fix them for you! So invest a little time in learning how to read error messages, and how to interpret your code as written in the light of what the compiler is telling you is wrong - it really is trying to be helpful!

So read this: How to Write Code to Solve a Problem, A Beginner's Guide Part 2: Syntax Errors[^] - it should help you next time you get a compilation error!

And spending a little time learning to understand syntax error messages will save you a huge amount of time in future: you waited at least 1/4 hour for Chris to reply, then your email system probably added another 10 minutes or so, plus the time it took you to type up the question once you had found this site and created an account. Chances are that you could have saved a significant chunk of that time if you knew how to read them!

I'm not saying we don't want to help you fix them - sometimes I can't see my own errors because I read what I meant to write - but fixing syntax errors is part of the job, and if you can't do it for yourself people are going to look at you as a bit weird should you get a job in the industry!
 
Share this answer
 
v2
Comments
CPallini 4-Jan-23 14:53pm    
5.BTW String is a predefined class, in the Arduino framework.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)



CodeProject, 20 Bay Street, 11th Floor Toronto, Ontario, Canada M5J 2N8 +1 (416) 849-8900