Discussion :

[lcoulon]

Bonjour,

J'ai des tableaux char d'une longueur assez importante :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
char test1[200];
char test2[200];
char test3[200];

Je cherche à créer une fonction qui me permettrait de lire le contenu d'un tableau ( en l'appelant dans la fonction par son nom ) puis de copier son contenu dans une mémoire eeprom.

Les fonctions d'écriture dans l'eeprom existent déja et celles ci fonctionnent bien : ( déja testées )

XE2BeginWrite(EEPROM_CONTROL, address_start);
Permet de préparer l'eeprom à écrire des bits à partir d'une adresse donnée. ( à spécifier par l'utilisateur )

XE2Write;
Ecrit un bit dans l'eeprom

XE2EndWrite();
Termine l'ecriture dans l'eeprom


Je voudrai pouvoir utiliser ma fonction comme ceci :

Eeprom2_store_table(test1, 0x000000);
ou encore :
Eeprom2_store_table(test2, 0x001000);

La fonction Eeprom2_store_table lirait tout le contenu du tableau dont le nom est indiqué, puis et copierait chacun de ses bits dans l'eeprom à partir de l'adresse spécifiée.


J'ai donc créé la fonction suivante, mais mon compilateur m'indique des erreurs que je ne sais pas résoudre :

Error [1505] redeclaration of 'table'
Comment puis faire pour indiquer à la fonction qu'elle doit attendre ici un nom de tableau ?

Warning [2054] suspicious pointer conversion
Cette ligne pose pb : je voudrai lire chaque bit du tableau appelé par la fonction :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for(i = 0; i < strlen(table); ++i)

Error [1129] [] operator requires a pointer and an integer as operands
Je ne sais pas comment spécifier correctement le pointer

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

E2Write(table[*p++]);

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
static void Eeprom2_store_table(char table, LWORD address_start)
 
{
    char table;
	BYTE i, *p;
	p = (BYTE*)&table;
 
 
	XE2BeginWrite(EEPROM_CONTROL, address_start);
 
    for(i = 0; i < strlen(table); ++i)
		{
    	XE2Write(table[*p++]);
    	}
    	XE2EndWrite();
}

Pourriez vous m'aider en me corrigeant svp ?
Je vous remercie beaucoup,

[Franck.H]

Le prototype est à l'origine des tes problèmes d'ailleurs cela m'étonne que tu ne l'as pas remarqué, tu as fait ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

static void Eeprom2_store_table(char table, LWORD address_start)

Dans ce prototype tu ne déclares pas un tableau de char mais un char tout court, ceci serait plus juste :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

static void Eeprom2_store_table(char * table, LWORD address_start)

Et si ta fonction ne modifie pas le tableau passé en argument :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

static void Eeprom2_store_table(const char * table, LWORD address_start)

De plus, dans la fonction il faut supprimer la déclaration de table car ceci est la source de ton premier problème :

Error [1505] redeclaration of 'table'

Tu utilises le même identifiant que celui qui se trouve en argument, il y a donc tentative de redéfinition de la variable, catégoriquement refusé par le compilateur !

[lcoulon]

Merci beaucoup,


Si à présent je cherche à stocker un tableau int , pouvez vous me dire ce que je dois modifier dans ma fonction ?

En procédant ainsi j'obtiens un Warning lors de la compilation :

Warning [2054] suspicious pointer conversion

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for(i = 0; i < strlen(table); ++i)

Fonction modifié pour un tableau int :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
static void int_table2eeprom (int * table, LWORD address_start)
 
{
    BYTE i, *p;
	p = (BYTE*)&table;
 
 
	XE2BeginWrite(EEPROM_CONTROL, address_start);
 
    for(i = 0; i < strlen(table); ++i)
		{
    	XE2Write(table[*p++]);
    	}
    	XE2EndWrite();
}

[diogene]

strlen ne marche que pour des tableaux de char terminé par 0 (chaines de caractères), pas pour des tableaux de int.

[lcoulon]

si j'ecris cela, est-ce correct ?

( utilisation de sizeof )

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
static void intTable2Eeprom (const int * table, LWORD address_start)
 
{
    BYTE i, *p;
	p = (BYTE*)&table;
 
 
	XE2BeginWrite(EEPROM_CONTROL, address_start);
 
    for(i = 0; i < sizeof(table); ++i)
		{
    	XE2Write(table[*p++]);
    	}
    	XE2EndWrite();
}

[diogene]

Non.

sizeof(table) donnera le nombre de bytes de table qui est un pointeur (donc sans doute 4) et non pas le nombre de bytes du tableau.

Si tu en a besoin, il faut passer cette information en paramètre à la fonction (à moins que la fin du tableau soit inscrite d'une manière ou d'une autre dans le tableau comme c'est le cas pour les chaines de caractères avec le '\0')