SPRINTF funktioniert nicht!

[Andre P. (stanleyku)]

Hallo Zusammen,

ich habe ein wahrscheinlich ziemlich triviales Problem. Ich möchte eine 
Zahl in eine Zeichenkette anhängen und auf meine 4x20 LC Display 
anzeigen. Die Zahlen funktionieren nur rein gar nicht. Ich weiß nicht 
mehr woran es noch liegen kann. Ich programmiere in der Umgebung von 
MicroC Pro und habe einen PIC18F66K80. Hier mein Code:

/*******************************************************************************
     MCU:             PIC18F66K80
     Oscillator:      HS, 18.0000 MHz
 ******************************************************************************/
 #define             IP_adress            PORTF


char lcd_line1[20], lcd_line2[20], lcd_line3[20], lcd_line4[20]   ;
unsigned char IDmodule=0;


// Lcd module connections
sbit LCD_RS at LATD4_bit;
sbit LCD_EN at LATD6_bit;
sbit LCD_D4 at LATD0_bit;
sbit LCD_D5 at LATD1_bit;
sbit LCD_D6 at LATD2_bit;
sbit LCD_D7 at LATD3_bit;

sbit LCD_RS_Direction at TRISD4_bit;
sbit LCD_EN_Direction at TRISD6_bit;
sbit LCD_D4_Direction at TRISD0_bit;
sbit LCD_D5_Direction at TRISD1_bit;
sbit LCD_D6_Direction at TRISD2_bit;
sbit LCD_D7_Direction at TRISD3_bit;
// End Lcd module connections

//------------------------------------------------------------------------------
//                          Startup Configuration
//------------------------------------------------------------------------------
void configure_mcu(void)
{

  
    ANCON0 = 0x00;
    ANCON1 = 0x00;                     // No analog inputs

    TRISA = 0x00;
    LATA = 0x00;

    TRISC = 0;                                                // set PORTC as output
    PORTC = 0;                                                // clear PORTC

    TRISD = 0;
    TRISE = 16;                                             // RE4 as input, RE5 as output
    WPUB = 0x00;                                            // Disable all PORTB pull-up resistors
    TRISF = 0xFF;                                                // set PORTF as input
}

void main() {
configure_mcu();
Lcd_Init();                        // Initialize Lcd
     IDmodule = 1; 
     
    Lcd_Cmd(_LCD_CLEAR);               // Clear display
    Lcd_Cmd(_LCD_CURSOR_OFF);          // Cursor off
    Delay_ms(100);

    sprintf(lcd_line1, "Modul ID = %d", IDmodule);
    Lcd_Out(1,1, lcd_line1);
    Delay_ms(100);

    sprintf(lcd_line2, "Length = %d", IDmodule);
    Lcd_Out(2,1, lcd_line2);         // Write text
    Delay_ms(20);
    
    sprintf(lcd_line3, "Rx_ID = %d", IDmodule);
    Lcd_Out(3,1, lcd_line3);         // Write text

    sprintf(lcd_line4, "Payload = %d", IDmodule);
    Lcd_Out(4,1, lcd_line4);         // Write text
}

Auf dem Display erscheint gerade:

Zeile1: [Modul ID = 3585]
Zeile2: [Length =  = 3041]
Zeile3: [Rx_ID = 3329]
Zeile4: [Payload = 3073]

Die Zahlen ändern sich teilweise wenn der uC neu bespielt wird. Aber 
warum sind die überhaupt unterschiedlich obwohl überall der gleiche Wert 
1 stehen müsste?! Übertragen wird das .hex-file mit einem PicKit3.

Kann mir jemand weiter helfen?

VG und Danke!!!

[Peter D. (peda)]

"%d" erwartet ein int (16bit) als Argument.

printf("%d", (int)IDmodule);

Es gibt aber auch oft einen Formatstring für 8Bit-Zahlen, mußt mal in 
Deinem Compilermanual nachschauen.

[Andre P. (stanleyku)]

EY DANKE!!!! War zu einfach.

[Peter II (Gast)]

Peter D. schrieb:
> "%d" erwartet ein int (16bit) als Argument.printf("%d", (int)IDmodule);
> Es gibt aber auch oft einen Formatstring für 8Bit-Zahlen, mußt mal in
> Deinem Compilermanual nachschauen.

wird nicht jeder Parameter von einer variablen Argumentenliste 
mindestens auf int erweitert?

[Eric B. (beric)]

Andre P. schrieb:
> char lcd_line1[20], lcd_line2[20], lcd_line3[20], lcd_line4[20]   ;

1) Pass auf, die sind alle 1 char zu "kurz". Du kannst 20 Zeichen auf 
dem Display darstellen und brauchst dann noch einen NUL ('\0') als 
String-Ende.

2) Warum das nicht als 'char lcd_lines[4][21]' declarieren?

[Andre P. (stanleyku)]

Aus Mangel an Erfahrung... Habs umgesetzt. Vielen Dank!

[Rolf M. (rmagnus)]

Peter D. schrieb:
> "%d" erwartet ein int (16bit) als Argument.printf("%d", (int)IDmodule);
> Es gibt aber auch oft einen Formatstring für 8Bit-Zahlen, mußt mal in
> Deinem Compilermanual nachschauen.

Hmm, dann ist dieses printf (bzw. der Compiler) aber nicht 
standardkonform, denn eigentlich müsste bei varargs alles, was kleiner 
als int ist, erst auf int erweitert werden.