Problème compilation

**Kifz-Fr** · 12/05/2006, 15h57

Bonjour à tous!
Donc voila j'ai un petit soucis lors d'une compilation.
Voila le code:

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
 ********************************************************
        // Structure du buffer
********************************************************
 
typedef struct
 
{
 
  unsigned char *data;
 
  long length;
 
  long offset;
 
 
 
} file_buffer_t;
 
file_buffer_t *readFile (const char *filename)
 
{
 
  file_buffer_t *file;
 
  FILE *fp;

et il me met comme erreur: "a function-definition is not allowed here before '{' token" et en faite il s'agit de la ligne "file_buffer_t *readFile (const char *filename)".
Et quand je rajoute un ";" apres cette ligne j'ai encore plus d'erreur créer.
Si quelqu'un pourrait me donner un petit coup de main

Merci d'avance.
Kifz

**theshark85** · 12/05/2006, 16h03

Y te manque une accolade en bas déjà après le FILE *fp;

J'ai testé et ça compile ainsi.

**KiLVaiDeN** · 12/05/2006, 16h05

Je dis peut-être une bêtise, mais ton * est peut-être trop collé à ton readfile

**Kifz-Fr** · 12/05/2006, 16h21

Merci pour vos réponses, mais j'ai pas l'impression que sa ne corrige pas mon probleme plein de nouvelle erreur se sont rajoutés après l'ajout du "}". Je vais vous passer le code complet c'est un serveur. Il n'est pas finit

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
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
// ********************************************************
// Les includes
// ********************************************************
#include <winsock2.h> // pour les fonctions socket
#include <cstdio> // Pour les Sprintf
#include <stdio.h> // Pour les Printf
 
// ********************************************************
// Les librairies
// ********************************************************
#pragma comment(lib,"ws2_32.lib")
 
// ********************************************************
// Définition des variables
// ********************************************************
 
WSADATA initialisation_win32; // Variable permettant de récupérer la structure d'information sur l'initialisation
int erreur; // Variable permettant de récupérer la valeur de retour des fonctions utilisées
int tempo; // Variable temporaire de type int
int nombre_de_caractere; // Indique le nombre de caractères qui a été reçu ou envoyé
int action; // Indique l'action choisie
char name [255]; // Nom du fichier
char ip [15]; // Indique l'adresse ip 
int port; //Indique le port 
char buffer[65535]; // Tampon contennant les données reçues ou envoyées
SOCKET id_de_la_socket; // Identifiant de la socket
SOCKADDR_IN information_sur_la_destination; // Déclaration de la structure des informations lié au serveur
int main (int argc, char* argv[])
        {
       // ********************************************************
       // Les Actions à effectuer
       // ********************************************************
        printf("\nBonjour, vous etes du cote client\nQu'est ce qu'on fait:\n1) Upload\n2) Download");
        printf ("\nVeuillez rentrer votre nombre ici : ");
        scanf("%d", &action);
        if (action == 1)
        {
        if(argc >= 2){
    strcpy(ip,argv[1]);
        printf("%s\n",ip);
        }
    else {
    printf("Ip : ");
        scanf("%s",ip);
        }       
        if(argc >= 3){
    port = atoi(argv[2]);
        printf("%d\n",port);
        }
    else {
    printf("port : ");
        scanf("%d",&port);
        }       
        if(argc >= 4){
    strcpy(name,argv[3]);
        printf("%s\n",name);
        }
    else {
    printf("nom du fichier : ");
        scanf("%s",name);
        }
        // ********************************************************
        // Initialisation de Winsock
        // ********************************************************
        erreur=WSAStartup(MAKEWORD(2,2),&initialisation_win32);
        if (erreur!=0)
                printf("\nDesole, je ne peux pas initialiser Winsock du a l'erreur : %d %d",erreur,WSAGetLastError());
        else
                printf("\nWSAStartup  : OK");
 
        // ********************************************************
        // Ouverture d'une Socket
        // ********************************************************
        id_de_la_socket=socket(AF_INET,SOCK_STREAM,0);
        if (id_de_la_socket==INVALID_SOCKET)
                printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d",WSAGetLastError());
        else
                printf("\nsocket      : OK");
 
        // ********************************************************
        // Activation de l'option permettant d'activer l'algorithme de Nagle
        // ********************************************************
        tempo=1;
        erreur=setsockopt(id_de_la_socket,IPPROTO_TCP,TCP_NODELAY,(char *)&tempo,sizeof(tempo));
        if (erreur!=0)
                printf("\nDesole, je ne peux pas configurer cette options du à l'erreur : %d %d",erreur,WSAGetLastError());
        else
                printf("\nsetsockopt  : OK");
 
        // ********************************************************
        // Etablissement de l'ouverture de session
        // ********************************************************
        information_sur_la_destination.sin_family=AF_INET;
        information_sur_la_destination.sin_addr.s_addr=inet_addr(ip); // Indiquez l'adresse IP de votre serveur  
        information_sur_la_destination.sin_port=htons(port); // Port écouté du serveur
        erreur=connect(id_de_la_socket,(struct sockaddr*)&information_sur_la_destination,sizeof(information_sur_la_destination));
        if (erreur!=0)
                printf("\nDesole, je n'ai pas pu ouvrir la session TCP : %d %d",erreur,WSAGetLastError());
        else
                printf("\nsetsockopt  : OK");
 
        // ********************************************************
        // Structure du buffer
        // ********************************************************
typedef struct
{
  unsigned char *data;
  long length;
  long offset;
 
} file_buffer_t;
file_buffer_t *
readFile (const char *filename)
{
  file_buffer_t *file;
  FILE *fp;
}
 
        // ********************************************************
        // Ouverture du fichier
        // ********************************************************
  fp = fopen (filename, "rb");
  if (!fp)
    {
      fprintf (stderr, "error: couldn't open \"%s\"!\n", filename);
      return NULL;
    }
 
        // ********************************************************
        // Creation du "file buffer"
        // ********************************************************
  file = (file_buffer_t *)malloc (sizeof (file_buffer_t));
  if (!file)
    {
      fprintf (stderr, "Error: memory allocation failed "
               "for \"%s\"\n", filename);
      fclose (fp);
      return NULL;
    }
 
        // ********************************************************
        // Copie le nom du fichier
        // ********************************************************
  strncpy (file->name, filename, sizeof (file->name));
 
        // ********************************************************
        // Taille du fichier
        // ********************************************************
  fseek (fp, 0, SEEK_END);
  file->length = ftell (fp);
  file->offset = 0;
  fseek (fp, 0, SEEK_SET);
 
        // ********************************************************
        // Allocation de memoire pour les données
        // ********************************************************
  file->data = (unsigned char *)malloc (file->length);
  if (!file->data)
    {
      fprintf (stderr, "Error: memory allocation failed "
               "for \"%s\"\n", filename);
      fclose (fp);
      free (file);
      return NULL;
    }
 
        // ********************************************************
        // Lecture du fichier ouvert auparavent
        // ********************************************************
  fread (file->data, 1, file->length, fp);
  fclose (fp);
 
  return file;
}
        // ********************************************************
        // Envoi des données
        // ********************************************************
        nombre_de_caractere=send(id_de_la_socket,buffer,strlen(buffer),0);
        if (nombre_de_caractere==SOCKET_ERROR)
                printf("\nDesole, je n'ai pas envoyer les donnees du a l'erreur : %d",WSAGetLastError());
        else
                printf("\nsend        : OK");
 
        // ********************************************************
        // Fermeture de la session TCP Correspondant à la commande connect()
        // ********************************************************
        erreur=shutdown(id_de_la_socket,2); // 2 signifie socket d'émission et d'écoute
        if (erreur!=0)
                printf("\nDesole, je ne peux pas fermer la session TCP du a l'erreur : %d %d",erreur,WSAGetLastError());
        else
                printf("\nshutdown    : OK");
 
        // ********************************************************
        // Fermeture de la socket correspondant à la commande socket()
        // ********************************************************
        erreur=closesocket(id_de_la_socket);
        if (erreur!=0)
                printf("\nDesole, je ne peux pas liberer la socket du a l'erreur : %d %d",erreur,WSAGetLastError());
        else
                printf("\nclosesocket : OK");
 
        // ********************************************************
        // Quitte proprement le winsock ouvert avec la commande WSAStartup
        // ********************************************************
        erreur=WSACleanup(); // A appeler autant de fois qu'il a été ouvert.
        if (erreur!=0)
                printf("\nDesole, je ne peux pas liberer winsock du a l'erreur : %d %d",erreur,WSAGetLastError());
        else
                printf("\nWSACleanup  : OK");
        }
 
        else if (action == 2)
        return 0;
        else
        printf ("Rentrez un nombre valide svp");               
        return 0;
        }

Kifz

**theshark85** · 12/05/2006, 16h35

Une question bête pourquoi, il y a

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 typedef struct
{
  unsigned char *data;
  long length;
  long offset;
 
} file_buffer_t;
file_buffer_t *
readFile (const char *filename)
{
  file_buffer_t *file;
  FILE *fp;
}

en plein millieu du code.

Donc un conseil réecrit tout ça un peu plus proprement et ça devrait déjà compiler un peu mieux.

Et aussi décaler la mise en page du prog car c'est pas top pour lire.

Donc déjà, les structs, les fonctions, c'est avant le main et non dans le code.
Et les varaibles globales c'est pas trés bien, à éviter. Voir à ne pas utiliser.
Et en plus de tout ceci, lis les erreurs que te donne le compilo et corrige les car il y en a plein que tu peux faire par toi meme comme les variables non déclaré.

Courage!!!

**Emmanuel Delahaye** · 13/05/2006, 13h16

Envoyé par KiLVaiDeN

Je dis peut-être une bêtise,

Oui !

**Emmanuel Delahaye** · 13/05/2006, 13h20

Envoyé par Kifz-Fr

Merci pour vos réponses, mais j'ai pas l'impression que sa ne corrige pas mon probleme plein de nouvelle erreur se sont rajoutés après l'ajout du "}". Je vais vous passer le code complet c'est un serveur.

HALTE ! Tu ne peux pas définir de fonctions à l'intérieur d'une fonction. Relis ton cours de C sur la structure d'un programme C et la façon de définir les fonctions.

Si tu ne connais pas le C, coder un serveur n'est pas à ta portée.

**KiLVaiDeN** · 13/05/2006, 14h32

Envoyé par Emmanuel Delahaye

Oui !

j'ai mal !

Es-tu sûr ? Car je n'ai pas l'impression qu'il veut faire un pointeur de fonction, mais plutot une fonction qui retourne un pointeur

**Emmanuel Delahaye** · 13/05/2006, 14h53

Envoyé par KiLVaiDeN

Es-tu sûr ? Car je n'ai pas l'impression qu'il veut faire un pointeur de fonction, mais plutot une fonction qui retourne un pointeur

Les espaces ne sont pas significatifs.

**Kifz-Fr** · 05/07/2006, 11h41

Désolé, de faire remonter ce post mais avec le BAC en ce moment j'avais pas trop eu le temps de m'en occuper. Et donc j'ai revu un peu le code et maintenant j'arrive à compiler. Le seul probleme c'est que je peux pas vérifier si il est fonctionnel vue que je n'ai pas codé le coté serveur

. Donc est ce qu'il vous semble juste?
Voici le code :

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
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
// ********************************************************
// Les includes
// ********************************************************
#include <winsock2.h> // pour les fonctions socket
#include <cstdio> // Pour les Sprintf
#include <stdio.h> // Pour les Printf
 
// ********************************************************
// Les librairies
// ********************************************************
#pragma comment(lib,"ws2_32.lib")
 
// ********************************************************
// Définition des variables
// ********************************************************
 
WSADATA initialisation_win32; // Variable permettant de récupérer la structure d'information sur l'initialisation
int erreur; // Variable permettant de récupérer la valeur de retour des fonctions utilisées
int tempo; // Variable temporaire de type int
int nombre_de_caractere; // Indique le nombre de caractères qui a été reçu ou envoyé
int action; // Indique l'action choisie
char filename [255];// Nom du fichier
char ip [15]; // Indique l'adresse ip 
int port; //Indique le port 
SOCKET id_de_la_socket; // Identifiant de la socket
SOCKADDR_IN information_sur_la_destination; // Déclaration de la structure des informations lié au serveur
 
int main (int argc, char* argv[])
        {
// ********************************************************
// Les Actions à effectuer
// ********************************************************
printf("\nBonjour, vous etes du cote client\nQu'est ce qu'on fait:\n1) Upload\n2) Download");
printf ("\nVeuillez rentrer votre nombre ici : ");
scanf("%d", &action);
   if (action == 1)
{
if(argc >= 2){
    strcpy(ip,argv[1]);
      printf("%s\n",ip);
}
else {
    printf("Ip : ");
      scanf("%s",ip);
}       
if(argc >= 3){
     port = atoi(argv[2]);
    printf("%d\n",port);
}
else {
    printf("port : ");
    scanf("%d",&port);
}       
if(argc >= 4){
     strcpy(filename,argv[3]);
    printf("%s\n",filename);
}
else {
      printf("nom du fichier : ");
    scanf("%s",filename);
        }
// ********************************************************
// Initialisation de Winsock
// ********************************************************
erreur=WSAStartup(MAKEWORD(2,2),&initialisation_win32);
if (erreur!=0)
                printf("\nDesole, je ne peux pas initialiser Winsock du a l'erreur : %d %d",erreur,WSAGetLastError());
else
                printf("\nWSAStartup  : OK");
 
// ********************************************************
// Ouverture d'une Socket
// ********************************************************
id_de_la_socket=socket(AF_INET,SOCK_STREAM,0);
if (id_de_la_socket==INVALID_SOCKET)
        printf("\nDesole, je ne peux pas creer la socket du a l'erreur : %d",WSAGetLastError());
else
        printf("\nsocket      : OK");
 
// ********************************************************
// Activation de l'option permettant d'activer l'algorithme de Nagle
// ********************************************************
tempo=1;
erreur=setsockopt(id_de_la_socket,IPPROTO_TCP,TCP_NODELAY,(char *)&tempo,sizeof(tempo));
if (erreur!=0)
     printf("\nDesole, je ne peux pas configurer cette options du à l'erreur : %d %d",erreur,WSAGetLastError());
 else
      printf("\nsetsockopt  : OK");
 
// ********************************************************
// Etablissement de l'ouverture de session
// ********************************************************
information_sur_la_destination.sin_family=AF_INET;
information_sur_la_destination.sin_addr.s_addr=inet_addr(ip); // Indiquez l'adresse IP de votre serveur  
information_sur_la_destination.sin_port=htons(port); // Port écouté du serveur
erreur=connect(id_de_la_socket,(struct sockaddr*)&information_sur_la_destination,sizeof(information_sur_la_destination));
if (erreur!=0)
        printf("\nDesole, je n'ai pas pu ouvrir la session TCP : %d %d",erreur,WSAGetLastError());
else
    printf("\nsetsockopt  : OK");
 
// ********************************************************
// Ouverture du fichier
// ********************************************************
FILE * pFile; 
pFile = fopen ( "filename" , "rb" );
if (pFile==NULL)
exit (1);
// ********************************************************
// Taille du fichier
// ********************************************************
long lSize; 
fseek (pFile , 0 , SEEK_END);
lSize = ftell (pFile);
rewind (pFile);
// ********************************************************
// Allocation de la mémoire
// ********************************************************
char * buffer;
buffer = (char*) malloc (lSize);
if (buffer == NULL)
exit (2);
// ********************************************************
// Copie du fichier dans le buffer
// ********************************************************
fread (buffer,1,lSize,pFile);
 
// ********************************************************
// On ferme le fichier et on retourne le buffer tout chaud :p
// ********************************************************
fclose (pFile);
free (buffer);
 
// ********************************************************
// Envoi des données
// ********************************************************
nombre_de_caractere=send(id_de_la_socket,buffer,strlen(buffer),0);
if (nombre_de_caractere==SOCKET_ERROR)
        printf("\nDesole, je n'ai pas envoyer les donnees du a l'erreur : %d",WSAGetLastError());
else
        printf("\nsend        : OK");
 
// ********************************************************
// Fermeture de la session TCP Correspondant à la commande connect()
// ********************************************************
erreur=shutdown(id_de_la_socket,2); // 2 signifie socket d'émission et d'écoute
if (erreur!=0)
        printf("\nDesole, je ne peux pas fermer la session TCP du a l'erreur : %d %d",erreur,WSAGetLastError());
else
        printf("\nshutdown    : OK");
 
// ********************************************************
// Fermeture de la socket correspondant à la commande socket()
// ********************************************************
erreur=closesocket(id_de_la_socket);
if (erreur!=0)
      printf("\nDesole, je ne peux pas liberer la socket du a l'erreur : %d %d",erreur,WSAGetLastError());
else
    printf("\nclosesocket : OK");
 
// ********************************************************
// Quitte proprement le winsock ouvert avec la commande WSAStartup
// ********************************************************
erreur=WSACleanup(); // A appeler autant de fois qu'il a été ouvert.
if (erreur!=0)
        printf("\nDesole, je ne peux pas liberer winsock du a l'erreur : %d %d",erreur,WSAGetLastError());
else
    printf("\nWSACleanup  : OK");
 
return 0;
}
}

Merci d'avance

**Médinoc** · 05/07/2006, 11h46

Code :

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

1
2
#include <cstdio> // Pour les Sprintf
#include <stdio.h> // Pour les Printf

Je ne suis pas allé plus loin : Tu inclues un header ET son équivalent C++, c'est très mauvais.
En regardant de plus près, j'ai vu que tu avais une indentation chaotique, énormément de variables globales, plusieurs (mauvaises, évidemment) utilisations de scanf(), des casts sur le retour de malloc(), et des appels à exit().

**Kifz-Fr** · 05/07/2006, 12h00

D'accord

J'ai plus qu'à reprendre tout mon code.
Merci pour ton aide.

Problème compilation

Réseau C

Discussions similaires

Partager

Partager