Problem in Funktion

Gast #3595599
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
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
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
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
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? :-)
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.
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.
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
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

Eine eigene Frage? Dieser Thread ist über 12 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

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.