PINx Abfrage

[Chris (Gast)]

Hey Leute, da ich persönlich viel if-Abfragen hintereinander sehr 
unübersichtliche´finde, habe ich mir gedacht um die Positionen eines 
DIL-Schalters abzufragen, nehme ich einfach ein switch-case.
Hier ist mein Code:

/*
 * LED_CONTROLLER.c
 *
 * Created: 26.04.2015 18:26:10
 *  Author: Christian
 */ 


#include <avr/io.h>
#include <stdbool.h>
#include <util/delay.h>

#ifndef F_CPU
#define F_CPU 1200000;
#endif

int main(void)
{
  
  DDRB = (1<<3) + (1<<4);
  PORTB = (1<<0) + (0<<1) + (1<<2);
  
    while(1)
    {
    
    volatile uint8_t pin_reg = PINB;
    switch(pin_reg){
      case 7: PORTB |= (1<<3);
          _delay_ms(1000);
          PORTB &= ~(1<<3);
          _delay_ms(1000);
          break;
      case 1: PORTB |= (1<<3);
          break;
          
    }
  }  
}

Das Problem ist hierbei, wenn ich einmal die Positionen 2 und 3 auf an 
gestellt habe, leuchtet die LED ordnungsgemäß. Wenn ich sie aber dann 
wieder auf 0 kippe, leuchtet die LED weitter, obwohl sie nun blinken 
sollte, warum ist das so?Ich finde einfach nicht den Fehler....
Vielen Dank schonmal für jede Hilfe :)

[Noch einer (Gast)]

Diese Idee finde ich nicht so überzeugend. Dein Beispiel beweist doch: 
Ketten von If-Abfragen sind übersichtlicher als ein case mit 
kombinierter Abfrage.

[Torsten C. (torsten_c)]

Dein 'pin_reg' beinhaltet die LED, also:

volatile uint8_t pin_reg = PINB & ~(1<<3);

Oder?

[Max H. (hartl192)]

Wenn du PORTB auf high setzt wirst du bei nächsten Mal PINB3 als '1' 
einlesen und PINB wird immer > 8 sein. Versuch mal die nicht benötigten 
bits auszumaskieren:

volatile uint8_t pin_reg = PINB & 0x07;

[Torsten C. (torsten_c)]

Wenn es drei DIL-Schalter an PB0..2 sind geht auch 0x07.

[Konrad S. (maybee)]

Chris schrieb:
> PORTB = (1<<0) + (0<<1) + (1<<2);

Bist du dir da sicher?

[Chris (Gast)]

Ok ok ich schau mal :)
Und ja bin mir sicher, muss ja die Pull-up Widerstände aktivieren.

[ov.board (Gast)]

Dann nehme lieber | anstelle von +

[Chris (Gast)]

Wo liegt der unterschied?

[Torsten C. (torsten_c)]

Das Ergebnis ist das gleiche.

Es geht ov.board um den guten Programmier-Stil:

| ist "oder" und das versteht man auch ohne Kommentar als "bit setzen"

Falls man mehrere "oder" auf mehrere Zeilen verteilt, und ein Bit 
versehentlich doppelt mit | setzt, ist das OK. Doppelt mit + ergibt 
einen Fehler.

[Chris (Gast)]

Ok, danke :)

[Konrad S. (maybee)]

Chris schrieb:
> PORTB = (1<<0) + (0<<1) + (1<<2);
...
>       case 7:

Das Plus ist nicht schön, aber es ist nicht das, was mich stört.

[hübscher (Gast)]

Konrad S. schrieb:
> Chris schrieb:
>> PORTB = (1<<0) + (0<<1) + (1<<2);
> ...
>>       case 7:
>
> Das Plus ist nicht schön, aber es ist nicht das, was mich stört.

Ich finde es nicht häßlich, auf jeden Fall aber interessant.

[ov.board (Gast)]

Mit "PORTB = ... (0<<1) ..." willst du wahrscheinlich kein PullUp 
aktivieren oder?

[Chris (Gast)]

Vielen Dank für den Hinweis, das is mir wirklich nicht aufgefallen ...

[Torsten C. (torsten_c)]

Und?

Geht es nun mit "PINB & 0x07"?

[Chris (Gast)]

Jap hat funktioniert ;)