Problem with simple TCP chat client-server

[ziele]

After 6 hours i figured what i made wrong.
In server.cpp

tcpserver->newConnection();

tcpserver->newConnection();

To copy to clipboard, switch view to plain text mode 

That line is wrong.


It's should be that

tcpserver->setMaxPendingConnections(1);
//and add this line
tcpclient = new QTcpSocket (this);

tcpserver->setMaxPendingConnections(1);
//and add this line
tcpclient = new QTcpSocket (this);

To copy to clipboard, switch view to plain text mode 

@Edit
But now, a message doesn't send to anyone( client->server | server -> client)

mhm...

@Edit 2
Now that i mentioned, if i delete this line

tcpserver->setMaxPendingConnections(1);

tcpserver->setMaxPendingConnections(1);

To copy to clipboard, switch view to plain text mode 

message is sending.

But, only with 1 pc :/

If i tried on 2 pc connected to one router, server side get me this:

QIODevice::write (QTcpSocket): device not open

QIODevice::write (QTcpSocket): device not open

To copy to clipboard, switch view to plain text mode 

in Application Output

Any ideas?

[anda_skoa]

tcpserver->newConnection();

tcpserver->newConnection();

To copy to clipboard, switch view to plain text mode 

That line is wrong.

Yes, as I said

It's should be that

//and add this line
tcpclient = new QTcpSocket (this);

//and add this line
tcpclient = new QTcpSocket (this);

To copy to clipboard, switch view to plain text mode 

No, because you don't want to connect to clients but accept client connections

But, only with 1 pc :/

Yes, because your handler for newConnection() overwrites a single QTcpSocket variable.

Any ideas?

Yes, store the accepted connections in a list or map, not in a single pointer variable.

Cheers,
_

[ziele]

I figured a solution.
I made a server to accept connect to any client not only localhost, and in client instead of localhost i connect to ip 192.168....
And that work on 2 different PC on same LAN. THANKS FOR HELP