Discussion :

[let_me_in]

salut tout le monde voici le programme (il veut me rendre fou) :

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
 
        char * chaine,* buff;
	int socketDescriptor, length, len=0;
	ofstream f;
...
        strcpy(chaine,"GET http://sourceforge.net/index.php HTTP/1.1\r\n"
		      "Host: sourceforge.net\r\n"
		      "User-Agent: Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.8.1) Gecko/20061023 SUSE/2.0-30 Firefox/2.0 MEGAUPLOAD 1.0\r\n"
		      "Accept:text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,;q=0.5\r\n"
		      "Accept-Language: fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3\r\n"
		      "Accept-Encoding: gzip,deflate\r\n"
		      "Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7\r\n"
		      "Keep-Alive: 300\r\n"
		      "Proxy-Connection: close\r\n\r\n");
...
        len=strlen(chaine);
        write(socketDescriptor,
		     chaine,
		     len);
 
	for (;length=recv(socketDescriptor,buff,50000, 0)!=0;) {
	    cout<<"writing"<<endl;
 
	    f<<buff;
 
	    sleep(1);
 
	    *buff=0;
 
	}
...

si c'est comme ça j'obtient bien la page demandée, mais si j'enleve le sleep(1) j'obtient une page qui n'a aucun sens, des ligne qui se repetent a des endroit different ..

quelqu'un voit ou est le probleme ?

merci d'avance.

PS: je suis sous linux avec gcc et j'envoi cette requete a SQUID.

[Laurent Gomila]

Déjà là :

Code :

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

1
2
char * chaine;
strcpy(chaine, ...);

Ton pointeur est initialisé n'importe comment et tu n'alloues aucun espace mémoire pour y copier ta chaîne.
Si c'est une chaîne constante fais simplement comme ceci :

Code :

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

const char* chaine = ...;

Sinon, utilise std::string (voir FAQ).

[let_me_in]

merci pour la reponse, mais j'ai alloué toutes les chaines correctement (les ... dans le code), je ne voulais pas encombrer le code, mais comme je l'ai dis en laissant le sleep(1) ça marche a merveille (j'espere que ça ne va pas faire un tour dans le betisier).

autres indication, en laissant le sleep(1) dans la boucle, ça m'affiche deux "wrinting" et il termine, mais en l'enlevant il m'affiche une disaine avec l'erreur citées plus haut.

une autre idée ?

merci encore

[craryb]

comment se décompose la lecture de la socket?

[let_me_in]

bon au cas ou le probleme viendrai du reste je poste tout 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
 
 
using namespace std;
 
int main(int argc, char *argv[])
{
	char * chaine,* buff;
	int socketDescriptor, length, len=0;
	struct sockaddr_in sockaddrStruct;
	ofstream f;
 
	sockaddrStruct.sin_addr.s_addr = inet_addr("127.0.0.1");
	sockaddrStruct.sin_family = AF_INET;
        sockaddrStruct.sin_port = htons(3128);//port);
	sockaddrStruct.sin_zero[8]=0;
 
	chaine = (char * )malloc(600);
 
	if (!chaine){
		cout<<"erreur d'allocation de \'chaine\'"<<endl;
		return -1;
	}
 
	buff=(char *)malloc(50000);
 
	if (!buff){
		cout<<"erreur d'allocation de \'buff\'"<<endl;
		return -1;
	}
	*buff=0;
 
	strcpy(chaine,"GET http://sourceforge.net/index.php HTTP/1.1\r\n"
		      "Host: sourceforge.net\r\n"
		      "User-Agent: Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.8.1) Gecko/20061023 SUSE/2.0-30 Firefox/2.0 MEGAUPLOAD 1.0\r\n"
		      "Accept:text/xml,application/xml,application/xhtml+xml,text/html;q=0.9,text/plain;q=0.8,image/png,;q=0.5\r\n"
		      "Accept-Language: fr,fr-fr;q=0.8,en-us;q=0.5,en;q=0.3\r\n"
		      "Accept-Encoding: gzip,deflate\r\n"
		      "Accept-Charset: ISO-8859-1,utf-8;q=0.7,*;q=0.7\r\n"
		      "Keep-Alive: 300\r\n"
		      "Proxy-Connection: close\r\n\r\n");
 
	f.open("f.txt");
 
	if(!f){
		cout<<"erreur d'ouverture du fichier"<<endl;
		return -1;
	}
 
 
	if ((socketDescriptor = socket(AF_INET, SOCK_STREAM, 0))<0){
 
		cout<<"erreur de creation de socket"<<endl;
		return -1;
	}
 
	if (connect(socketDescriptor,
		     (struct sockaddr *)&sockaddrStruct,
		     sizeof(sockaddrStruct)
		     )<0){
		cout<<"erreur de connexion de la socket"<<endl;
		return -1;
 
 
	}
	len=strlen(chaine);
	length=write(socketDescriptor,
		     chaine,
		     len);
 
	for (;length=recv(socketDescriptor,buff,50000, 0)!=0;) {
	    cout<<"writing"<<endl;
 
	    f<<buff;
 
	    sleep(1);
 
	    *buff=0;
 
	}
 
	return EXIT_SUCCESS;
}

[let_me_in]

comment se décompose la lecture de la socket?

desolé j'ai pas compris la question .

PS : c'est mon premier projet en C++.

[nicroman]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
	for (;length=recv(socketDescriptor,buff,50000, 0)!=0;) {
	    cout<<"writing"<<endl;
 
	    f<<buff;
 
	    sleep(1);
 
	    *buff=0;
 
	}

D'une part... je me demande toujours quelle est la précédence des opérateurs (d'autant qu'elle peut changer d'un langage à un autre) ... donc les trucs a=b!=0 .... a va contenir "true"/"false" ?
Si c'est pour faire !=0, autant ne rien mettre du tout:

Code :

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

1
2
3
 
while (lenth=recv(...)) {
}

Pour le probleme lui même, apres chaque receive, il n 'y a que 'length' caractères valides dans le buffer.
Et celui ci peut très bien contenir des 0 (receptions d'image par exemple).
Donc le f<<buff (qui se traduit en écriture d'une chaine de caractères) ne marchera pas correctement (a priori il s'arrete au premier 0 rencontré, donc parfois aucun avec lecture non autorisée à la clé).
Et le sleep(1); est complétement inutile.

Si tu ne t'attends qu'à du texte en réponse, tu peux rajouter un:

Code :

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

1
2
 
buff[length]=0;

avant l'écriture, mais c'est un quick-fix pour mieux planter dans d'autres cas.

[let_me_in]

merci beaucoup pour le tuyeau, ça marche a present, et j'ai remplacé le f<< par un write, merci beaucoup.