Почему я получаю 2 лишних нуля?

Question

[Денис]

Пытаюсь вывести на lcd дисплей массив 8 байт, я ожидал получить "0xFF0xFF0xFF0xFF0xFF0xFF0xFF0xFF" после выполнения этой программы,

char * read_rom(){
unsigned int i;
char ch[8];
char ch1[1];
for(i=0;i<=8;i++){
temporary[i]=0xFFFF;//RX();
sprintf (ch1, "0x%X", temporary[i]);
lcd_display(ch1);
}
return temporary;
}

но в результате получаю: "0xFF000xFF000xFF000xFF000xFF000xFF000xFF000xFF00"

Объясните неумехе, почему?

Answer 1

[Армянское Радио]

Два бага сразу:
Лишняя итерация цикла:
for(i=0;i<=8;i++){
i или должна быть <8 или <=7
Не использован модификатор длины:
sprintf (ch1, "0x%X", temporary[i]);
Исправление:
sprintf (ch1, "0x%2X", temporary[i]);

Answer 2

[Виталий Пухов]

temporary должен быть как byte[] и temporary[i]=0xFF;

Answer 3

[DancingOnWater]

Вы в 8 битовый знаковый чар впихиваете 16 беззнаковый и выводите знаковый char

Answer 4

[xandox]

к Армянское Радио +
ch1 объвлен как char ch1[1] - то бишь один байт, а вы в него пытаетесь аж 4 запихнуть, C вам конечно слово не скажет, но потом когда-нибудь можете сильно удивиться.

Comments

[Армянское Радио]

Не четыре байта, а два - 0xFFFF=65535 =0b1111'1111'1111'1111;
GCC кстати будет ругаться на такое:
warning: overflow in implicit constant conversion [-Woverflow]

[xandox]

Армянское Радио: он же строку генерирует - в строке каждый символ байт. а то что он тут да пока что кладет 6 байт ситуацию не улучшает.

[xandox]

Там даже 5 байт. Ноль же еще.