Discussion :

[Anouschka]

Bonjour,

J'ai écrit cette fonction très simple :

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
 #include <stdio.h>
#include "asso.h"
// fonction longueur : calcule le nombre de cases mémoires effectivement utilisées dans un tableau de taille 30 max.
// => permet de calculer la longueur d'un vecteur. k doit être mis à 1 pour que le calcul soit correct.     
int longueur(double vecteur[30], int k)
{
    if (k == 31 || vecteur[k] == 0)
    {
        k--;
        return k;
    }
    else
    {    
        k++;
        longueur(vecteur, k);
    }
}

Je l'appelle ensuite dans le fichier suivant :

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
#include <stdio.h>
#include "asso.h"
 
int main()
{
	int tab[3]; // tableau de test
	int n,k;
 
	// remplissage de tab	
	for (k=1;k<31;k++)
	{
		if (k<4)
		{
			tab[k] = 1;
		}		
		else
		{
			tab[k] = 0;
		}
	}
	// calcul de sa longueur
//	n = longueur(tab,1);
	printf("le tableau a une longueur de %d \n", n);
	return 0;
}

Je compile ensuite le tout avec gcc 4.1.1 :

Code :

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

1
2
3
gcc -c fonction.c
gcc -c main.c
gcc fonction.o main.o -o main.out

La compilation se passe bien, mais à l'exécution (ie quand je tape ./main.out), mon programme m'affiche une magnifique ligne blanche... et ne s'arrête que si je fais [ctrl]+c.

Quelqu'un pourrait-il me dire où est mon erreur svp?

Merci d'avance pour vos réponses

Agnès

[Skyounet]

Tu as un tableau de 3 entiers et tu essai d'en mettre 31 dedans?

Faute de frappe?

Tu as un tableau de int, alors pourquoi dans la fonction tu prends un tableau de double en argument?

Compile avec ca et corrige toutes les erreurs

Code :

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

1
2
3
4
 
gcc -Wall -ansi -pedantic -W -c fonction.c
gcc -Wall -ansi -pedantic -W -c main.c
gcc fonction.o main.o -o main.out

[Médinoc]

Attention, vecteur[30] n'est pas supposé valide, et pourtant risque d'être consulté...

[Anouschka]

Faute de frappe?

Oui, ou plus exactement correction incomplète de ma part
Enfin, après avoir corrigé mes erreurs (et m'être rappelée que les tableaux commençaient à 0 et pas à 1), ça marche.

Merci à vous

[Emmanuel Delahaye]

J'ai écrit cette fonction très simple :
Quelqu'un pourrait-il me dire où est mon erreur svp?

Je recommande ceci :

#ifndef H_ASSO
#define H_ASSO

#include <stddef.h>

size_t longueur(double *vecteur, size_t n, size_t k);

#endif /* guard */

http://emmanuel-delahaye.developpez....tes.htm#size_t

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
 
#include <stdio.h>
#include "asso.h"
/* fonction longueur : calcule le nombre de cases mémoires effectivement 
   utilisées dans un tableau de taille 30 max.
   => permet de calculer la longueur d'un vecteur. k doit être mis à 1 pour 
   que le calcul soit correct.
 */
size_t longueur(double *vecteur, size_t n, size_t k)
{
   if (k != n && vecteur[k] != 0)
   {
      k = longueur(vecteur, n, k + 1);
   }
   return k;
}

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
 
#include <stdio.h>
#include "asso.h"
 
int main()
{
   double tab[30]; // tableau de test
   size_t n, k;
 
   // remplissage de tab
   for (k = 0;k < 30;k++)
   {
      if (k < 4)
      {
         tab[k] = 1;
      }
      else
      {
         tab[k] = 0;
      }
   }
   // calcul de sa longueur
   n = longueur(tab, sizeof tab / sizeof * tab, 0);
   printf("le tableau est rempli a %u/%u \n"
          , (unsigned) n
          , (unsigned)(sizeof tab / sizeof *tab));
 
   return 0;
}

[Anouschka]

Je vous remercie de vos conseils qui m'ont aidé à optimiser mon code (ce que je cherche à faire !)