R2R Netzwerk Problem

[Sebastion W. (Gast)]

Hallo Liebe Forum Nutzer!

Im folgenden Programm wird mittels 5 Schalter eine Spannung von 1-5 Volt 
ausgewählt, danach wird mit einem anderen Schalter ein Interrupt 
ausgelöst, der die Bitmaske auf den PORTD überträgt, damit man die reale 
Spannung anliegen hat. Nur mein Problem ist es, dass man die Spannung 
nicht 0 setzen, kann, also wenn alle Schalter aus sind und der Interrupt 
auslöst. Wenn kein Schalter an ist, und ich den Interrupt auslöse, 
bleibt die Spannung gleich dem vorherigen Wert.

Schonmal vielen Dank für deine Hilfe!

#ifndef F_CPU
#define F_CPU 16e6                      //Takt der CPU einstellen, 16MHz
#endif

#include <avr/io.h>
#include <avr/interrupt.h>

#define S1INIT    (DDRC  &=~(1<<PC0)) // DDRB  Bit2 löschen (als Eingang konfigurieren)
#define S1STATUS  (PINC  &  (1<<PC0)) // PortB Bit2 lesen   (high-aktiv)
 
#define S2INIT    (DDRC  &=~(1<<PC1)) // DDRB  Bit3 löschen (als Eingang konfigurieren)
#define S2STATUS  (PINC  &  (1<<PC1)) // PortB Bit3 lesen   (high-aktiv)

#define S3INIT    (DDRC  &=~(1<<PC2)) // DDRB  Bit2 löschen (als Eingang konfigurieren)
#define S3STATUS  (PINC  &  (1<<PC2)) // PortB Bit2 lesen   (high-aktiv)
 
#define S4INIT    (DDRC  &=~(1<<PC3)) // DDRB  Bit3 löschen (als Eingang konfigurieren)
#define S4STATUS  (PINC  &  (1<<PC3)) // PortB Bit3 lesen   (high-aktiv)

#define S5INIT    (DDRC  &=~(1<<PC4)) // DDRB  Bit3 löschen (als Eingang konfigurieren)
#define S5STATUS  (PINC  &  (1<<PC4)) // PortB Bit3 lesen   (high-aktiv)

uint8_t output;        //Byte Variable output

ISR(PCINT1_vect){      //Interrupt Service Routine

  PORTD = output;      //Ausgänge setzen
}

int main(void){  

    DDRD = 0xFF;        //PORTD output
  
  sei();                  //Enable Global Interrupts ("cli();" zum deaktivieren)

  PCICR  = 1<<PCIE1;      //Pin Change Interrupt Control Register, PCIE1 flag HIGH
    PCMSK1 = 1<<PCINT13;    //Pin Change Mask Register 1, Pin Change Interrupt on PCINT11 (PC5)

  S1INIT;
  S2INIT;
  S3INIT;
  S4INIT;
    
  while (1){  

    //1 Volt
    switch(S1STATUS){

      case 1: output = 51; 
      break;      
    }
    
    //2 Volt
    switch(S2STATUS){

      case 1: output = 102;   
      break;    
    }

    //3 Volt
    switch(S3STATUS){

      case 1: output = 153; 
      break;      
    }

    //4 Volt
    switch(S4STATUS){

      case 1: output = 204;   
      break;    
    }

    //5 Volt
    switch(S5STATUS){

      case 1: output = 255;
      break;
        
      }
  }
}

[MaWin (Gast)]

Du solltest output als volatile deklarieren.

Aber ebenso solltest du mal ein Kapitel über das Entprellen von Tastern 
lesen.

Und die Übersichtlichkeit durch Worte wie "INIT" und "STATUS" ist auch 
miserabel.

Ein switch case 1 könnte einfacher mit einem if realisiert werden.

Und die Funktion des Programms wenn mehr als 1 Schalter aktiv ist, ist 
ebenfalls zweifelhaft.

Und ob FCPU wirklich sinnvollerweise eine Floatingpointzahl sein 
sollte....

[Jim M. (turboj)]

Sebastion W. schrieb:
> Wenn kein Schalter an ist, und ich den Interrupt auslöse,
> bleibt die Spannung gleich dem vorherigen Wert.

Ja. Genau das hast Du so programmiert. Wenn Du eine andere 
Funktionalität haben wolltes musst Du was umprogrammieren.

Wegen Deiner ziemlich hirntoten switch(SxSTATUS) Methode ist das sehr 
viel komplizierter als nötig.


Hint:

switch (PINC &0x1F) {

  case 1:
// ...
  case 2:
// ...
}

[Ratgeber (Gast)]

Am besten fängst Du mit dem Tutorial hier im Forum an.

Bei Deinem Problem ist einfach zu viel falsch, dass eine einzelne oder 
auch 2 oder 3 Korrekturen, die Lösung sein könnten. Das wäre nur der 
Fall, falls das Konzept und Dein Verständnis vom Programmieren und Deine 
C-Kenntnisse wenigstens näherungsweise der Aufgabenstellung angemessen 
wären.
Und - ich schreibe das mit höflichen Absichten -: Das ist einfach nicht 
der Fall. Tut mir leid.

[Matthias S. (Firma: matzetronics) (mschoeldgen)]

Andere Idee: Man spart sich den MC und schaltet ein R Netzwerk direkt 
mit Schaltern?

[Sebastion W. (Gast)]

Also dass mein Lehrer nicht so gut ist, wusste ich schon vorher, aber 
dass er so miserabel ist...