Problem in Funktion

[Markus (Gast)]

Hallo,

warum bekomme ich immer einen Fehler in dieser Funktion (s.unten) Dachte 
erst ich hätte ein } zu viel, aber ich finde nichts....

Kann mir bitte jemand helfen?!
Danke

from main.c:22:
./jalousie.h:49: error: expected ';', ',' or ')' before numeric constant
make.exe: *** [main.o] Error 1

in der .h:

struct StructJalousienHeben 
{
   uint16_t TimeRichtungAus; 
   uint16_t auf;
   uint16_t ab;
   uint16_t Richtung;
   uint16_t Netz;
};

in der .c

uint8_t JalStartHeben (struct StructJalousienHeben* Jal) 
{
        // Ausschalten bei einer Fahrt wenn noch mal Auf gedrückt wird
     if (jal_status[Jal->auf]==1 || jal_status[Jal->ab]==1)
      {      
      // erst Netz aus
      AusgangUnsetBit(Jal->Netz); 
      AusgangOut();    
      
      if (_100msImp) Jal->TimeRichtungAus++;

            if (Jal->TimeRichtungAus > 20) 
        {
          // dann Richtung Aus
          AusgangUnsetBit(Jal->Richtung); 
          AusgangOut();
          jal_status[Jal->auf]=0;
          jal_status[Jal->ab]=0;
          
          Jal->TimeRichtungAus=0;
          usart_writePC("aus-auf [%i],[%i]\n",Jal->auf,Jal->ab);
        }
      }  
    // Heben wenn noch nichts an
    else if (jal_status[Jal->Netz]==0 && jal_status[Jal->Richtung]==0)
      {          
        // erst Richtung ein 
        AusgangSetBit(Jal->Richtung); 
        AusgangOut();    
      
      
        if (_100msImp) Jal->TimeRichtungAus++;

        if (Jal->TimeRichtungAus > 20) 
          {
            // dann Netz ein 
            AusgangSetBit(Jal->Netz); 
            AusgangOut();
            jal_status[Jal->auf]=1;
            jal_status[Jal->ab]=0;
            
            Jal->TimeRichtungAus=0;
            usart_writePC("auf [%i],[%i]\n",Jal->auf,Jal->ab);
          }
      }  

return 1;
}

[Stephan B. (matrixstorm)]

Hallo Markus
Haenge doch bitte die Dateien an, sodass man auch die Zeilennummern und 
den Zusammenhang sieht.

MfG

[Bitflüsterer (Gast)]

Soweit ist erstmal nichts auffälliges zu sehen.

Deine Fehlermeldung enthält Zeilennummern. Wir wissen aber nicht welche 
Zeilennummern Dein Code hat.

Bitte poste mal den kompletten Code (Zeile 22 von main.c ist nicht 
enthalten) oder kennzeichne Zeile 49 der h-Datei (die, wie ich vermute, 
auch nicht gezeigt ist).

[Dr. Sommer (Gast)]

Markus schrieb:
> ./jalousie.h:49: error: expected ';', ',' or ')' before numeric constant

Soll das heißen diese Funktion ist im Header jalousie.h definiert? Was 
hat sie dort zu suchen? Und in deinem Code gibts keine Zeile 49. Und was 
jal_status, AusgangUnsetBit (DEnglisch o.O), AusgangOut, etc... sind 
weiß hier auch keiner.

[spess53 (Gast)]

Hi

>./jalousie.h:49: error: expected ';', ',' or ')' before numeric constant

Und was steht in Zeile 49 der jalousie.h?

Ich tippe mal auf '};'.

MfG Spess

[Bitflüsterer (Gast)]

Die folgende Frage ist scherzhaft, jedenfalls rethorisch gemeint:

Was meint Ihr denn alle, was die ominäösen Zahlen in den Fehlermeldungen 
bedeuten? Die Lottozahlen von nächster Woche? :-)

[Markus (Gast)]

Im Anhang die Dateien.... Jedoch habe ich viell Kommentar gelösht, die 
Zeilennumer 49 dürfte nicht mehr passen....

[Bitflüsterer (Gast)]

> ... die Zeilennumer 49 dürfte nicht mehr passen....

Schön. Aber Du hast doch unsere Antworten so verstanden, dass wir gerne 
die Zeile identifizieren würden, für die der Fehler gemeldet wird?

Also schreibe bitte, wie die Fehlermeldung mit dem neuen Code lautet.

Es scheint aber so zu sein, dass der eigentliche Fehler in einer der vor 
jalousie.h eingefügten ist.

Poste am besten alles.

[Holger L. (max5v)]

uint16_t time_open[20] = 
{500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500 
,500,500,};
Das letzte Komma ist zu viel was aber wohl nicht viel ausmacht.

Hast du mal versucht die Zeilen

usart_writePC("aus-auf [%i],[%i]\n",Jal->auf,Jal->ab);
usart_writePC("auf [%i],[%i]\n",Jal->auf,Jal->ab);

zu deaktivieren ?

Kenne deine Usart -Funktion nicht, aber das sieht mir ein wenig 
abenteuerlich aus.

[Mike (Gast)]

Holger L. schrieb:
> Kenne deine Usart -Funktion nicht

Vielleicht ist die von sprintf abgeleitet ...

[Johann L. (gjlayde)]

Holger L. schrieb:
> uint16_t time_open[20] =
> {500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500 
,500,500,};
> Das letzte Komma ist zu viel was aber wohl nicht viel ausmacht.

Nein, das Komma ist nicht zu viel. Das ist C.

[Bitflüsterer (Gast)]

> Nein, das Komma ist nicht zu viel. Das ist C.
Das kommt mir wie eine humorvolle Bemerkung vor. Ha!

[Holger L. (max5v)]

Johann L. schrieb:
> Holger L. schrieb:
>> uint16_t time_open[20] =
>> {500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500,500
> ,500,500,};
>> Das letzte Komma ist zu viel was aber wohl nicht viel ausmacht.
>
> Nein, das Komma ist nicht zu viel. Das ist C.

Gut, kannst du das kurz erklären ?
In meinen Augen ist es überflüssig, habe mich aber auch nur leicht in C 
über Internet Tutorials eingearbeitet. Da gehen sehr viele Informationen 
verloren.

[Christian B. (chbalnuweit)]

Johann L. schrieb:
> Nein, das Komma ist nicht zu viel. Das ist C.

Also das C-Buch, dass ich gelesen habe, nutzt kein Komma nach dem 
letzten Array-Element.

[Johann L. (gjlayde)]

Holger L. schrieb:
> Johann L. schrieb:
>> Holger L. schrieb:
>>> Das letzte Komma ist zu viel was aber wohl nicht viel ausmacht.
>>
>> Nein, das Komma ist nicht zu viel. Das ist C.
>
> Gut, kannst du das kurz erklären ?

Das Komma kann weggelassen werden, was nix an der Semantik ändert. Es 
ist aber auch nicht falsch, d.h. es erzeugt weder einen Syntax-Fehler 
noch ein unerwünschtes Array-Element.  Mit dem Problem des OP hat es 
also nix zu tun.

An seiner Stelle würde ich mal bei den gefühlt 20 Header anfangen, die 
da per Gießkannenprinzip reingezogen werden; teilweise mehrfach oder 
über andere Header.  Vermutlich lieht das Problem in dem Header-Wahn, 
oder daß an entscheidender Stelle z.B. stdint.h fehlt so daß uint16_t 
nicht definiert ist.

[Markus P. (sebastianwurst)]

Ich konnte nur 1-2 Header datei wegnehmen, Problem lag darin, das ich 
die "JAL" Variable als Konstante-Variable in der config.h benutz. Das 
mocht er nicht?! Da ich die auch "include" hatte er da Problem 
angemeldet...