Discussion :

[mathieudvlmnck]

Bonjour, je voudrais inverser les éléments de mon vecteur. Voici la fonction que j'ai crée pour inverser mon vecteur.

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
#include <stdio.h>
 
void inverse(int elem[], int longueur, int *p);
 
int main()
{
 
	int n;
	int vect[n];
	int inv[n];
	printf("Quelle est la longueur du vecteur? \n");
	scanf("%d", &n);
	for(int i=0; i<n; i++)
	{
	printf("\n Vecteur valeur '%d' : ", i+1);
	scanf("%d", &vect[i]);
	}
	inverse(vect, n, inv);
	for(int j =0;j<n;j++) 
	{ 
		printf(" '%d', ", inv[j]);
	} 
return 0;
}
void inverse(int elem[], int longueur, int *p)
{
	int k = 0;
	for(int i = longueur-1 ; i > 0 ; i--)
	{
		*(p+k) = *(elem+i);
		k++;
	}
}

Le code fonctionne correctement 1 fois sur 3
J'ai 0 warnings cependant, je ne trouve pas mon erreur ?

[edgarjacobs]

Hello,

Sans parler de l'inversion des éléments, les lignes 9 et 10 posent porblème: n n'a pas été initalisé, et donc peut valoir vaut n'importe quoi.

Déplace ta ligne 12 après la 8, ça ira mieux....

[mathieudvlmnck]

Merci edgarjacobs pour cet élément de réponse.
Je me retrouve donc avec mes lignes 11 et 12 avant ma 8.
Mon code s'exécute mais il reste un problème.

la dernière variable de mon vecteur se voit attribué un entier assez ... particulier !

[foetus]

Tu as un buffer overflow dans ta procédure inverse
Ta plage pour tes indices de boucle est mauvaise

Et au passage, tu peux mettre l'incrémentation de k dans ta boucle : for (int i = (longueur - 1); i > 0; i--, k++) {

[Sve@r]

Bonjour, je voudrais inverser les éléments de mon vecteur. Voici la fonction que j'ai crée pour inverser mon vecteur.

Code c :

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

1
2
3
4
5
6
7
8
9
void inverse(int elem[], int longueur, int *p)
{
	int k = 0;
	for(int i = longueur-1 ; i > 0 ; i--)
	{
		*(p+k) = *(elem+i);
		k++;
	}
}

Bonjour

Je ne comprends pas, si tu comme tu dis tu veux inverser les éléments de ton tableau, pourquoi tu ne les inverses alors pas dans ledit tableau mais dans un second et différent tableau ?

Sinon il te faut descendre jusqu'à 0 et non t'arrêter à 1 (car un tableau commence à 0 !!!). De plus foetus t'indique que k peut se mettre dans la boucle mais moi je te dis que k est inutile car il peut se déduire (calculer) depuis i (ou i se calcule depuis k)

Et enfin cette écriture *(p+k) n'est absolument pas plus économique ni plus rapide que l'écriture classique p[k] et au contraire rend le code moins lisible.