gcc : problème d'exécution

Version imprimable

Bonjour,

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

Code:

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:

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:

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

02/06/2006, 14h53
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:

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
02/06/2006, 15h01
Médinoc

Attention, vecteur[30] n'est pas supposé valide, et pourtant risque d'être consulté...
02/06/2006, 15h39
Anouschka

Citation:

Envoyé par Skyrunner

Faute de frappe?

Oui, ou plus exactement correction incomplète de ma part :oops:
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 :)

Citation:

Envoyé par agnesN7

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

Je recommande ceci :

Citation:

#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:

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:

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;
}

07/06/2006, 15h42
Anouschka

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