while ( true )
{ 
  // ...
  if ( !Client->isConnected() )
  {
    Client->Connect( "127.0.0.1", 1234 )
  }
  else
  {
    MessageBox( NULL, Client->Recieve().c_str(), "Debug", NULL ); 
    Client->Send( "Test :-)" );
  }
  //...
}

Wie würdet ihr die Klasse aufbauen? Ich hoffe meine Frage ist verständlich

Ich muss das jedenfalls in eine Klasse packen, da ich diese noch an eine Funktion weitergeben muss.

Vielen Dank im Voraus

Gruß

char *Client::Receive()
{
	char buf[512];
	long rc = recv( s, buf, strlen( buf ), 0 );
	buf[rc] = '\0';
	if(rc==0 || rc == SOCKET_ERROR)
	{
		connected = false;
		closesocket( s );
	}
	return buf; // Stack around the variable 'buf' was corrupted.......
}

Wie mache ich es richtig bzw. wie würdet ihr es machen. Danke

Gruß

std::string Client::Receive()
{
		char buf[512];
		long rc = recv( s, buf, strlen( buf ), 0 );
		buf[rc] = '\0';
		if(rc==0 || rc == SOCKET_ERROR)
		{
			connected = false;
			closesocket( s );
		}
		std::string Temp = buf;
		return Temp;
}

Also????

Das soll einfach möglich sein!!! Aber das kriege ich nicht, also scheiterts doch am Klassendesign...

class Client
	{
		private:
			char buf[512];
			SOCKET s;
			SOCKADDR_IN addr;
			long rc;
			bool connected;

		public:
			Client();
			~Client();

		public:
			bool Init( const std::string& IP, int Port );
			std::string Receive();
			void Send( const std::string& Str );
			bool isConnected();

		private:
			int startWinsock();
	};

Client::Client() : connected( false )
{
	rc = startWinsock();
}

Client::~Client()
{
	if ( s )
	{
		closesocket( s );
	}
	WSACleanup();
}

bool Client::Init( const std::string& IP, int Port )
{
	s = socket( AF_INET, SOCK_STREAM, 0 );
	if ( s == INVALID_SOCKET )
	{
		connected = false;
		return false;
	}

	// Verbinden
	memset( &addr, 0, sizeof( SOCKADDR_IN ) );
	addr.sin_family = AF_INET;
	addr.sin_port = htons( Port );
	addr.sin_addr.s_addr = inet_addr( IP.c_str() );

	if ( connect( s, (SOCKADDR*)&addr, sizeof( SOCKADDR ) ) == SOCKET_ERROR  )
	{
		connected = false;
		return false;
	}

	connected = true;

	return true;
}

std::string Client::Receive()
{
		long rc = recv( s, buf, sizeof( buf ), 0 );
		buf[rc] = '\0';
		if(rc==0 || rc == SOCKET_ERROR)
		{
			connected = false;
			closesocket( s );
		}
		return buf;
}

void Client::Send( const std::string& Str )
{
	if ( s )
	{
		rc = send(s,Str.c_str(),Str.length() + 1,0);
	}
}

bool Client::isConnected()
{
	return connected;
}

int Client::startWinsock()
{
	WSADATA wsa;
	return WSAStartup( MAKEWORD( 2, 0 ), &wsa );
}

while ( true )
{
  // ...
  if ( !Client->isConnected() )
  {
    Client->Init( "127.0.0.1", 1234 )
  }
  else
  {
    MessageBox( NULL, Client->Recieve().c_str(), "Debug", NULL );
    Client->Send( "Test :-)" );
  }
  //...
}

Irgendwas mit Heap kaputt... Der Fehler muss jedoch DEFINITIV in diesen Zeilen stecken... Was mache ich falsch? Vielen dank im Voraus...

Ja ich weiß, ist nicht sooo wenig Code, aber irgendwie komme ich nicht weiter und ich würde das gerne so gut wie möglich lösen und dabei könnt ihr mir natürlich am Besten helfen Danke!

Gruß

int Client::recv(void* buf, const int& len)
{
	if (connected == false)
		return -1;

	return (::recv(s, reinterpret_cast<char*>(buf), len, 0));
}

wo ist das Problem?

#include <winsock.h>
#include <string>
#include <iostream>

#pragma comment( lib, "ws2_32.lib" )

class Client
    {
        private:
            char buf[512];
            SOCKET s;
            SOCKADDR_IN addr;
            long rc;
            bool connected;

        public:
            Client();
            ~Client();

        public:
            bool Init( const std::string& IP, int Port );
            std::string Receive();
            void Send( const std::string& Str );
            bool isConnected();

        private:
            int startWinsock();
    };

//------------------------
Client::Client() : connected( false )
{
    rc = startWinsock();
}

Client::~Client()
{
    if ( s )
    {
        closesocket( s );
    }
    WSACleanup();
}

bool Client::Init( const std::string& IP, int Port )
{
    s = socket( AF_INET, SOCK_STREAM, 0 );
    if ( s == INVALID_SOCKET )
    {
        connected = false;
        return false;
    }

    // Verbinden
    memset( &addr, 0, sizeof( SOCKADDR_IN ) );
    addr.sin_family = AF_INET;
    addr.sin_port = htons( Port );
    addr.sin_addr.s_addr = inet_addr( IP.c_str() );

    if ( connect( s, (SOCKADDR*)&addr, sizeof( SOCKADDR ) ) == SOCKET_ERROR  )
    {
        connected = false;
        return false;
    }

    connected = true;

    return true;
}

std::string Client::Receive()
{
        long rc = recv( s, buf, sizeof( buf ), 0 );
        buf[rc] = '\0';
        if(rc==0 || rc == SOCKET_ERROR)
        {
            connected = false;
            closesocket( s );
        }
        return buf;
}

void Client::Send( const std::string& Str )
{
    if ( s )
    {
        rc = send(s,Str.c_str(),Str.length() + 1,0);
    }
}

bool Client::isConnected()
{
    return connected;
}

int Client::startWinsock()
{
    WSADATA wsa;
    return WSAStartup( MAKEWORD( 2, 0 ), &wsa );
}

int main ()
{

	std::cout << "Hmm" << std::endl;

	Client * BlubClient = new Client;

	while ( true )
	{
	  // ...
	  if ( !BlubClient->isConnected() )
	  {
		  BlubClient->Init( "127.0.0.1", 1234 );
	  }
	  else
	  {
		   std::cout << "VERBUNDEN!" << std::endl;
		MessageBox( NULL, BlubClient->Receive().c_str(), "Debug", NULL );
		BlubClient->Send( "Test :-)" );
	  }
	  //...
	} 	

	delete BlubClient;
}

Was nicht funktioniert? Er sendet die Nachricht "Test :-)" nicht und sobald der Server beendet wird gibt er eine Messagebox mit Schrott aus und das jedesmal aufs neue! Das soll so nicht sein. Und überhaupt ist die Klasse für dieses vorhaben in Ordnung?

Als Server könnt ihr diesen nehmen: http://www.c-worker.ch/tuts/socksrv.c

Erst senden, dann empfangen, von alleine sendet der Server ja nichts.

Na sowas aber auch, echt nicht ? Ist wohl kein auto-Server oder wie ?

die beiden

MessageBox( NULL, BlubClient->Receive().c_str(), "Debug", NULL );
        BlubClient->Send( "Test :-)" );

tauschen? hat der threadersteller das schon getestet?

Kompilieren...
main.cpp
.\main.cpp(88) : warning C4267: 'Argument': Konvertierung von 'size_t' nach 'int', Datenverlust möglich
.\main.cpp(121) : error C2664: 'MessageBoxW': Konvertierung des Parameters 2 von 'const char *' in 'LPCWSTR' nicht möglich
        Die Typen, auf die verwiesen wird, sind nicht verknüpft; die Konvertierung erfordert einen reinterpret_cast-Operator oder eine Typumwandlung im C- oder Funktionsformat.

@frager: Du musst in den Projekteinstellungen von Unicode auf Multibyte umstellen

Ist das Klassendesign denn für diese AUfgabe in Ordnung? Oder sollte ich noch was ändern? Danke

Aber ich hab ein kleines Problem und zwar würde ich gerne auch Texte mit Umlaute fehlerfrei übertragen. Wie mache ich das? Bis jetzt kommt jedesmal ein anderes Zeichen an, wenn ich z.B. ein ä, ö, ü, ß... versenden will Was kann ich machen? Das wäre mir noch wichtig. Vielen Dank.