IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Windows Discussion :

Problème avec la fonction 'send' Winsock


Sujet :

Windows

  1. #1
    Membre extrêmement actif
    Profil pro
    Inscrit en
    Mai 2011
    Messages
    923
    Détails du profil
    Informations personnelles :
    Localisation : France, Vienne (Poitou Charente)

    Informations forums :
    Inscription : Mai 2011
    Messages : 923
    Par défaut Problème avec la fonction 'send' Winsock
    Bonjour,
    J'ai pioché un code sur internet et pour pouvoir communiquer avec le serveur smtp.
    CA marche mais pas en localhost.

    Code C++ : 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
    void CSmtp::SendData()
    {
    	int idx = 0,res,nLeft = strlen(SendBuf);
    	fd_set fdwrite;
    	timeval time;
     
    	time.tv_sec = TIME_IN_SEC;
    	time.tv_usec = 0;
     
    	assert(SendBuf);
     
    	if(SendBuf == NULL)
    		throw ECSmtp(ECSmtp::SENDBUF_IS_EMPTY);
     
    	while(1)
    	{
    		FD_ZERO(&fdwrite);
     
    		FD_SET(hSocket,&fdwrite);
     
    		if((res = select(hSocket+1,NULL,&fdwrite,NULL,&time)) == SOCKET_ERROR)
    		{
    			FD_CLR(hSocket,&fdwrite);
    			throw ECSmtp(ECSmtp::WSA_SELECT);
    		}
     
    		if(!res)
    		{
    			//timeout
    			FD_CLR(hSocket,&fdwrite);
    			throw ECSmtp(ECSmtp::SERVER_NOT_RESPONDING);
    		}
     
    		if(res && FD_ISSET(hSocket,&fdwrite))
    		{
    			if(nLeft > 0)
    			{
    				if((res = send(hSocket,&SendBuf[idx],nLeft,0)) == SOCKET_ERROR)
    				{
    					FD_CLR(hSocket,&fdwrite);
    					throw ECSmtp(ECSmtp::WSA_SEND);
    				}
    				if(!res)
    					break;
    				nLeft -= res;
    				idx += res;
    			}
    			else
    				break;
    		}
    	}
     
    	FD_CLR(hSocket,&fdwrite);
    	Sleep(250);
    }

    J'ai du ajouter la fonction Sleep(250); et ca marche.

    Question :
    Qu'est ce qui ne va pas dans le code 'SendData' pour ne pas que cela marche en localhost ?

    Merci

  2. #2
    Expert éminent
    Avatar de Médinoc
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Septembre 2005
    Messages
    27 391
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 41
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Septembre 2005
    Messages : 27 391
    Par défaut
    Là franchement je ne vois pas, surtout que ton Sleep(250) est après la boucle. Peut-être fermes-tu le socket avant que la transmission soit terminée? (si ton send() est non-bloquant). [ Edit: Voir aussi cette réponse sur SO et ses commentaires ]
    De mémoire, select() retourne dès qu'il y a de la place dans le buffer interne d'envoi, et non quand le buffer interne d'envoi est vide.

    PS: Le placement de ton if(SendBuf == NULL) throw ECSmtp(ECSmtp::SENDBUF_IS_EMPTY); est incorrect: il devrait être avant l'appel à strlen().
    SVP, pas de questions techniques par MP. Surtout si je ne vous ai jamais parlé avant.

    "Aw, come on, who would be so stupid as to insert a cast to make an error go away without actually fixing the error?"
    Apparently everyone.
    -- Raymond Chen.
    Traduction obligatoire: "Oh, voyons, qui serait assez stupide pour mettre un cast pour faire disparaitre un message d'erreur sans vraiment corriger l'erreur?" - Apparemment, tout le monde. -- Raymond Chen.

  3. #3
    Membre extrêmement actif
    Profil pro
    Inscrit en
    Mai 2011
    Messages
    923
    Détails du profil
    Informations personnelles :
    Localisation : France, Vienne (Poitou Charente)

    Informations forums :
    Inscription : Mai 2011
    Messages : 923
    Par défaut
    Citation Envoyé par Médinoc Voir le message
    Là franchement je ne vois pas, surtout que ton Sleep(250) est après la boucle. Peut-être fermes-tu le socket avant que la transmission soit terminée? (si ton send() est non-bloquant). [ Edit: Voir aussi cette réponse sur SO et ses commentaires ]
    De mémoire, select() retourne dès qu'il y a de la place dans le buffer interne d'envoi, et non quand le buffer interne d'envoi est vide.

    PS: Le placement de ton if(SendBuf == NULL) throw ECSmtp(ECSmtp::SENDBUF_IS_EMPTY); est incorrect: il devrait être avant l'appel à strlen().
    Je suis d'accord pour le code incorrect,
    j'avais chopper ce code source sur Internet en toute confiance, je fais généralement confiance au code source trouvé sur Internet

Discussions similaires

  1. Problème avec une fonction date.
    Par kmayoyota dans le forum ASP
    Réponses: 8
    Dernier message: 09/09/2004, 12h33
  2. Problème avec la fonction findfirst ()
    Par Angelico dans le forum Windows
    Réponses: 3
    Dernier message: 05/08/2004, 20h40
  3. [Requete SQL en VBA] Problème avec la fonction FLOOR
    Par zubral dans le forum Langage SQL
    Réponses: 4
    Dernier message: 13/07/2004, 13h24
  4. Problème avec les fonctions
    Par jvachez dans le forum PostgreSQL
    Réponses: 1
    Dernier message: 13/01/2004, 12h06
  5. [Postgresql]Problème avec les fonctions ...
    Par fet dans le forum Requêtes
    Réponses: 4
    Dernier message: 02/10/2003, 09h04

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo