Discussion :

[PKO06]

Bonjour à tous,

Je souhaite creer un tableau en deux dimensions. Celui-ci stockera les éléments d'une table sql.

J'ai une structure qui ressemble à ça :

Code :

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

1
2
3
4
5
6
 
struct queryResult{
	long rows;
	long cols;
	char*** resultArray;
};

que j'initialise ainsi :

Code :

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

struct queryResult res = {0,0,NULL};

Une fonction "execQuery" qui prend en paramètre un pointeur sur une fonction callback.
Ma fonction callback a comme paramètres un pointeurs sur la structure, le nombre de colonne de la table et un tableau contenant les résultats de la ligne et un autre les noms des colonnes.

Je ne sais pas si je m'y prend bien et si ce que j'essaye de faire est possible. Je vous montre mes premiers essais de cette fonction callback :

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
17
18
19
20
21
22
23
24
25
26
27
 
static int sqlite_callback(void* arg, int argc, char **argv, char **azColName){
	struct queryResult *res = arg;
	int i;
 
 
	char **resultLigne = NULL;
 
	if( ( res->resultArray = (char ***) realloc (res->resultArray, (res->rows +2) * sizeof(char**) ) ) ==NULL ){
		fprintf (stderr, "Memoire insuffisante\n");
		free (res->resultArray);
		res->resultArray = NULL;
		return 1;
	}
	for(i=0; i<argc; i++){
		if( ( resultLigne[i] = (char*) malloc ( strlen(argv[i]) * sizeof(char) ) ) ==NULL){
			fprintf (stderr, "Memoire insuffisante\n");
			free (res->resultArray);
			res->resultArray = NULL;
			return 1;
		}
		strcpy(resultLigne[i],argv[i]);
	}
	res->resultArray[res->rows]=resultLigne;
	res->rows++;
	return 0;
}

Est ce que je m'y prend bien avec mon tableau (enfin qu'est ce qui coince plutôt).

Merci de me mettre sur la piste.

[Médinoc]

Bonjour,
Ce qui me parait bizarre, c'est :

• Tu ajoutes deux lignes dans ton realloc(), mais tu n'en mets qu'une.
• D'ailleurs, tu utilises mal realloc() : On n'utilise pas la même variable pour le premier paramètre et le retour de ralloc(), on passe par un temporaire (problème à cause du retour NULL).
• Tu n'alloues pas le tableau de char* lui-même (le second niveau d'allocation).
• Ta fonction de libération de mémoire laisse à désirer.

Sans compter que ça fait beaucoup d'allocations, mais bon : On ne doit pas chercher à optimiser tant que ça ne marche pas tout court.

[PKO06]

Merci pour ta réponse. C'est vrai que ce n'est pas optimisé du tout mais je pensais faire ca en deux temps et déjà être sur de l'allure de mon tableau.

Les deux lignes dans le realloc seront utilisées plus tard, je devrai mettre deux lignes lors du premier parcours de ma table. Pour les tests je n'ajoute qu'une ligne à présent.

J'ai initialisé le tableau char* ainsi : (certe toujours aussi sale)

Code :

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

1
2
3
4
5
6
7
 
if(  ( resultLigne = (char**)malloc(argc*sizeof(char*)) ) == NULL ){
		fprintf (stderr, "Memoire insuffisante\n");
		free (res->resultArray);
		res->resultArray = NULL;
		return 1;
}

Cette ligne est elle correcte ?

Code :

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

res->resultArray[res->rows]=resultLigne;

Est ce que je pourrais initialiser resultLigne de manière statique et la définir comme une ligne de resultArray comme ci-dessus en faisant ainsi :

Code :

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

char* resultLigne[argc]

Et dernière question, cj'accède à mes infos de cette manière

Code :

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

1
2
3
4
 
for(j=0;j<argc;j++){
	essai = *(res->resultArray[res->rows]+j);
}

Est ce la bonne solution.

Merci

[Médinoc]

Cette ligne est elle correcte ?

Code :

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

res->resultArray[res->rows]=resultLigne;

Vue la façon dont tu t'en sers, je pense que oui.

Est ce que je pourrais initialiser resultLigne de manière statique et la définir comme une ligne de resultArray comme ci-dessus en faisant ainsi :

Code :

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

char* resultLigne[argc]

Non, puisque ce serait une variable locale.

Et dernière question, cj'accède à mes infos de cette manière

Code :

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

1
2
3
4
 
for(j=0;j<argc;j++){
	essai = *(res->resultArray[res->rows]+j);
}

Est ce la bonne solution.

Normalement tu pourrais essayer directement ceci:

Code :

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

1
2
3
for(j=0;j<argc;j++){
	essai = res->resultArray[res->rows][j];
}

[PKO06]

Merci Messire pour tous ces conseils