variable or field declared void

[Harry]

Ich bekomme bei meinem code folgende Fehlermeldung:

exit status 1
variable or field 'Lauflicht4' declared void

Ich habe auch eine Lösung die funktioniert, die mir aber nicht so richtig gefällt und ich weis auch nicht warum mein erster code nicht funktioniert.

Code:

#include <Adafruit_NeoPixel.h>
#define LED_PIN     6
#define LED_COUNT  60
#define Poti1G A0  // Eingang Poti für Geschwindigkeit
#define Poti2H A1  // Eingang Poti für Helligkeit

Adafruit_NeoPixel strip(LED_COUNT, LED_PIN, NEO_GRBW + NEO_KHZ800);

uint32_t rot   = strip.Color(255,   0,   0);
uint32_t gruen = strip.Color(  0, 255,   0);
uint32_t blau  = strip.Color(  0,   0, 255);
uint32_t gelb  = strip.Color(255, 255,   0);
uint32_t weiss = strip.Color(255, 255, 255);

uint32_t Color1;
uint32_t Color2;
uint32_t Color3;
uint32_t Color4;
uint32_t Color5;
uint32_t Color6;

int Farbenzahl;
int Geschwindigkeit;
int Helligkeit = 50;

void setup() {
  strip.begin();
  strip.show();
  strip.setBrightness(Helligkeit);
}

void loop() {
  Geschwindigkeit = analogRead(Poti1G);
  Helligkeit = analogRead(Poti2H) / 4;
  strip.setBrightness(Helligkeit);
  Lauflicht4(7, rot, gruen, blau, gelb);
}


void Lauflicht4(Farbenzahl, Color1,  Color2, Color3, Color4){
}

Die Lösung welche mir nicht gefällt weil ich meine Variablen gerne alle am Anfang definieren möchte und bei dieser Lösung die void Zeile etwas lang und unübersichtlich wird.

Code:

#include <Adafruit_NeoPixel.h>
#define LED_PIN     6
#define LED_COUNT  60
#define Poti1G A0  // Eingang Poti für Geschwindigkeit
#define Poti2H A1  // Eingang Poti für Helligkeit

Adafruit_NeoPixel strip(LED_COUNT, LED_PIN, NEO_GRBW + NEO_KHZ800);

uint32_t rot   = strip.Color(255,   0,   0);
uint32_t gruen = strip.Color(  0, 255,   0);
uint32_t blau  = strip.Color(  0,   0, 255);
uint32_t gelb  = strip.Color(255, 255,   0);
uint32_t weiss = strip.Color(255, 255, 255);

int Geschwindigkeit;
int Helligkeit = 50;

void setup() {
  strip.begin();
  strip.show();
  strip.setBrightness(Helligkeit);
}

void loop() {
  Geschwindigkeit = analogRead(Poti1G);
  Helligkeit = analogRead(Poti2H) / 4;
  strip.setBrightness(Helligkeit);
  Lauflicht4(7, rot, gruen, blau, gelb);
}

void Lauflicht4(int Farbenzahl, uint32_t Color1, uint32_t Color2, uint32_t Color3,uint32_t Color4){
}

Warum funktioniert die erste Lösung nicht und wie bekomme ich es hin meine Variablen am Anfang zu definieren?

Harry

[Tommy56]

Es wäre besser, wenn Du die Komplette Fehlermeldung mit Zeilennummer und Fehleranzeige postest.

Für was sollen die globale Variable Farbenzahl, sowie die Color1 bis 6 gut sein?

Gruß Tommy

[Harry]

(21.02.2020 17:56)Tommy56 schrieb:  Es wäre besser, wenn Du die Komplette Fehlermeldung mit Zeilennummer und Fehleranzeige postest.

Code:

Lauflicht_neu_01:47:17: error: variable or field 'Lauflicht4' declared void

void Lauflicht4(Farbenzahl, Color1,  Color2, Color3, Color4){

                 ^~~~~~~~~~

Mehrere Bibliotheken wurden für "Adafruit_NeoPixel.h" gefunden
Benutzt: C:\Arduino\LED
exit status 1
variable or field 'Lauflicht4' declared void

Das neue Programm steht erst am Anfang.

(21.02.2020 17:56)Tommy56 schrieb:  Für was sollen die globale Variable Farbenzahl, sowie die Color1 bis 6 gut sein?

Das ganze soll ein Lauflicht mit verschiede Optionen werden.
Farbzahl = Zahl der LEDs in der gleichen Farbe hintereinander.
Color1-6 = Die Farben welche wieder gegeben werden.

Lauflicht4(7, rot, gruen, blau, gelb) steht für

Lauflicht4 = Lauflicht mit 4 verschiedene Farben (Die Farbanzahl soll später noch mit übergeben werden)

rot, gruen, blau, gelb = die Reihenfolge der Farben.

Harry

[Tommy56]

Dein Funktionskopf ist falsch. Du musst zu den funktionslokalen Variablen auch deren Typ zur Übergabe angeben.
Die Fehlermeldung passt nicht zum Fehler.

Gruß Tommy

[MicroBahner]

Was soll dein 'Lauflicht4' denn eigentlich sein? Da kommt der Compiler nämlich auch nicht klar und versteht nicht was das soll. Deshalb auch die unklare Fehlermeldung.

In der 2. Variante ist das die Definition einer leeren Funktion. Ziemlich sinnfrei, aber für den Compiler symtaktisch ok.

Also erkläre erstmal, was Du da mit dem 'Lauflicht4' machen willst.

[MicroBahner]

Wenn es tatsächlich eine Funktion werden soll MUSST Du die Funktionsparameter dort deklarieren, und auch den Typ der jeweiligen Parameter angeben.
Grundlagen -> Funktionsdefinitionen.
Dann musst Du die Funktion aber auch mit Leben füllen - sonst passiert da nichts.

[Harry]

Kannst du mir bitte erklären was du damit meinst?
Welcher Teil ist der Funktionskopf?
Den Typ habe ich doch ganz am Anfang angegeben.

Code:

uint32_t Color1;
uint32_t Color2;
uint32_t Color3;
uint32_t Color4;
uint32_t Color5;
uint32_t Color6;
int Farbenzahl;

(21.02.2020 18:20)Tommy56 schrieb:  Dein Funktionskopf ist falsch. Du musst zu den funktionslokalen Variablen auch deren Typ zur Übergabe angeben.
Die Fehlermeldung passt nicht zum Fehler.

Das müsste jetzt alles ein:

Code:

Arduino: 1.8.10 (Windows 7), Board: "Arduino Nano, ATmega328P (Old Bootloader)"

Lauflicht_neu_01:46:17: error: variable or field 'Lauflicht4' declared void

void Lauflicht4(Farbenzahl, Color1,  Color2, Color3, Color4){

                 ^~~~~~~~~~

C:\Users\Test\AppData\Local\Temp\arduino_modified_sketch_317744\Lauflicht_neu_01​.ino: In function 'void loop()':

Lauflicht_neu_01:42:3: error: 'Lauflicht4' was not declared in this scope

   Lauflicht4(7, rot, gruen, blau, gelb);   // LEDs in gleicher Farbe, Farben,

   ^~~~~~~~~~

C:\Users\Test\AppData\Local\Temp\arduino_modified_sketch_317744\Lauflicht_neu_01​.ino: At global scope:

Lauflicht_neu_01:46:17: error: variable or field 'Lauflicht4' declared void

void Lauflicht4(Farbenzahl, Color1,  Color2, Color3, Color4){

                 ^~~~~~~~~~

Mehrere Bibliotheken wurden für "Adafruit_NeoPixel.h" gefunden
Benutzt: C:\Arduino\LED
exit status 1
variable or field 'Lauflicht4' declared void

Harry

[Tommy56]

Warum lernst Du nicht endlich ein paar Grundlagen? Bald gibt es auf solche Fragen nach 269 Beiträgen keine Antworten mehr, wenn Du nicht selbst Lernwillen zeigst.

Code:

void Lauflicht4(byte Farbenzahl, uint32_t Color1,  uint32_t Color2, uint32_t Color3, uint32_t Color4){

Gruß Tommy