Wie setze ich Ausgänge mit einer for-Schleife?

[stephan b. (stu)]

Hallo zusammen,
ich schreibe gerade ein kleines Programm für ein Projekt in der 
Technikerschule. Und zwar muss ich mit einem Atmega32 8 Analoge Eingänge 
abfragen. Das ist auch kein Problem. Ich habe in einer for-Schleife 
einen AnalogDigitalwandler aufgerufen der den Wert des jeweiligen 
Eingang in ein Array "Fach[]" schreibt. Jetzt möchte ich in einer 
weiteren for-Schleife die Ausgänge in Port D setzen wenn der Wert 
innerhalb einer Toleranz liegt. Also wenn Fach[1] kleiner 800 und größer 
500 dann setze PortD 1. Der Code sieht bisher so aus:

void ADC_init(void)
{
  uint16_t result;
  
  ADMUX = (1<<REFS1) | (1<<REFS1);    
  ADCSRA = (1<<ADPS1) | (1<<ADPS0);    
  ADCSRA |= (1<<ADEN);          
  
  ADCSRA |= (1<<ADSC);          
  while(ADCSRA & (1<<ADSC)){}        
  result = ADCW;              
}

uint16_t ADC_Read (uint8_t channel)
{
  ADMUX = (ADMUX & ~(0x1f)) | (channel & 0x1f);
  ADCSRA |= (1<<ADSC);          
  while (ADCSRA & (1<<ADSC)){}      
  return ADCW;              
}

int main(void)
{
  int i;
  uint16_t Fach[8];
  for (i=0; i<7; i++)
  {  
    ADC_Read(i);
    Fach[i] = ADCW;
  }  
  
  for (i=0; i<7; i++)
  {
    if (Fach[i] < 800)
    {
      if (Fach[i] > 500 )
      {
        PORTC |= (1<<PD[i]);
      }
    } 
    else
    {
      PORTC &= ~(1<<PD[i]);
    }
  }
}

Ich müsste jetzt wissen wie der Syntax ist um PortD[i] zu setzen. So 
nimmt AVR-Studio das Programm nicht an.

Schonmal danke im voraus.

[Coder (Gast)]

1. Wenn Du schon Port C setzen kannst, warum du das nicht auf Port D 
übertragen.

2. Was ist PortD[]  ?

3. Quellcode unvollständig

[Karl H. (kbuchegg)]

stephan blaha schrieb:


> Ich müsste jetzt wissen wie der Syntax ist um PortD[i] zu setzen.

Du kannst nicht PortD[i] setzen.

Aber du kannst einen Ausgang setzen, indem du mit einem Bitmuster 
veroderst. Das Bitmuster sagt aus, welcher/welche Pins zu setzen sind. 
Dieses Bitmuster kannst du allerdings entsprechend aus i ableiten.

[Karl H. (kbuchegg)]

   if (Fach[i] < 800)
    {
      if (Fach[i] > 500 )
      {
        PORTC |= (1<<PD[i]);
      }
    } 
    else
    {
      PORTC &= ~(1<<PD[i]);
    }

da fehlt ein else Zweig. Was soll passieren wenn dein Fach[i] kleiner 
als 500 ist.

Alternativ (und besser ist es allerdings) wenn du in diesem Fall deine 
Beschreibung wörtlich nimmst:
> Also wenn Fach[1] kleiner 800 und größer 500 dann
denn dann sparst du dir einen else Zweig und kannst den daher auch nicht 
vergessen.

  if (Fach[i] > 500 && Fach[i] < 800)
    PORTC |= (1 << i );
  else
    PORTC &= ~( 1 << i );

[stephan b. (stu)]

  for (i=0; i<7; i++)
  {
    if (Fach[i] < 800)
    {
      if (Fach[i] > 500 )
      {
        PORTC |= (1<<PD[i]); //Hier habe ich wohl zu schnell geändert
      }
    } 
    else
    {
      PORTC &= ~(1<<PD[i]); //ursprünglich sollte Port C gesetzt werden
    }
  }

Ich wollte aber ungern für jeden Ausgang eine If Bedingung machen in der 
ich das einzelne Bit setze bzw. Rücksetze. ich finde es in einer for 
Schleife eleganter und vor allem weniger arbeit. Wenn es allerdings nur 
so geht werde ich wohl für jeden Eingang eine If Anweisung schreiben 
müssen.
Trotzdem danke

[Alex (Gast)]

for (i=0; i<7; i++)
{
   PORTC = (PORTC && ~(1<<i)) || (((int)Fach[i] > 500 && Fach[i] < 800) << i);
}

[Karl H. (kbuchegg)]

stephan blaha schrieb:

> ich finde es in einer for
> Schleife eleganter und vor allem weniger arbeit. Wenn es allerdings nur
> so geht werde ich wohl für jeden Eingang eine If Anweisung schreiben
> müssen.

Schau dir mal den 2.ten Teil meiner 2.ten Antwort GENAU an.

[Krapao (Gast)]

  uint8_t zustand = PORTC;
  for (i=0; i<7; i++)
  {
    if ( Fach[i] < 800 )
    {
      if ( Fach[i] > 500 ) 
      {
        zustand |= (1<<i);
      }
    } 
    else
    {
      zustand &= ~(1<<i);
    }
  }
  PORTC = zustand; // Einmaliges Setzen

Von der Programmlogik her halte ich deinen Code für nicht durchdacht, 
denn er hat eine logische Lücke für Fach[] <= 500.

Welchen Wert bekommt der Portpin dann zugewiesen? Ich habe das mal so 
gemacht wie du es geschrieben hast, nämlich

Fach[] <= 500                Pin in PORTC unverändert lassen
Fach[] > 500 && Fach[] < 800 Pin in PORTC setzen
Fach[] >= 800                Pin in PORTC löschen