Tableau dans Progmem mauvais indice?

Question

J'ai toujours du mal à faire fonctionner ce code correctement. Mon but est de montrer les chaînes qui sont placées dans PROGMEM sur un écran LCD. Le tableau avec des pointeurs vers ces chaînes se trouve également dans PROGMEM. La fonction est appelée avec une variable qui à son tour est traduite en un numéro d'index pour lire le tableau. Ofcourse pgmspace.h est inclus dans le code.

Les erreurs que je continue à recevoir sont: souscript de tableau est de type « char » [-Wchar-] indices - initialisation un entier en pointeur sans cast [activé par défaut]

quelqu'un pourrait-il indiquer ce que je J'ai manqué ici? Travaillant sur le code pour AVR GCC, mon IDE est Eclipse.

const char wf0[] PROGMEM= "OFF "; 
    const char wf1[] PROGMEM= "SIN "; 
    const char wf2[] PROGMEM= "TRI "; 
    const char wf3[] PROGMEM= "S+T "; 
    const char wf4[] PROGMEM= "PUL "; 
    const char wf5[] PROGMEM= "P+S "; 
    const char wf6[] PROGMEM= "P+T "; 
    const char wf7[] PROGMEM= "P+ST"; 
    const char wf8[] PROGMEM= "NOI "; 
    const char wf9[] PROGMEM= "N+S "; 
    const char wf10[] PROGMEM= "N+T "; 
    const char wf11[] PROGMEM= "NST "; 
    const char wf12[] PROGMEM= "N+P "; 
    const char wf13[] PROGMEM= "NPS "; 
    const char wf14[] PROGMEM= "NPT "; 
    const char wf15[] PROGMEM= "NPTS"; 

    const char * const arrayWaveform[] PROGMEM= {wf0,wf1,wf2,wf3,wf4,wf5,wf6,wf7,wf8,wf9,wf10,wf11,wf12,wf13,wf14,wf15}; 

... 

void showWaveform (char ctrlValue) 
{ 
    char hex = (ctrlValue & 0xf0)>>4; 

    char tempText[4]; 
    char* data = pgm_read_byte(&arrayWaveform[hex]); // <<shows up both errors here 
    strcpy_P (tempText, data); 
    for (char x=0;x<4;x++) 
     { 
     char2LCD(tempText[x]); // <<shows up error: array subscript has type 'char' 
     } 
} 

Answer 1

Cela devrait faire:

void showWaveform (char ctrlValue) 
{ 
    char hex = (ctrlValue & 0xf0)>>4; 
    char *pstr, tempText[5]; 
    memcpy_P(&pstr, arrayWaveform+hex, sizeof(char*)); 
    strncpy_P(tempText, pstr, sizeof(tempText)); 
    for (int x=0; x<4 && tempText[x] != 0; x++) 
    { 
    char2LCD(tempText[x]); 
    } 
} 

Il y avait plusieurs problèmes avec votre code. Tout d'abord arrayWaveform lui-même est stocké dans PROGMEM, vous ne pouvez donc pas simplement y accéder en utilisant des crochets. En second lieu, tempText doit être long d'au moins 5 éléments (sinon strcpy_P peut écrire après la fin du tableau).

Vous voudrez peut-être lire attentivement le docs à propos de PROGMEM, pour bien comprendre ce qui se passe. S'il vous plaît noter que si vous avez une copie récente de AVR-GCC, il y a aussi le new __flash syntax qui permet de se débarrasser de toutes les machines pgm_read_ * et * _P. Je ne l'ai jamais testé, donc YMMV.

Answer 2

Exactement comme le dit votre compilateur, vous utilisez char s pour accès au tableau. Normalement, les indices de tableau doivent être int s.

Sur les deux lignes, vous devez convertir vos variables en int s, ou simplement les déclarer comme int s pour commencer. En ce qui concerne l'autre erreur, je suppose que pgm_read_byte renvoie un char brut que vous essayez d'attribuer à un char*, ce qui est impossible. Vous devriez probablement revenir un pointeur de pgm_read_byte

Answer 3

Dans tempText[x] et arrayWaveform[hex] l'indice est de type signed char qui peut prendre des valeurs negavtive trop

changement à unsigned char

Voir GCC Warnings:

Wchar-subscripts Avertir si un indice de tableau a le type char. Ceci est une cause fréquente d'erreur, car les programmeurs oublient souvent que ce type est signé sur certaines machines. Cet avertissement est activé par -Wall.