Bitte um Codebewertung/ µ-Controller Neuling

[Dominik T. (locaded)]

Erstmal Hallo an alle Forenuser,

ich habe angefangen mich mit µ-Controllern zu beschäftigen und freue 
mich nun , mich mit euch hier in diesem Forum austauschen zu können.

Ich habe ein kleines µ-Controller Programm zum testen geschrieben, und 
würde euch nun bitten es kurz zu bewerten, auf Funktion zu prüfen und 
Verbesserungsvorschläge zu geben, bevor ich es auf den Controller 
überspiele.

Ich möchte gernen einen Attiny 2313 verwenden, mit einem externen 8MHZ 
Kristall.
Das Programm soll die Serielle Schnittstelle regelmäßig in einer 
Schleife  auslesen und bei
dem anliegenden Datensignal "1" im Uart Data Register, Ausgänge 
verzögert setzten.

 

//This File check the Seriell Reciever Input and switch corresponding   
//Outputs        

#include <avr/io.h> 
#define F_CPU 8000000UL
#include <util/delay.h> 
#include <avr/interrupt.h> 


void init_ports(void); 
void init_USART(unsigned int ubrr_value); 
void Aus_Ein_Next(); 


/////////////////////////////////////////////////////////////////////////////////////

int main(void) 
{ 
// Initialisation Ports, seriell Transmission, Interrupts
   init_ports(); 
    
   init_USART(51);//8M/(16*9600)-1 
   //sei();//Global Interrupt Enable 
/////////////////////////////////////////////////////////////////////////////////////
//Check UDR DATA and select Programm 
    while(1) 
    { 
      if ( UDR == 1) 
      Aus_Ein_Next() 
                
    } 
} 

/////////////////////////////////////////////////////////////////////////////////////
void init_ports(void) 
{ 
//Port D pins 1 to 3 Config as Digital Out 

   DDRD=(1<<DDD4)|(1<<DDD5)|(1<<DDD6); 
} 
/////////////////////////////////////////////////////////////////////////////////////
void init_USART(unsigned int ubrr_value) 
{ 
   //Set Baud rate 
   UBRRL = (unsigned char) ubrr_value;
   UBRRH = (unsigned char) (ubrr_value>>8);

   /*Set Frame Format 
   >> Asynchronous mode 
   >> No Parity 
   >> char size 8 
   */ 

   UCSRC=3<<UCSZ0; // contains UCSZ1,too ( leftshift of 3 not 1)


   //Enable The receiver, transmitter and interrupt 
   UCSRB=(1<<RXEN)|(1<<TXEN); //(1 << RXCIE) for interrupts (sreg 1 rxc, in ucsra 1)
} 
/////////////////////////////////////////////////////////////////////////////////////
void Aus_Ein_Next()
{ 

PORTD = 1<< PB4 ; //   PB4 -->  AUS
_delay_ms(2000);
PORTD = 1<< PB5 ; //    PB5 --> EIN
_delay_ms(2000);
PORTD = 1<< PB6 ; //    PB6 --> NEXT
}

Ich freue mich auf Eure Antworten!

MfG Domi

[Piefke (Gast)]

Dominik Tews schrieb:
> bevor ich es auf den Controller
> überspiele.

Was hindert dich, das Programm selber zu testen?

> Ich freue mich auf Eure Antworten!

Mal sehen, wann sich das ändert.

[Peter II (Gast)]

man sollte vorher abfragen ob auch neue Daten anliegen

if ( UDR == 1)

und nicht einfach auslesen, gehen tut das schon aber sauber ist es 
nicht.

[Werner (Gast)]

Mit deinen endlosen Delays in Aus_Ein_Next() verbaust du dir einiges. 
Interruptgesteuerter Empfang der seriellen Daten wäre deutlich 
zukunftsorientierter (und der nächste Schritt, wenn das Polling 
funktioniert)

Dominik Tews schrieb:
> ... bevor ich es auf den Controller überspiele.

Das tut nicht weh und kostet nix. Aber spätestens, wenn es dort nicht 
funktioniert, ist es an der Zeit, sich den Ablauf Schritt für Schritt im 
Simulator anzusehen.

[Ingo (Gast)]

Deine Formatierung ist schlecht, mal mit und mal ozne Leerzeichen. 
Versuchs einheitlich

[Dominik T. (locaded)]

Danke für die schnellen Antworten. Werde versuchen, Eure Vorschläge 
umzusetzen.

@Piefke, leider ist meine bestellte Hardware noch nicht da.
         Und ich hoffe mal ich freu mich immer auf Eure antworten.


@Peter II, ich hab leider keine Programmierausbildung, weshalb mein 
Programmierstil bestimmt nicht der beste ist.
Ich bin aber gerne bereit mich verbessern zu lassen :) .
Als nächstes versuch ich also raus zu finden, wie ich abfragen kann, ob 
neue Daten anliegen.

@Werner, ja an Interrupts habe ich auch schon gedacht, mich aber 
irgendwie noch nicht ran getraut.

@Ingo, oh ja, da hast du Recht! Ordnung ist wichtig. Werde ich gleich 
verbessern.
Update: hab gerade bemerkt, das es schon zu spät ist, den ersten Beitrag 
zu editieren. Die schlechte Formatierung bleibt also zumindest in diesem 
Beitrag schlecht. Aber in meinem Programm wurde die Formatierung bereits 
verbessert.


MfG Domi

[Piefke (Gast)]

Dominik Tews schrieb:
> @Piefke, leider ist meine bestellte Hardware noch nicht da.

Dann warte ab, bis sie da ist.
Programmierprobleme lassen sich nicht virtuell lösen. Gerade als 
Anfänger muß man Schritt für Schritt vorgehen und nach jedem Schritt neu 
testen.

Da du noch keine Hardware hast, hast du sicherlich auch noch keine LED 
zum Blinken gebracht.Fange damit an!
Und zwar das Programm selber zu schreiben und nicht zu sehen, dass bei 
anderen Leuten das alles so einfach aussieht.
Zum Schwimmernlernen braucht man auch Wasser und kein Video, wie Andere 
das machen.