Dernier élément d'un tableau

Version imprimable

Bonjour,

Programme pour inverser les éléments d'un tableau à l'aide de pointeurs :
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 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 ?

Dans les boucles, tu pourrais aussi mettre :

Code:

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:

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

21/09/2009, 10h58
chicabonux

Compris pour le strictement inférieur :ccool:

Par contre pourquoi ton code est-il meilleur que le mien ?
21/09/2009, 11h03
nicolas.sitbon

Citation:

Envoyé par chicabonux

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é
21/09/2009, 15h05
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...
21/09/2009, 15h10
nicolas.sitbon

Citation:

Envoyé par Médinoc

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

done.

Citation:

Envoyé par Médinoc

(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.