Problem mit Sockets

[krustx]

Hey ho, ich habe gestern angefangen ne kleine Socketklasse zu schreiben, habe mir dazu das c-worker tut genommen.
Es gibt ein kleines Problem mit dem Port...

Ich zeige euch mal mein kleines Projekt:
socket.cpp:

// Class automatically generated by Dev-C++ New Class wizard

#include "socket.h" // class's header file
// class constructor
Socket::Socket(char IpAddress[],u_short Port)
{
	this->Port = Port;
	strcpy(this->IpAddress,IpAddress);
}

bool Socket::Connect(){
    if(StartWinsock() == 0){
       SOCKET s = socket(AF_INET,SOCK_STREAM,0);
       if (s != INVALID_SOCKET){
          SOCKADDR_IN addr;
          memset(&addr,0,sizeof(SOCKADDR_IN));
          addr.sin_family = AF_INET;
          addr.sin_port = htons((u_short)this->Port);
          addr.sin_addr.s_addr = inet_addr(this->IpAddress);
          if (connect(s,(SOCKADDR*)&addr,sizeof(SOCKADDR))!= SOCKET_ERROR){
               return true;
          }else{
                MessageBox(0,"Nicht connectet","",0);
                return false;
             //Fehler
          }
       }else{
             MessageBox(0,"Socket nicht erstellt","",0);
             return false;
       
       //Fehler             
       }
    }else{
          MessageBox(0,"Winsock nicht gestartet","",0);
          return false;
       //Fehler   
    }   
}

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

// class destructor
Socket::~Socket()
{
	// insert your code here
}

socket.h:

// Class automatically generated by Dev-C++ New Class wizard

#ifndef SOCKET_H
#define SOCKET_H

#include <winsock.h>
#include <windows.h>
// Genereller Socket
class Socket
{
	public:
		// class constructor
		Socket(char IpAddress[],u_short Port);
		bool Connect();
		// class destructor
		~Socket();
	private:
       int StartWinsock();
       char IpAddress[];
       u_short Port;
};

#endif // SOCKET_H

in meiner main dann

   Socket s("127.0.0.1",1889);
   if(s.Connect() == true){
           MessageBox(0,"Verbindung hergestellt","",0);               
   }

Das Problem liegt anscheinent hier:
addr.sin_port = htons((u_short)this->Port); (ich weiß, dass der carst schwachsinnig ist)
Wenn ich den Port normal dort eintrage, als Zahl, dann gibt es keine Probleme, die Verbindung wird aufgenommen.
Ich verwende devc++...
Ich schreibe noch nicht so lange in c++, würde mich freuen, wenn mir jemand sagen könnte, woran es liegt und wie ich es beheben kann.

[black0utCpp]

Gibts auch eine genaue Fehlermeldung ?
Überhaupt keine Fehlerbehandlung ist auch keine Option
Wenn der Cast sinnlos ist warum steht er dann noch da ?

[krustx]

if (connect(s,(SOCKADDR*)&addr,sizeof(SOCKADDR)) != SOCKET_ERROR
diese Bedingung ist nicht erfüllt, der socket connectet nicht.
ich hatte den Port ursprünglich als int deklariert und dann zum u_short gecastet, als mir auffiel, dass die funktion einen u_short als parameter erwartet.
hat aber alles nicht geholfen.

Überhaupt keine Fehlerbehandlung ist auch keine Option

sehe in dem satz keinen sinn, ich überprüfe doch nach jedem schritt, ob ein fehler aufgetreten ist.
Wier schon gesagt, addr.sin_port = htons(1889); fuktioniert wunderbar.

[black0utCpp]

Schau mal welchen Error Code WSAGetLastError() ausspuckt
also um nochmal sicher zu gehen kompillieren kannst du es nur
beim Connecten läufts schief

[black0utCpp]

Mach das u_short mal public vllt hilft das was

[krustx]

10061,WSAECONNREFUSED
No connection could be made because the target computer actively refused it. This usually results from trying to connect to a service that is inactive on the foreign host—that is, one with no server application running.

Es kann keine Verbindung hergestellt werden, weil auf dem Port keine Serveranwendung läuft. Ich habe allerdings eine Serveranwendung auf selbigem Port laufen.
Danke schonmal für deine Bemühungen.
Edit: daran kann es nicht liegen, da char[] IpAddress ja ebenfalls private ist...
habs versucht, funktioniert natürlich auch nicht.

[black0utCpp]

Kein Ding müsste eigtl an der server Anwendung liegen
kann ich vllt die Anwendung mal sehen ?

[krustx]

es liegt definitiv nicht an der serveranwendung. sobald ich der htons() Funktion den Port direkt übergebe:
htons(1889)
funktioniert es wunderbar, die Serveranwendung zeigt dann auch eine Verbindung an.
Das Problem wird clientseitig sein.
Die Serverapp ist in .net geschrieben, aber das tut nicht zur sache.
Ich habe mir mal this->Port ausgeben lassen, da kommt 12849 raus oO, obwohl ich im Konstruktior den Wert verändert habe.

[black0utCpp]

Komisch schreib einfach mal ohne den this Zeiger
du solltest auch WSACleanup() benutzen

[krustx]

habe ich schon versucht.
ist wirklich crazy, wenn ich
this->Port = 1889;
eine zeile vor
addr.sin_port = htons(this->Port);
schreibe funktioniert es nicht, aber wenn ich mir dann den Inhalt von this->Port ausgeben lasse, kommt auf einmal doch 1889 raus.

[black0utCpp]

Schön das Programmiersprachen IMMER logisch sind

[Shadow992]

Ich komme mit dem "this->Port=Port;" ganz erhlich nicht ganz klar...
Es kann sein, dass ich mich jetzt total irre, aber so wäre es doch um einiges einfacher/schöner:

PHP Code:

Socket::Socket(char IpAddress[],u_short Port1)
{
    Port = Port1;
    strcpy(this->IpAddress,IpAddress);
} 


Ähnliches gilt für "this->IpAddress".

Desweiteren verstehe ich nicht was genau "u_short" soll?
Warum nimmst du denn nicht einfach "unsigned short"?
Das ist sogar standartmäßig implementiert und damit sollten die meisten Funktionen klar kommen. Wie genau "u_short" definiert ist weiß ich nicht, aber ich finde man sollte den Standart einhalten, wenn möglich.

[krustx]

die funktion ist mit einem u_short in der winsock2.h so definiert.
Port = Port1; habe ich natürlich auch schon ausprobiert xD funktioniert ebenfalls nicht.
ich schnappe mir die werte jetzt einfach in der connect funktion, das geht soweit.

[vst0rm]

hi

in der socket.h Datei deklarierst du ja

//socket.h
char IpAddress[];

und in der socket.cpp Datei kopierst du

//socket.cpp
strcpy(this->IpAddress,IpAddress);

aber du initialisiert, so weit ich das gesehen habe, keinen Speicher für den "IpAddress" Pointer.
Versuch mal mit "new" Speicher zuzuweisen ;-)

[MrSm!th]

Oder nimm einen festen Wert als Arraygröße.
Eine IP kann ja nicht länger als "***.***.***.***" werden.