Kommandos elegant auswerten, aber wie?

[Elias B. (bouni)]

Hallo allerseits,

ich habe ein Programm geschrieben um einen Getränkeautomaten über die 
Serielle Schnittstelle anzusprechen um ihn Getränke ausgeben zu lassen.
Das funktioniert auch schön, der Code gefällt mir aber nicht.

Ich werte fortlaufend ein Byte aus dem Ringpuffer aus, bis ein Kommando 
kommt das für mich bestimmt ist. Dann springe ich in die entsprechende 
Funktion und werte die nächsten paar Bytes aus die laut Spezifikation 
des MDB Protokolls kommen sollten und bile die Checksumme.

Hier am Beispiel der Reset Funktion

/*
 * MDB Setup Function
 */ 
uint8_t mdb_setup(void){                 
  //wait until data is in buffer
  while(mdb_read_buffer() != 1);  
  // read subcommand
  uint8_t subcmd = mdb_data;
  // Subcommand is Setup data
  if(subcmd == 0x00){
    uint8_t checksum = 0;
    checksum += MDB_SETUP;
    checksum += subcmd;
        
    while(mdb_read_buffer() != 1);  
    vmc_cfg_data.feature_level = mdb_data;
    checksum += mdb_data;
  
    while(mdb_read_buffer() != 1);  
    vmc_cfg_data.display_cols = mdb_data;
    checksum += mdb_data;
    
    while(mdb_read_buffer() != 1);  
    vmc_cfg_data.display_rows = mdb_data;
    checksum += mdb_data;

    while(mdb_read_buffer() != 1);  
    vmc_cfg_data.display_info = mdb_data;
    checksum += mdb_data;

    while(mdb_read_buffer() != 1);  
    if(checksum != mdb_data){
      vmc_cfg_data.feature_level  = 0;    
      vmc_cfg_data.display_cols   = 0;    
      vmc_cfg_data.display_rows   = 0;    
      vmc_cfg_data.display_info   = 0;  
      return 0;    
      }

    mdb_usart_send_char(0x01,0x00);     // Send Cashless Device SETUP data
    mdb_usart_send_char(0x01,0x00);     // Send Cashless Device Feature Level
    mdb_usart_send_char(0x19,0x00);     // Send Cashless Device Country Code Highbyte
    mdb_usart_send_char(0x78,0x00);     // Send Cashless Device Country Code Lowbyte
    mdb_usart_send_char(0x05,0x00);     // Send Cashless Device Scale Factor
    mdb_usart_send_char(0x02,0x00);     // Send Cashless Device Decimal Places
    mdb_usart_send_char(0x05,0x00);     // Send Cashless Device App max Response Time (10sec)
    mdb_usart_send_char(0x00,0x00);     // Send Cashless Device Misc Options
    mdb_usart_send_char(0x9F,0x01);     // Send checksum

    while(mdb_read_buffer() != 1);  
    // If ACK is received
    if(mdb_data == 0x00){ return 1; }
    return 0;
  }

/*
 * Weiterer Code . . .
 */
  return 0;  
}

geht es nicht irgendwie schöner, als immer ein Byte zu lesen, es zu 
speichern und der Checksumme hinzuzuaddieren* und dan wieder das nächste 
Byte zu lesen?

Danke schon mal für die Antworten :)

Gruss Elias



* Die Cheksumme bildet sich bei MDB nur durch addition aller Bytes

[Peter D. (peda)]

Ich würde fürs Empfangen und Senden einen FIFO einrichten.
Und den Empfang würde ich erst auswerten, nachdem ein komplettes Paket 
eingegangen ist. Dazu muß man aber das Protokoll kennen, wie ein Paket 
aufgebaut ist.
Dann kann man in der Zwischenzeit andere Tasks ausführen und die CPU 
bleibt nicht hängen, wenn der Empfang gestört ist.
Und man hat keinen unhandlichen Spaghetticode.


Peter