Fehler in wait-funktion

[Christian (Gast)]

hi,
ich möchte eine wait-funktion in c mit einem Timer erstellen.
dazu habe ihc es wie folgt gemacht

...
#define wait_flag = 0x00;
...

void wait(unsigned int count);
void Timer1_Init(void);
void __attribute__((interrupt, no_auto_psv)) _T1Interrupt(void);
...

int main (void){
...
  ADPCFGbits.PCFG0 = 1;
  TRISBbits.TRISB0 = 0;
  LATBbits.LATB0 = 0;
...
  while(1){
    LATBbits.LATB0 = !LATBbits.LATB0;
    wait(200);
  }
}

void wait(unsigned int count)
{
    unsigned int countzahl;
    wait_flag = 1;
    countzahl = count/(0,00000006781684027777780*256*1000);
    PR1 = countzahl;
    TMR1 = 0;           // set timer to 0
    T1CONbits.TON = 1;
    while(wait_flag!=0){ }
}

{
    // PR1 and TCKPS are set to call interrupt every 1000ms.´
    // Period = PR1 * prescaler * Tcy = 57600 * 256 * 67ns = 1000ms
    T1CONbits.TON = 0;    // Clear Timer 1 configuration
    T1CONbits.TCKPS = 3;  // Set timer 1 prescaler (0=1:1, 1=1:8, 2=1:64, 3=1:256)
    T1CONbits.TSYNC = 1;  // Synchronize external clock input (1: Synchronize, 0: Do not synchronize)
    T1CONbits.TCS = 0;    // Timer Clock Source bit (1: External clock from pin TxCK, 0: Internal clock (Fosc/4))
    //PR1 = 57600/2;         // Set Timer 1 period (max value is 65535 (16 bit))  -> has to be set to 576 (10ms)
    IPC0bits.T1IP = 1;    // Set Timer 1 interrupt priority
    IFS0bits.T1IF = 0;    // Clear Timer 1 interrupt flag
    IEC0bits.T1IE = 1;    // Enable Timer 1 interrupt
    //T1CONbits.TON = 1;    // Turn on Timer 1 // made in _C1Interrupt(void)
}

void __attribute__((interrupt, no_auto_psv)) _T1Interrupt(void)
{
    wait_flag = 0;
    T1CONbits.TON = 0;  // stopts timer
    IFS0bits.T1IF = 0;  // Clear Timer 1 interrupt flag
}

aber er gibt mir den Fehler

Main.c: In function 'wait':
Main.c:84:5: error: lvalue required as left operand of assignment
Main.c:85:26: error: invalid digit "8" in octal constant
Main.c:85:25: warning: left-hand operand of comma expression has no 
effect
Main.c: In function '_T1Interrupt':
Main.c:111:5: error: lvalue required as left operand of assignment
make[2]: *** [build/default/production/Main.o] Error 255
make[2]: *** Waiting for unfinished jobs....
nbproject/Makefile-default.mk:101: recipe for target 
'build/default/production/Main.o' failed
make[1]: *** [.build-conf] Error 2
make: *** [.build-impl] Error 2

was ist daran falsch? bzw. was bedeutet die Fehlermeldung?

[Andreas S. (Firma: Schweigstill IT) (schweigstill)]

Fehlerhafter Ausdruck:

countzahl = count/(0,00000006781684027777780*256*1000);

Korrekter Ausdruck:

countzahl = count/(0.00000006781684027777780*256*1000);

Inwiefern dieser Ausdruck überhaupt sinnvoll ist, steht noch zur 
Diskussion. Um zumindest weitere Rundungsfehler bei der Vorberechnung 
des Divisors zu reduzieren, kann es sinnvoll sein, die Ganzzahlfaktoren 
auch noch in Fließkommazahlen zu wandeln:

countzahl = count/(0.00000006781684027777780*256.0*1000.0);

[Max (Gast)]

Christian schrieb:
> 0,00000006781684027777780

sollte 0.00000006781684027777780 (Punkt statt komma) sein.
Falls noch mehr spaß rauskommt: mehr detials erzählen

[Torsten R. (Firma: Torrox.de) (torstenrobitzki)]

Zeile 84:

wait_flag = 1;

wait_flag hast Du als Makro definiert und löst zu 0x00 auf, da steht 
also:

0x00 = 1;

Zeile 85:

(0,00000006781684027777780*256*1000);

Ist ein Ausdruck mit Komma-Operator. Der rechte Operant ist ein oktal 
kodiertes Literal (wegen der führenden 0) und in oktal-Zahlen gibt es 
die Ziffer 8 nicht. Wahrscheinlich meinst Du 0.000...

mfg Torsten

[Christian (Gast)]

mhh,
ein bisschen schäme ich mich.
mit zeile 85 war copy paste. damn

danke euch

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Christian schrieb:
> bzw. was bedeutet die Fehlermeldung?
Die Ursache ist, dass ein Integerwert, der mit einer 0 beginnt, als 
Oktalzahl interpretiert wird.
Es ist z.B. eine schlechte Idee, aus Gründen der bessern Lesbarkeit 
sowas zu schreiben:

a = 2345;
b = 1234;
c = 0123;
d = 0012;
e = 0001;

c und d werden hier unerwarete Werte enthalten: c == 83 bzw. d == 8

Fazit:
00000006781684027777780
Du hast Glück, dass deine Zahl eine 8 enthält...


Christian schrieb:
> TMR1 = 0;           // set timer to 0
Das ist eine Idee, die in die Sackgasse führt. Denn damit lässt sich 
dieser Timer für nichts Anderes mehr verwenden. Ausserdem ist explizites 
Warten sowieso nur Rechenzeitvergeudung...

[Alpenmatrose (Gast)]

Andreas Schweigstill schrieb:
> countzahl = count/(0.00000006781684027777780*256.0*1000.0);

Oder man spart sich diesen Moloch und rechnet:
> countzahl = count * 57.6;

Ganz davon ab finde ich Magic Number sehr unschön, weil sie die 
Wartbarkeit des Codes erschweren.

Also lieber:

#define MAGIC_CONSTANT1    (0.00000006781684027777780)
#define MAGIC_CONSTANT2    (256)
#define MAGIC_CONSTANT3    (1000)
#define VERY_MAGIC_CONSTANT (MAGIC_CONSTANT1 * MAGIC_CONSTANT2 * MAGIC_CONSTANT3)

...
countzahl = count * VERY_MAGIC_CONSTANT;

[Alpenmatrose (Gast)]

Nachtrag: Da bin ich jetzt natürlich durcheinander gekommen, bei Nutzung 
der Defines wie von mir angegeben muss es natürlich wieder lauten

countzahl = count / VERY_MAGIC_CONSTANT;