Discussion :

[YouPoP]

Bonjour,

J'ai un problème de realloc que voici: je désire remplir un tableau de int, qui doit grandir ou rapetisser au fur et à mesure que les données (i_noSensor) sont collectées. Le tout fonctionne sur les 2 premières cases du tableau, mais la réallocation pour les cases suivantes est impossible, et je ne comprend vraiment pas pourquoi.

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
28
29
30
31
32
33
 
int i_noSensors = 0;
int *i_sensorTable = malloc(i_noSensors * sizeof(*i_sensorTable));   // set the sensors table array
 
void getDetectedSensors()
{
    int i, j = 0;
    int * temp;
 
    for(i = 0; i < i_activeSensors; i++)    // scan all active sensors
    {
        if(sa_sensors[i].distance < 255)    // if 255 nothing is detected
        {
            i_noSensors++;
            temp = (int *)realloc((int *)i_sensorTable, i_noSensors * sizeof(i_sensorTable));
//REALLOC2(i_sensorTable,i_sensorTable, i_noSensors);
            //i_sensorTable = realloc(i_sensorTable, i_noSensors * sizeof(*i_sensorTable));
            if ( temp == NULL )
            {
                 fprintf(stderr,"\nReallocation impossible");
                 free(i_sensorTable);
                 exit(EXIT_FAILURE);
            }
            else
            {
                i_sensorTable = temp;
                i_sensorTable[j] = i;
                printf("\ni_sensorTable[%d]: %d", j, i_sensorTable[j]);
                j++;
            }
        }
    }
}

voici les résultats:

Code :

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

1
2
3
4
5
6
 
i_sensorTable[0]: 0
i_sensorTable[1]: 1
Reallocation impossible
Process returned -1073741819 (0xC0000005)   execution time : 3.822 s
Press any key to continue.

Le code n'est certainement pas le meilleur non plus, puisque c'est la première fois que j'utilise ces fonctions.

Merci à l'avance pour votre aide

[ram-0000]

Unbe petite erreur ici

Code :

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

temp = (int *)realloc((int *)i_sensorTable, i_noSensors * sizeof(*i_sensorTable));

Mais je ne pense pas que cela soit ton problème car avec un peu de chance, sizeof(pointer) = sizeof(int). Mais en tout cas, c'est pas beau et c'est la porte ouverte aux bugs

Un autre problème ici dans l'initialisation,

Code :

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

int *i_sensorTable = malloc(i_noSensors * sizeof(*i_sensorTable));

Au début, noSensors vaut 0 et je ne sais pas/plus ce que fait un malloc(0)

[YouPoP]

Unbe petite erreur ici

Code :

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

temp = (int *)realloc((int *)i_sensorTable, i_noSensors * sizeof(*i_sensorTable));

Mais je ne pense pas que cela soit ton problème car avec un peu de chance, sizeof(pointer) = sizeof(int). Mais en tout cas, c'est pas beau et c'est la porte ouverte aux bugs

Un autre problème ici dans l'initialisation,

Code :

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

int *i_sensorTable = malloc(i_noSensors * sizeof(*i_sensorTable));

Au début, noSensors vaut 0 et je ne sais pas/plus ce que fait un malloc(0)

Le sizeof est corrigé et l'initialisation est maintenant à 1, merci pour les remarques. Mais ca ne règle toujours pas le problème de fond malheureusement...

[Médinoc]

Essaie en fragmentant un peu plus ton code, notamment pour la réallocation.

[Invité(e)]

Bonjour,

Le code donné a l'air de fonctionner.
La variable globale i_noSensors est elle modifiée ailleurs dans le code ?

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
28
#include <stdio.h>
#include <stdlib.h>
 
int i_noSensors = 0;
int *i_sensorTable = NULL;
 
int main(void)
{
    int i, j = 0;
    int * temp;
 
    for(i = 0; i < 255; i++)   
    {
        i_noSensors++;
        temp = realloc(i_sensorTable, i_noSensors * sizeof *i_sensorTable);
        if ( temp == NULL ) {
            fprintf(stderr,"\nReallocation impossible");
            free(i_sensorTable);
            exit(EXIT_FAILURE);
        } else {
            i_sensorTable = temp;
            i_sensorTable[j] = i;
            printf("\ni_sensorTable[%d]: %d", j, i_sensorTable[j]);
            j++;
        }
    }
    return 0;
}

[ram-0000]

Le code donné a l'air de fonctionner.

Je suis assez d'accord

Peut être rajouter des traces avant le realloc pour afficher les valeurs de i_sensorTable et de i_noSensors.

Mais la variables globales, c'est laid ...

[YouPoP]

pour les passes 0 et 1, la taille de i_sensorTable et de temp est de 4, elle ne grandit jamais comme elle devrait(4, 8, 12...) lors du realloc...

Je merde quelque part et je ne sais pas ou malheureusement...

[diogene]

la taille de i_sensorTable et de temp est de 4

et ne changera pas puisqu'elle correspond à la taille d'un pointeur (4 dans ton cas). Seule la taille du tableau pointé doit changer.

[Invité(e)]

pour les passes 0 et 1, la taille de i_sensorTable et de temp est de 4, elle ne grandit jamais comme elle devrait(4, 8, 12...) lors du realloc...

Je merde quelque part et je ne sais pas ou malheureusement...

À chaque passe, que vaut i_noSensors ?