Probleme with TcpSocket::Receive

[Mourtaza]

Hello everybody, I am coding one simple application and I have a problem with this code :

Code: [Select]

#include<SFML/Graphics.hpp>
#include<SFML/Network.hpp>
#include<iostream>
#include<string>
#include"Naruto.h"
    using namespace sf;
    using namespace std;
int main(void)
{
    RenderWindow app(VideoMode(800,600,32),"Naruto");
    TcpSocket mySocket;
        bool connexion(true);
        while(connexion)
        {
            if(mySocket.Connect("192.168.1.176",5000) == Socket::Done)
                connexion = false;
        }
        Naruto naruto("Image/NarutoNormal.bmp");
            naruto.changerPosition(400,100);

        Naruto sakura("Image/SakuraNormal.bmp");
            sakura.changerPosition(400,400);

        Texture terrTxt;
            terrTxt.LoadFromFile("Image/terrain.bmp");
                Sprite terrain;
                    terrain.SetTexture(terrTxt);

    while(app.IsOpen())
    {
        Event event;
            while(app.PollEvent(event))
            {
                if(event.Type == Event::Closed)
                    app.Close();
                Packet paquet;

                if(Keyboard::IsKeyPressed(Keyboard::Up))
                {
                    naruto.avancer();
                    paquet << "haut";
                    mySocket.Send(paquet);
                    paquet.Clear();

                }
                if(Keyboard::IsKeyPressed(Keyboard::Down))
                {
                    naruto.reculer();
                    paquet << "bas";
                    mySocket.Send(paquet);
                    paquet.Clear();

                }
                if(Keyboard::IsKeyPressed(Keyboard::Left))
                {
                    naruto.allerAGauche();
                    paquet << "gauche";
                    mySocket.Send(paquet);
                    paquet.Clear();
                }
                if(Keyboard::IsKeyPressed(Keyboard::Right))
                {
                    naruto.allerADroite();
                    paquet << "droite";
                    mySocket.Send(paquet);
                    paquet.Clear();
                }
                string myText("");
                if(mySocket.Receive(paquet))
                {
                    paquet >> myText;
                }

            }
        app.Clear();
        app.Draw(terrain);
        app.Draw(naruto.getPersonnage());
        app.Draw(sakura.getPersonnage());
        app.Display();
    }
    return EXIT_SUCCESS;
}

The probleme is at the reception of a packet just before clearing the window etc...
My window is always "transparent" , it did not display what he does.
I think, and I know that there is a problem in the reception of a packet, but what ? Even if i don't write anything in this condition.

[Laurent]

if(mySocket.Receive(paquet))

Check the doc, Receive doesn't return a boolean.

[Mourtaza]

I tried with

Code: [Select]

if(mySocket.Receive(paquet) == Socket::Done)

but it's there's the same problem.

[Laurent]

but it's there's the same problem

Yeah, you're not doing anything with myText so whether this code is correct or not doesn't make any difference.

But do you actually receive something, or is the program blocked on the Receive call? What happens exactly?

[Mourtaza]

I think

Code: [Select]

app.Clear();
.....
app.Display(); is never call. If I don't write this condition :

Code: [Select]

if(mySocket.Receive(paquet) == Socket::Done)
{
......
}

There is no problem to move my character and display the window etc..
But if I don't write it, I can"t move the other character by receiving packet from my server.

[Laurent]

Receive is blocking, it won't return until it receives something. So it means that your program never receives anything from the network, or receives something but at a very low rate.

Calling a blocking Receive function in the main loop is not recommended because it will prevent your program from running at its normal rate. You should either use a thread or non-blocking sockets.

[Mourtaza]

non blocking-socket = UDP Socket ?

I was just trying to do it in a Thread, but my socket is not declared in this thread, so I try to write in the parameters of my thread, as a reference or not. But I have an error which says that the constructor of NonCopyable is private.

[Laurent]

non blocking-socket = UDP Socket ?

Read the doc / tutorials

Code: [Select]

socket.SetBlocking(false);

I was just trying to do it in a Thread, but my socket is not declared in this thread, so I try to write in the parameters of my thread, as a reference or not. But I have an error which says that the constructor of NonCopyable is private.

You must pass it as a pointer to avoid problems.

[Mourtaza]

Thanks, I find it by the way, finally I suceed, my windows are displayed etc.. all it's ok but it don't work good if I press 2 keys at the same time, just one key is "seen" by the server, is it normal ? Should I post the code ?