Hilfe bei Konzeptaufbau

[strategielos (Gast)]

Hallo Leute,
ich benötige Hilfe beim erstellen eines besseren Konzepts für eine 
einfache Steuerung.
Zur Steuerung, ich muß ein Mischventil je nach Temperatur weiter öffnen 
oder schließen. Leider hat dieses Ventil keine Rückmeldung wo es sich 
befindet.
Daher muß ich beim einschalten der Steuerung zuerst das Ventil für eine 
gewisse Zeit schließen damit es sicher zu ist(Ventil besitzt Endschalter 
die aber nicht nach außen geführt sind).
Der Programmteil denn ich so sch...e finde sieht so aus.

if (PINA & (1<<IN_ANFORDERUNG))
    {
      testheizen=0;
      if (testheizenaus<10)
      {
        testheizenaus++;
      }
      else
      {
        anforderung = FALSE;
      }
    }
    else
    {
      testheizenaus=0;
      if (testheizen<10)
      {
        testheizen++;
      }
      else
      {
        anforderung = TRUE;
      }
    }
      if (!anforderung||reset_Mischer)
    {  
      
      if (!einmal)
      {
        if  (!reset_Mischer )
        {
          resetzeit = INIT_ZEIT;
          PORTA |= (1<<OUT_MISCHER_AUF);// sicherstellen das nicht geöffnet wird!
          PORTA &= ~(1<<OUT_MISCHER_ZU);
          reset_Mischer =TRUE;
        }
        if( (resetzeit==0)&&(reset_Mischer) )
        {
          PORTA |= (1<<OUT_MISCHER_ZU);
          state=0;
          reset_Mischer = FALSE;  
          einmal = TRUE;      
        }
      }        
      PORTA |= (1<< OUT_PUMPE);
      heizen = FALSE;
    }
    else
    {
      einmal = FALSE;
      heizen = TRUE;            
    }
              
      
    
    
    if (heizen)  // Anforderung Raumtermostate
    {
      switch(state)
      {
        case 0:
        {
          if ( !(umwaelzen) )  // es wird nich nicht umgewälzt
          {
            umwaelzen = TRUE;
            umwelzzeit = MENU_VARIABLE[VorUmwaelzzeit];
            PORTA &= ~(1<<OUT_PUMPE);
            printf_P(buffer,"Umwälzen für%2i Sekunden\n\r",umwelzzeit);
            uart_puts(buffer);
              
          }
          if ( (umwelzzeit==0)&&(umwaelzen) )
          {
            umwaelzen = FALSE;
            PORTA |= (1<<OUT_PUMPE);
            state = 1;
          }              
          break;      
        }
        case 1:
        {
          //PORTA &= ~(1<<OUT_PUMPE);
          if(Temp_array[VORLAUF]< (MENU_VARIABLE[SollTempVorlauf]-MENU_VARIABLE[Hysterese]))
          {  
            if ( !(move_mischer) )
            {
              move_mischer = TRUE;
              regelzeit = MENU_VARIABLE[Regelzeit_Auf];
              PORTA |= (1<<OUT_MISCHER_ZU);// sicherstellen das nicht geschloßen wird!
              PORTA &= ~(1<<OUT_MISCHER_AUF);
            }
            if( (regelzeit==0)&&(move_mischer) )
            {
              move_mischer = FALSE;
              PORTA |= (1<<OUT_MISCHER_AUF);
              state = 2;        
            }
          }
          else if (Temp_array[VORLAUF]> (MENU_VARIABLE[SollTempVorlauf]+MENU_VARIABLE[Hysterese]))
          {
            if ( !(move_mischer) )
            {
              move_mischer = TRUE;
              regelzeit = MENU_VARIABLE[Regelzeit_Zu];
              PORTA |= (1<<OUT_MISCHER_AUF);// sicherstellen das nicht geöffnet wird!
              PORTA &= ~(1<<OUT_MISCHER_ZU);
            }
            if ( (regelzeit==0)&&(move_mischer) )
            {
              move_mischer = FALSE;
              PORTA |= (1<<OUT_MISCHER_ZU);
              state = 2;
            }    
          }
          else
          {
            state=2;
          }          
          break;
        }
        case 2:
        {
          //PORTA &= ~(1<<OUT_PUMPE);
          if(! (delayreaktion) )
          {
            delayreaktion = TRUE;
            regelzeit = MENU_VARIABLE[Regelintervall];
          }
          if( (regelzeit ==0)&&(delayreaktion) )
          {
            delayreaktion =FALSE;
            state =1;          
          }
          break;
        }
      }
      if (!(umwaelzen))
      {
        if (Temp_array[PUFFER]>(MENU_VARIABLE[PufferMinTemp]))  // Kontrolle Temperatur Puffer!
        {
          PORTA &= ~(1<<OUT_PUMPE);
          puffertemp_to_lo = FALSE;
        }      
        else if(Temp_array[PUFFER]<((MENU_VARIABLE[PufferMinTemp])-2))
        {
        
          PORTA |= (1<<OUT_PUMPE);
          puffertemp_to_lo = TRUE;
        }  
      }          
    }
    else  // Keine Heizungs anforderung!
    {
      state=0;
      PORTA |= (1<< OUT_PUMPE);    
    }
ISR (TIMER0_OVF_vect)      // every 1ms
{
  TCNT0 = (unsigned char)(signed short)-(F_CPU / 1024 * 10e-4 + 0.5);  // preload for 1mer
  encoder();
  prellhelp--;
  if (prellhelp == 0)
  {
    prellhelp=10;
    entprell();  
  }    
  runtimer--;
  if (runtimer == 0)
  {
    runtimer = runperiod;
    PORTD ^= 1<<7;
  
  }
  sec--;
  if (sec == 0)
  {
    toggle++;
    sec = SEC;
    if (umwelzzeit!=0)umwelzzeit--;
    if (regelzeit!=0)regelzeit--;
    if (resetzeit!=0)resetzeit--;  
  }

irgend wie werde ich den Spagetti Code nicht los!
Bitte gebt mir einen Rat wie ihr das lösen würdet.
Vielen Dank

Konzeptlos

[PittyJ (Gast)]

Zumindest würde ich dem State ordentliche Namen geben. Nach ein paar 
Wochen weiss man nicht mehr, was State 2 bedeutet.

Also so etwas wie

enum
{
  Auf,
  Zu,
  Kalibrieren
} State;


..

State = Auf;
..

case Auf:

Weiterhin könnte man mal schauen, ob man nicht Teile in Funktionen 
auslagert.

[strategielos (Gast)]

PittyJ schrieb:
> Weiterhin könnte man mal schauen, ob man nicht Teile in Funktionen
> auslagert.

Ja genau das wollte ich habe aber keinen Plan was ich in Funktionen 
bringe.

Danke für deine >Hilfe

Konzeptlos

[PittyJ (Gast)]

Damals im Studium hiess es: alles was länger als ein Bildschirm ist, 
bekommt eine extra Procedure. Das ist vielleicht etwas hart.

Aber du hast doch selber schon 5 Leerzeilen zwischen 2 Abschnitte 
gesetzt.
Jetzt mach einfach aus den beiden Abschnitten eine Funktion.

Zweimal ist ein ähnlicher Block beginnend mit
  if ( !(move_mischer) )
Auch eine Function mit Parameter auf/zu.

usw.