Problem in Funktion

Gast #3595599
Lesenswert?
• ▲
▼
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:
1
struct StructJalousienHeben 
2
{
3
   uint16_t TimeRichtungAus; 
4
   uint16_t auf;
5
   uint16_t ab;
6
   uint16_t Richtung;
7
   uint16_t Netz;
8
};

in der .c
1
uint8_t JalStartHeben (struct StructJalousienHeben* Jal) 
2
{
3
        // Ausschalten bei einer Fahrt wenn noch mal Auf gedrückt wird
4
     if (jal_status[Jal->auf]==1 || jal_status[Jal->ab]==1)
5
      {      
6
      // erst Netz aus
7
      AusgangUnsetBit(Jal->Netz); 
8
      AusgangOut();    
9
      
10
      if (_100msImp) Jal->TimeRichtungAus++;
11

12
            if (Jal->TimeRichtungAus > 20) 
13
        {
14
          // dann Richtung Aus
15
          AusgangUnsetBit(Jal->Richtung); 
16
          AusgangOut();
17
          jal_status[Jal->auf]=0;
18
          jal_status[Jal->ab]=0;
19
          
20
          Jal->TimeRichtungAus=0;
21
          usart_writePC("aus-auf [%i],[%i]\n",Jal->auf,Jal->ab);
22
        }
23
      }  
24
    // Heben wenn noch nichts an
25
    else if (jal_status[Jal->Netz]==0 && jal_status[Jal->Richtung]==0)
26
      {          
27
        // erst Richtung ein 
28
        AusgangSetBit(Jal->Richtung); 
29
        AusgangOut();    
30
      
31
      
32
        if (_100msImp) Jal->TimeRichtungAus++;
33

34
        if (Jal->TimeRichtungAus > 20) 
35
          {
36
            // dann Netz ein 
37
            AusgangSetBit(Jal->Netz); 
38
            AusgangOut();
39
            jal_status[Jal->auf]=1;
40
            jal_status[Jal->ab]=0;
41
            
42
            Jal->TimeRichtungAus=0;
43
            usart_writePC("auf [%i],[%i]\n",Jal->auf,Jal->ab);
44
          }
45
      }  
46

47
return 1;
48
}
Gast #3595606
Lesenswert?
• ▲
▼
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).
Gast #3595607
Lesenswert?
• ▲
▼
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.
Gast #3595614
Lesenswert?
• ▲
▼
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? :-)
Gast #3595645
Lesenswert?
• ▲
▼
> ... 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.
#3595669
Lesenswert?
• ▲
▼
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
1
usart_writePC("aus-auf [%i],[%i]\n",Jal->auf,Jal->ab);
2
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.
#3595738
Lesenswert?
• ▲
▼
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.
Persönliche Seite #3595745
Lesenswert?
• ▲
▼
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.

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.