Discussion :

[chicabonux]

Bonjour,

Programme pour inverser les éléments d'un tableau à l'aide de pointeurs :

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
34
#include<stdio.h>
 
int main(int argc, char *argv[])
{
	int A[50];
	int *P1,*P2,AIDE,N;
 
	printf("entrer la dimension du tableau a (max.50) : ");
	scanf("%d",&N);
 
	for(P1=A;P1<A+N;P1++)
	{
		printf("element %d : ",P1-A);
		scanf("%d",P1);
	}
 
	for(P1=A;P1<A+N;P1++)
		printf("%d", *P1);
	printf("\n");
 
	for(P1=A,P2=A+(N-1);P1<P2;P1++,P2--)
	{
		AIDE=*P1;
	    *P1=*P2;
	    *P2=AIDE;
	}
 
	printf("tableau resultat : \n");
	for(P1=A;P1<A+N;P1++)
		printf("%d", *P1);
	printf("\n");
 
	return 0;
}

Je ne comprends pas pourquoi on met P2=A+(N-1) dans la boucler de traitement et P1<A+N dans les autres boucles ?
Le dernier élément de A est A+N ou A+(N-1) ? P2 doit bien pointer sur le dernier élément de A ?

[nicolas.sitbon]

Dans les boucles, tu pourrais aussi mettre :

Code :

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

P1 <= A + N - 1

C'est le strictement inférieur qui t'induit en erreur, on atteindra jamais cet élément (qui n'existe pas d'ailleurs).
Autrement, le code est vraiment crade...

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
#include <stdio.h>
 
static void dump(int const * tab, size_t size)
{
   for(int const * next = tab; next < tab + size; next++)
      printf("%d ", *next);
 
   puts("");
}
 
static void swap(int * tab, size_t size)
{
   for(int * next = tab, *last = tab + size - 1; next < last; next++, last--)
   {
      int tmp = *next;
      *next = *last;
      *last = tmp;
   }
}
 
static void fill(int * tab, size_t size)
{
   for(int * next = tab; next < tab + size; next++)
   {
      int ret = 0;
 
      do
      {
         printf("element %td : ", next - tab); /* substracting pointers give a ptrdiff_t */
         fflush(stdout);
         ret = scanf("%d", next);
         scanf("%*[^\n]"), getchar();
      }
      while(ret != 1);
   }
}
 
int main(void)
{
   size_t size = 0;
   int ret = 0;
 
   puts("entrer la taille du tableau (max.50) :");
   ret = scanf("%zu", &size);
   scanf("%*[^\n]"), getchar(); /* flush stdin */
 
   if(ret == 1  &&  size <= 50)
   {
      int tab[size]; /* VLA, checked size */
 
      fill(tab, size);
 
      dump(tab, size);
 
      swap(tab, size);
 
      puts("tableau resultat :");
 
      dump(tab, size);
   }
 
   return 0;
}

[chicabonux]

Compris pour le strictement inférieur

Par contre pourquoi ton code est-il meilleur que le mien ?

[nicolas.sitbon]

Par contre pourquoi ton code est-il meilleur que le mien ?

- code redondant
- paramètres inutilisés
- mauvais typage de variable
- mauvais nommage de variable
- mauvaise indentation
- pas de gestion des erreurs
- mauvais format dans printf
- portée injustifiée des variables
- utilisation injustifié d'une fonction d'entrée sortie formaté pour une sortie non formaté

[Médinoc]

Tu as oublié le const dans la fonction dump().

(et à ta place, j'aurais fais une fonction swap() séparée, mais bon, c'est du détail)

Edit: Je pensais plus au swap() de deux éléments, plutôt que celui de tout le tableau. Mais bon, une fois la seconde fonction extraite, elle est assez petite pourqu'on n'ait pas à la modifier encore plus...

[nicolas.sitbon]

Tu as oublié le const dans la fonction dump().

done.

(et à ta place, j'aurais fais une fonction swap() séparée, mais bon, c'est du détail)

Je viens de l'intégrer, néanmoins, comme tu l'as dit, c'est du détail, en fait il y a plein de choses qui pourraient être modifié/amélioré, tout dépend des objectifs qu'on se fixe.
Cordialement.