ZitierenBei einem PC ist ein "int" üblicherweise 32-Bit lang, ein "short" 16
Auf einem µC ist das aber nicht gesagt.
zuständig ist meistens "types.h"
Schau mal in das Compilerlisting, was denn nun Sache ist.
Hi
ich habe heute zwar schon in einem anderen Forum geschrieben aber keine Antwort bekommen. Die Anzahl der Leute die dieses Forum betrachten ist denke ich zu gering. Versuche ich es hier.
Ich arbeite mit mikroC PRO und habe in meinem Programm einen Fehler den ich nicht finde.Der Mikrocontroller ist ein PIC16F873A.
Projekt: Impulse zähle. Nach einer zuvor eingestellten Soll Zahl soll bei erreichen der Soll Zahl ein Impuls ausgegeben werden.Soll- und Ist-Zahl werden im EEprom gespeichert.
Das Problem ist das die Werte im laufenden Betrieb nicht richtig verglichen werden. Sie werden nur einmal richtig verglichen, und das ist gleich nach dem Brennen des Pic mit mikroC PRO. Ich vermute den Fehler bei den Datentypen. Habe aber schon alles mögliche probiert.Bekomme es nicht hin.Die Stelle an der verglichen wird ist in main: if(((int)istwert1)==((int)sollwert1)).
Hat jemand eine Idee?
Code:// Lcd pinout settings sbit LCD_RS at RC0_bit; sbit LCD_EN at RC1_bit; sbit LCD_D4 at RC2_bit; sbit LCD_D5 at RC3_bit; sbit LCD_D6 at RC4_bit; sbit LCD_D7 at RC5_bit; // Pin direction sbit LCD_RS_Direction at TRISC0_bit; sbit LCD_EN_Direction at TRISC1_bit; sbit LCD_D4_Direction at TRISC2_bit; sbit LCD_D5_Direction at TRISC3_bit; sbit LCD_D6_Direction at TRISC4_bit; sbit LCD_D7_Direction at TRISC5_bit; //########################## globale Variablen ##################### + char txt1[] = "Druckeransteu."; unsigned short menue = 0; unsigned int sollwert1,zaehler=0; //########################## Funktionsdekleration ################## unsigned short rotary(); //unsigned short anzeige(unsigned int); unsigned int eprom_lesen(unsigned short); unsigned short eprom_schreiben(unsigned short,unsigned int); //################## Funktionen ######################### void anzeige(unsigned int x) { unsigned short anzeige[10] ; IntToStr(x,anzeige); Lcd_Out(2,1,anzeige); // LCD_Chr_CP(46); // ch = x % 10; // LCD_Chr(2,12,48+ch); } //-------------------------------------------------------- unsigned short rotary_stell(unsigned short adresse) //Sollwert einstellen { unsigned short wert1, wert2, e = 1, anzeige[10] ; unsigned int i ; if (adresse == 0x00 ) {i = eprom_lesen(adresse) ;} // if (adresse == 0x02 ) {i = eprom_lesen(adresse) ;} Lcd_Cmd(_LCD_CLEAR); Lcd_Out(1,2,"Sollstueckzahl"); Lcd_Out(2,3,"+/-"); IntToStr(i,anzeige); Lcd_Out(2,6,anzeige); while ( e == 1 ) { wert1 = (PORTB & 0x18); //aktuelle stellung nach wert1 if ((wert1 ^ wert2) >= 1) //wurde der Rotary bewegt oder wert1 = wert2 { wert2 = (wert2 << 1 ); //verschieben des zweiten aufgenommenden wertes if ((wert1 ^ wert2) == 0x10) {i ++;} if ((wert1 & wert2) == 0x10) {i -- ;} if (i > 200) {i = 1 ;} if (i < 1) {i = 200 ;} wert2 = wert1 ; IntToStr(i,anzeige); Lcd_Out(2,6,anzeige); } if (Button(&PORTB, 5, 20, 1) && e == 1) { while((PORTB & 0x20)==0x20); e =0; if (adresse == 0x00 ) {eprom_schreiben(adresse,i );} // if (adresse == 0x02 ) {eprom_schreiben(adresse,i );} LCD_Cmd(_LCD_CLEAR); delay_ms(600); LCD_Out(2,6,anzeige); LCD_Out(2,8,"/"); } } return e; } //---------------------------------------------------------- unsigned int eprom_lesen(unsigned short adr) { int zahl; unsigned short temp; temp = Eeprom_Read(adr); zahl = temp; zahl = zahl <<8; temp = Eeprom_Read(adr+1); zahl = zahl + temp; return zahl; } unsigned short eprom_schreiben(unsigned short xxx,unsigned int xxxx ) { unsigned int temp; temp = xxxx; Eeprom_Write ( xxx+1,temp ); temp = (xxxx>>8); Eeprom_Write ( xxx,((short)temp) ); return 0; } //####################### main ####################### void main() { unsigned short enter=0 ; //unsigned long ; unsigned int istwert1; char sollwert2[7],istwert2[7]; TRISA = 0x3F; //Eingang TRISB = 0x38; //Ausgang und Eingang 00111000 TRISC = 0x00; //Ausgang PORTA = 0x00; //Alle auf 0 PORTB = 0x00; //Alle auf 0 PORTC = 0x00; //Alle auf 0 INTCON = 0x00; OPTION_REG = 0x80 ; PIE1 = 0x00 ; PIR1 = 0x00 ; CCP1CON = 0x00 ; ADCON1 = 0x06; //alle Digital Lcd_Init(); // Initialize LCD delay_ms(200); Lcd_Cmd(_LCD_CLEAR); // Clear display Lcd_Cmd(_LCD_CURSOR_OFF); // Cursor off Lcd_Out(1,2,txt1); delay_ms (2000); Lcd_Cmd(_LCD_CLEAR); sollwert1=eprom_lesen(0x00); IntToStr(sollwert1,sollwert2); LCD_Out(1,5,"Aktuell"); LCD_Out(2,6,sollwert2); LCD_Out(2,8,"/"); istwert1 = eprom_lesen(0x03); IntToStr(istwert1,istwert2); LCD_Out(2,1,istwert2); do { if (Button(&PORTB, 5, 20, 1) ) // Menueauswahl { while((PORTB & 0x20)==0x20); enter=0; menue = rotary_stell(enter); // Sollwert einstellen LCD_Out(1,5,"aktuell"); anzeige(istwert1); } if ((PORTA & 0x02)==0x02) { istwert1=istwert1+1; eprom_schreiben(0x03,istwert1 ); anzeige(istwert1); while((PORTA & 0x02)==0x02); } if (((int)istwert1)==((int)sollwert1)) //res=strcmp(istwert2,sollwert2) ; //if(res==0) { PORTB=PORTB|0x80; } else { PORTB=PORTB|0x40; } } while(1); }
Bei einem PC ist ein "int" üblicherweise 32-Bit lang, ein "short" 16
Auf einem µC ist das aber nicht gesagt.
zuständig ist meistens "types.h"
Schau mal in das Compilerlisting, was denn nun Sache ist.
mfg robert
Wer glaubt zu wissen, muß wissen, er glaubt.
Vielleicht mal so versuchen:
Oder:Code://########################## globale Variablen ##################### + char txt1[] = "Druckeransteu."; unsigned short menue = 0; unsigned int sollwert1, istwert1, zaehler=0;
if( ( (int)istwert1 - (int)sollwert1 ) == 0)
Bild hier
Atmel’s products are not intended, authorized, or warranted for use
as components in applications intended to support or sustain life!
Danke PicNick und radbruch. Jetzt habe ich es.
Man war das eine Geburt! Ich glaub es waren mehrere Fehler. Zum einen habe ich vergessen den geänderten Sollwert auch wirklich mit dem Ist Wert zu vergleichen. Habe ihn nur angezeigt aber nicht damit gearbeitet. Und den Rest werden die Datentypen gemacht haben. Da habe ich jetzt mehr unsigned char verwendet. Für meine Zwecke ausreichend.
Sieht zwar noch ein bisschen wüst aus aber wer mal drauf schauen möchte........
Ach so. Falls noch jemand Ideen hat, wie ich meine Programmierung besser machen kann. Nur zu
Code:// Lcd pinout settings sbit LCD_RS at RC0_bit; sbit LCD_EN at RC1_bit; sbit LCD_D4 at RC2_bit; sbit LCD_D5 at RC3_bit; sbit LCD_D6 at RC4_bit; sbit LCD_D7 at RC5_bit; // Pin direction sbit LCD_RS_Direction at TRISC0_bit; sbit LCD_EN_Direction at TRISC1_bit; sbit LCD_D4_Direction at TRISC2_bit; sbit LCD_D5_Direction at TRISC3_bit; sbit LCD_D6_Direction at TRISC4_bit; sbit LCD_D7_Direction at TRISC5_bit; //########################## globale Variablen ##################### + char txt1[] = "Druckeransteu."; unsigned short menue = 0; unsigned int sollwert1,zaehler=0; //########################## Funktionsdekleration ################## unsigned short rotary(); //unsigned short anzeige(unsigned int); unsigned int eprom_lesen(unsigned short); unsigned short eprom_schreiben(unsigned short,unsigned int); //################## Funktionen ######################### unsigned void anzeige(unsigned int x) { unsigned short anzeige[10] ; IntToStr(x,anzeige); Lcd_Out(2,1,anzeige); // LCD_Chr_CP(46); // ch = x % 10; // LCD_Chr(2,12,48+ch); } //------------------------------------------------------------- /* unsigned short rotary(unsigned short enterf) { unsigned short wert1, wert2; short i = 1; LCD_Cmd(_LCD_CLEAR); LCD_Out(1,2,"Soll-Stueckzahl"); while ( enterf == 1 ) { wert1 = (PORTB & 0x18); //aktuelle stellung nach wert1 if ((wert1 ^ wert2) >= 1) //wurde der Rotary bewegt oder wert1 = wert2 { //LCD_Cmd(LCD_CLEAR); wert2 = (wert2 << 1 ); //verschieben des zweiten aufgenommenden wertes if ((wert1 ^ wert2) == 0x02) {i --;} if ((wert1 & wert2) == 0x02) {i ++ ;} if (i >= 1) {i = 1 ;} if (i <= 1) {i = 1 ;} wert2 = wert1 ; // if(i == 1 ) { LCD_Out(1,2,"Differenzwert");} } if (Button(&PORTB, 5, 20, 1) && enterf == 1) { while((PORTB & 0x10)==0x10); enterf =0; } } return i; } */ //-------------------------------------------------------- unsigned short rotary_stell(unsigned short adresse) //Sollwert einstellen { unsigned short wert1, wert2, e = 1, anzeige[10] ; unsigned int i ; if (adresse == 0x00 ) {i = eprom_lesen(adresse) ;} // if (adresse == 0x02 ) {i = eprom_lesen(adresse) ;} Lcd_Cmd(_LCD_CLEAR); Lcd_Out(1,2,"Sollstueckzahl"); Lcd_Out(2,3,"+/-"); IntToStr(i,anzeige); Lcd_Out(2,6,anzeige); while ( e == 1 ) { wert1 = (PORTB & 0x18); //aktuelle stellung nach wert1 if ((wert1 ^ wert2) >= 1) //wurde der Rotary bewegt oder wert1 = wert2 { wert2 = (wert2 << 1 ); //verschieben des zweiten aufgenommenden wertes if ((wert1 ^ wert2) == 0x10) {i ++;} if ((wert1 & wert2) == 0x10) {i -- ;} if (i > 200) {i = 1 ;} if (i < 1) {i = 200 ;} wert2 = wert1 ; IntToStr(i,anzeige); Lcd_Out(2,6,anzeige); } if (Button(&PORTB, 5, 20, 1) && e == 1) { while((PORTB & 0x20)==0x20); e =0; if (adresse == 0x00 ) {eprom_schreiben(adresse,i );} // if (adresse == 0x02 ) {eprom_schreiben(adresse,i );} sollwert1=eprom_lesen(adresse); LCD_Cmd(_LCD_CLEAR); delay_ms(600); LCD_Out(2,6,anzeige); LCD_Out(2,8,"/"); } } return e; } //---------------------------------------------------------- unsigned int eprom_lesen(unsigned short adr) { unsigned int zahl; unsigned short temp; temp = Eeprom_Read(adr); zahl = temp; //zahl = zahl <<8; //temp = Eeprom_Read(adr+1); //zahl = zahl + temp; return zahl; } unsigned short eprom_schreiben(unsigned short xxx,unsigned int xxxx ) { unsigned int temp; temp = xxxx; //Eeprom_Write ( xxx+1,temp ); //temp = (xxxx>>8); Eeprom_Write ( xxx,((short)temp) ); return 0; } //####################### main ####################### void main() { unsigned short enter=0 ; //unsigned long ; unsigned int istwert1; unsigned char sollwert2[7],istwert2[7]; TRISA = 0x3F; //Eingang TRISB = 0x38; //Ausgang und Eingang 00111000 TRISC = 0x00; //Ausgang PORTA = 0x00; //Alle auf 0 PORTB = 0x00; //Alle auf 0 PORTC = 0x00; //Alle auf 0 INTCON = 0x00; OPTION_REG = 0x80 ; PIE1 = 0x00 ; PIR1 = 0x00 ; CCP1CON = 0x00 ; ADCON1 = 0x06; //alle Digital Lcd_Init(); // Initialize LCD delay_ms(200); Lcd_Cmd(_LCD_CLEAR); // Clear display Lcd_Cmd(_LCD_CURSOR_OFF); // Cursor off Lcd_Out(1,2,txt1); delay_ms (2000); Lcd_Cmd(_LCD_CLEAR); sollwert1=eprom_lesen(0x00); IntToStr(sollwert1,sollwert2); LCD_Out(1,5,"Aktuell"); LCD_Out(2,6,sollwert2); LCD_Out(2,8,"/"); istwert1 = eprom_lesen(0x03); IntToStr(istwert1,istwert2); LCD_Out(2,1,istwert2); do { if (Button(&PORTB, 5, 20, 1) ) // Menueauswahl { while((PORTB & 0x20)==0x20); enter=0; menue = rotary_stell(enter); // Sollwert einstellen LCD_Out(1,5,"aktuell"); anzeige(istwert1); } if ((PORTA & 0x02)==0x02) { istwert1=istwert1+1; eprom_schreiben(0x03,istwert1 ); anzeige(istwert1); while((PORTA & 0x02)==0x02); } // istwert1=istwert1+1; // anzeige(istwert1); if (((unsigned char)istwert1)==((unsigned char)sollwert1)) //res=strcmp(istwert2,sollwert2) ; //if(res==0) { PORTB=0x80; istwert1=0; delay_ms(500); anzeige(istwert1); } else { PORTB=0x40; } } while(1); }
Berechtigungen
Lesezeichen