Discussion :

[noussa309]

bonsoir ,je veux faire un tri par ordre décroissant a partir d'un fichier qui contient des valeurs(lecture),ces valeurs seront trié dans un autre fichier(en ecriture).
voila mon code je ne sais pas ou est l'erreur,ces valeur s’écrit dans le nouveau fichier mais pas avec l'ordre décroissant.

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
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
int main()
{
 
  char FICH_A[30], FICH_FUS[30];
  FILE *FA,*FFUS;
  void tri(int* TABA,int N);
int changer_pos(int *tabl1, int *tabl2);
 
 
 
 int TABA[100],TFUS[200];
  int LA,N,I ; 
do
    {
 
     FA = fopen("D:\\tester.txt", "r");
     if (!FA) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: ");
    }
  while (!FA);
 
  for (LA=0; !feof(FA); LA++)
       fscanf(FA,"%d\n", &N);
        while (!feof(FA))
     putchar(fgetc(FA));
 
 
 
  fclose(FA);
 
 
  FA = fopen("D:\\tester.txt", "r");
 
  for (I=0; I<N; I++)
       fscanf(FA,"%d\n", TABA+I);
 
  fclose(FA);
 
do
    {
 
 
  FFUS = fopen("D:\\tester3.txt", "w");
 
     if (!FFUS) 
         printf("\aERREUR: Impossible d'ouvrir "
                "le fichier: %s.\n", FICH_FUS);
    }
  while (!FFUS);
 
  for (I=0; I<LA; I++)
       fprintf(FFUS,"%d\n", *(TABA+I));
        for (I=0; I<LA; I++)
       printf("%d\n", *(TABA+I));
 
 
  fclose(FFUS);
  getch();
   return 0;
}
void tri(int* TABA,int N)
{
    int i, _i;
    for(_i=0;_i<N;++_i)
    {
        for(i=0;i<N-1;i++)
        {
            if(TABA[i] > TABA[i+1])
            {
            int  changer_pos(int *tabl1, int *tabl2);
            printf ("%d",&TABA[i]);
            }
        }
    }
}
 
 
 
int changer_pos(int *tabl1, int *tabl2)
{
    int inter = 0;
 
    inter = *tabl1;
    *tabl1 = *tabl2;
    *tabl2 = inter;
}

merci d'avance.

[diogene]

Ce code est vraiment mal fichu

1- indentation et présentation déficientes

2- mauvais choix d'identificateurs (ex : i et _i)

3- déclarations des prototypes dans les fonctions.
Notamment dans tri() :

Code :

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

1
2
3
4
5
            if(TABA[i] > TABA[i+1])
            {
            int  changer_pos(int *tabl1, int *tabl2);
            printf ("%d",&TABA[i]);
            }

La ligne en rouge n'est pas un appel de fonction mais une déclaration de prototype.
Le printf() utilise un %d pour l'affichage d'une adresse

4- changer_pos() déclare retourner un int et ne retourne rien

[bpy1401]

Bonjour,

j'ai passé ce bout de code dans un outil d'analyse de qualité du code (QAC pour ne pas le citer), et j'obtiens ceci.

Déclarations non utilisées:
6: char FICH_A[30]
8: void tri(int* TABA,int N);
9: int changer_pos(int *tabl1, int *tabl2);
13: TFUS[200];
73: int changer_pos(int *tabl1, int *tabl2);

Code :

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

1
2
3
  74:              printf ("%d",&TABA[i]);
                   ^
Msg(7:0179) [U] Argument type does not match conversion specifier number 1

Code :

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

1
2
Msg(7:0744) [U] 'changer_pos()' has been declared with a non void result type but ends with an implicit 'return ;' statement.
MISRA Rule 83

Code :

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

1
2
3
 37:    for (I=0; I<N; I++)
                     ^
Msg(4:3353) The variable 'N' is possibly unset at this point.

Si il n'y rien à lire, N ne sera pas assigné et pourras valoir n'importe quoi.


J'ai vu aussi que le test d'ouverture des fichiers est inclu dans une boucle while. Si cela se passe mal, il y a de fortes chances que cela se passe mal aussi le coup d'après et ainsi de suite eternellement. C'est une boucle sans fin.

Et pour finir, evite les variables du type _I à coté d'un I , N, ... Soit plus explicite sur les nom pour la compréhension par un autre relecteur.

Cordialement