Problem mit Modulo

[Tilo (Gast)]

Hallo

Ich verwende WinARM mit gcc 4.1.1 und will einen ARM-Prozessor 
programmieren.

Timer1 löst alle 1ms einen Interrupt aus.
Alle 250ms soll ein Messwert gespeichert werden.
Die Messwerte sollen in einem Ringpuffer zwischengespeichert
und dann ausgegeben werden.

// Interruptroutine
if (timer1count % 250==0 ) {
 [...]
 timer1count=0;
}
else timer1count++;

Meine Idee war, timer1count so lange hochzuzählen, bis die Division
durch 250 restlos auf geht.

if (timer1count == 250) ... funktioniert. Mit Modulo wird die
if-Bedingung immer ausgeführt.

Weiß jemand warum?

Vielen Dank,

[johnny.m (Gast)]

Statt

if (timercount % 250 == 0)

kann man auch schreiben

if (!(timercount % 250))

Allerdings sollte das nicht zu einem Problem führen. Die sicherste und 
einfachste Methode ist sicher

if (timercount >= 250)

Dabei muss der Prozessor nämlich nicht jedes mal die Modulo-Operation 
durchführen (weiß nicht, wie die implementiert ist), sondern nur einen 
einfachen Vergleich. Und sollte mal aus irgendwelchen Gründen ein Wert 
größer als 250 in der Variable stehen, dann wird auch in dem Fall der 
Zähler zurückgesetzt. Der einfache Vergleich ist wahrscheinlich die 
beste Wahl. Ich kenne Deinen genauen Anwendungszweck nicht, aber ich 
würde nie auf die Idee kommen, so etwas mit Modulo zu machen, wenn eh 
nur Werte kleiner gleich 250 auftreten können.

Dass das mit Modulo nicht funktioniert, wundert mich allerdings schon 
ein bisschen. Logisch korrekt ist es ja an sich...

[Roland Schmidt (Gast)]

0 % 250 ==> 0

[Tilo (Gast)]

Danke Roland, die 0 hab ich natürlich nicht bedacht :)

Ein einfacher Vergleich ist natürlich besser.
Warum ich auf die Idee mit Modulo gekommen bin weiß ich nicht mehr.

[unsichtbarer WM-Rahul (Gast)]

>Warum ich auf die Idee mit Modulo gekommen bin weiß ich nicht mehr.

Ich aber!
>Die Messwerte sollen in einem Ringpuffer zwischengespeichert
>und dann ausgegeben werden.

Quatsch:

>if (timer1count % 250==0 ) {
> [...]
> timer1count=0;
>}
>else timer1count++;

Sinnvoll:

 timer1count++;
 timer1count = timer1count % 250; // so kann timer1count nie grösser als  249 werden.

249 % 250 = 249
250 % 250 = 0
1   % 250 = 1
.
.
.
.
.

[Dirk (Gast)]

Hi, machen wir es ein bischen anders geht es auch

Peusdo C Code:

volatile u8 timer=0;


ISR(TIMER){
timer++;
if(!(timer%8)){
 PORTD ^= (1<<0);
}

if(!(timer%64)){
 PORTD ^= (1<<1);
}
}