PIC18F97J60 -> C18 Codeoptimierung

Ingo S. (schicki) OP 14.07.2015 20:43 #4198513

Lesenswert?

•

Hallo Zusammen,

ich habe ein kleines Programm geschrieben. Das Programm funktioniert im 
wesentlichen. Bei der Implementierung des kleinen Protokolls bin ich auf 
ein winziges Problem gestoßen.

Der Controller ließt Datum, Uhrzeit und Temperatur aus. Das 
funktioniert. Nun soll die Schnittstelle prüfen, ob Daten anliegen, und 
da liegt das Problem.

Der Controller bleibt mir da immer bei der Prüfung der Schnittsstelle 
stehen, und die Uhrzeit usw. wird nicht weiter aktualisiert. Diese wird 
nur aktualisiert, wenn ich ständig Daten an den Controller stehe. Der 
restlcihe Code funktioniert soweit das ist der einige Makel an der 
ganzen Sache. Irgenwie sehe ich den Wald vor lauter Bäumen nicht mehr. 
Hat jemand eien Tipp wie ich das optimieren kann?

Danke und Gruß
Ingo

Hier wäre der betroffene Auszug aus dem Quellcode:

dt.Stunden = DS1337_Read_Stunden(&dt);
dt.Minuten = DS1337_Read_Minuten(&dt);
dt.Sekunden =DS1337_Read_Sekunden(&dt);
dt.Tag = DS1337_Read_Tag(&dt);
dt.Monat = DS1337_Read_Monat(&dt);
dt.Jahr = DS1337_Read_Jahr(&dt);
dt.Wochentag = DS1337_Read_Wochentag(&dt);
strcpy(chr_Wochentag, Wochentag_Namen[dt.Wochentag]); 
int_Temp_CPU = GetTempLM75(1);
Warten_MS(100);
LcdPrintf(2,0, (const MEM_MODEL rom char *)"%.3s", chr_Wochentag);  
LcdPrintf(2,4, (const MEM_MODEL rom char *)"%.2u", dt.Tag);
LcdPrintf(2,7, (const MEM_MODEL rom char *)"%.2u", dt.Monat);
LcdPrintf(2,12, (const MEM_MODEL rom char *)"%.2u", dt.Jahr);
LcdPrintf(2,15, (const MEM_MODEL rom char *)"%.2u", dt.Stunden);
LcdPrintf(2,18, (const MEM_MODEL rom char *)"%.2u", dt.Minuten);
LcdPrintf(2,21, (const MEM_MODEL rom char *)"%.2u", dt.Sekunden);
LcdPrintf(2,30, (const MEM_MODEL rom char *)"%.4i", int_Temp_CPU);
//if (!DataRdy1USART())
// STX|CMD|VALUE|ETX|\n
while (!DataRdy1USART());
gets1USART(unc_Data,6);
uint_Data_0 = (int)unc_Data[0];
uint_Data_1 = (int)unc_Data[1];
uint_Data_2 = (int)unc_Data[2];
uint_Data_3 = (int)unc_Data[3];

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Malli Gast 14.07.2015 20:51 #4198528

Lesenswert?

•

▲
▼

Ingo S. schrieb:
> Diese wird nur aktualisiert, wenn ich ständig Daten an den Controller  > stehe.

Was ist los? Sind die Eingänge von analog auf digital umgestellt?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ingo S. (schicki) OP 14.07.2015 21:25 #4198575

Lesenswert?

•

▲
▼

Sämtliche Eingänge und Analog-Ports funktionieren wie sie sollen.

Mir bleibt nur das Programm bei der Serielen Schnittstelle hängen. und 
kehrt nicht mehr zum aus dieser Scnleife zurück. Sprich Uhrzeit anzeigen 
usw.

Uhrzeit, Datum usw. aktualisieren

Schnittstelle prüfen -> Daten vorhanden ja -> Daten auswerten und ggf. 
Aktionen auswerten. Geht

keine Daten vorhanden -> Uhrzeit, Datum usw. aktualisieren

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Malli Gast 14.07.2015 21:42 #4198594

Lesenswert?

•

▲
▼

Tut mir Leid aber Du schreibst so verwirrend, dass es einem schwer fällt
das eigentliche Problem zu verstehen.

Dein Controller bleibt hier
while (!DataRdy1USART());

hängen, wenn Du ihm keine Daten sendest, habe ich das richtig
verstanden? Wenn ja, dann würde ich sagen macht der Code genau das was
er soll.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ingo S. (schicki) OP 14.07.2015 21:51 #4198611

Lesenswert?

•

▲
▼

ganz genau.

Er soll prüfen ob Daten da sind wenn ja weiter machen wenn nicht was 
anderes machen. Wie würdest Du dies tun?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Malli Gast 14.07.2015 21:56 #4198612

Lesenswert?

•

▲
▼

Warum nicht einfach if else?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ingo S. (schicki) OP 14.07.2015 21:58 #4198616

Lesenswert?

•

▲
▼

geht irgendwie nicht. Da ich mehrere Zeichen sende geht das irgendwie 
nicht.

STX|CMD|VALUE|ETX|\n

HAbs geprüft und komm da leider nicht so weiter wie ich will wenn ich 
nur 1 Byte sende dann klappt das aber bei mehreren Bytes leider nicht 
Die while Schleife habe ich bei If then else ausgeklammert...

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Malli Gast 14.07.2015 22:13 #4198635

Lesenswert?

•

▲
▼

Dann lies doch einfach immer ein gesamtes Datenpaket ein, etwa so:

if( keine Daten da )
{
   for( ... ) // Anzahl an Bytes
   {
      Daten einlesen
      while( keine Daten da );
   }
}
else
{
   // ...
}

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Malli Gast 14.07.2015 22:14 #4198638

Lesenswert?

•

▲
▼

Das kann man dann noch beliebig aufbauschen um auch eine Synchronisation 
mit dem Startbyte zu machen etc pp

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Ingo S. (schicki) OP 14.07.2015 22:14 #4198639

Lesenswert?

•

▲
▼

ok, klingt plausibel, teste ich mal danke...

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Malli Gast 14.07.2015 22:27 #4198656

Lesenswert?

•

▲
▼

Schön ist das allerdings nicht. Ich würde das höchstens zum Testen 
verwenden. Arbeite am Besten mit dem Interrupt, den das USART Modul 
auslösen kann, wenn ein Datenpaket eingetroffen ist.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

PIC18F97J60 -> C18 Codeoptimierung

Antwort schreiben