hab da nen Problem:

Ich hab nen Drucker dem schick ich ein Label das er abarbeiten soll

<STX>TZTextname.00I;10<CR>Daten<CR><ETX>

Wenn ich diesen Befehl sende bekomm ich ein <ACK> als antwort
und wenn der Drucker gedruckt hat: <STX>1Ok<ETX>

Heist ich sende per Befehl: "Set_VTEXT"
die variabelen Daten

und will per Befehl: "IsPrint"
abfragen ob auf dem Socket Daten liegen -> diese abholen -> und überprüfen ob die Daten des Druckers mit den erwarteten Daten übereinstimmen um zu sagen er hat gedruckt.

Mein Problem war bisher das das <ACK> als eine Antwort kam und irgendwann später <STX>1Ok<ETX> als Antwort kam und mein Programm per recv() ewig auf die Antwort gewartet hat und dadurch mein Programm geblockt hat...

nun hab ich gelesen das man mit Hilfe des Befehls select() nachfragen kann ob überhaupt erstmal was auf dem Socket liegt was man mit recv() abholen kann

also hab ich das versucht einzubauen ...
nur leider liefert mir das select immer nur eine fehlermeldung (10038) zurück...

was mach ich falsch??

bool CMainFrame::cmdServer(CString cmd, CString ip, int port){
	/////////////Connect Anfang////////////////
	long rc;
	SOCKET s;
	SOCKADDR_IN addr;
	bool ret=false;
	inUse = true;
	FD_SET fdSet;

	CStringArray r_cmd;
	int iNum_CMD = ParseTokens( r_cmd, cmd, "|");
	CString * param;
	int anz_param = 0;
	if(iNum_CMD>1){
		cmd = r_cmd[0];
		param = new CString[iNum_CMD-1];

		anz_param=iNum_CMD-1;

		for(int cmd_i=0; cmd_i<(iNum_CMD-1); cmd_i++){
			param[cmd_i] = r_cmd[cmd_i+1];
		}
	}

	rc=startWinsock();
	if(rc!=0) {
		CString str;
		str.Format("%d",rc);
	    AfxMessageBox("Fehler: startWinsock, fehler code: "+str);
		WSACleanup();
		ret=false;
		return ret;
	} 

	s=socket(AF_INET,SOCK_STREAM,0);
	if(s==INVALID_SOCKET) {
		CString str;
		str.Format("%d",WSAGetLastError());
		AfxMessageBox("Fehler: Der Socket konnte nicht erstellt werden, fehler code: "+str);
		closesocket(s);
		WSACleanup();
		ret=false;
		return ret;
	}

	memset(&addr,0,sizeof(SOCKADDR_IN)); // zuerst alles auf 0 setzten 
	addr.sin_family=AF_INET;
	addr.sin_port=htons(port);
	addr.sin_addr.s_addr=inet_addr(ip);

	rc=connect(s,(SOCKADDR*)&addr,sizeof(SOCKADDR));
	if(rc==SOCKET_ERROR){
		CString str;
		str.Format("%d",WSAGetLastError());
		AfxMessageBox("Fehler: connect gescheitert, fehler code: "+str);
		closesocket(s);
		WSACleanup();
		ret=false;
		//OnVerbindungVerbindungtrennen();
		return ret;
	} 
	/////////////Connect Ende////////////////	

	/////////////Befehle Anfang////////////////

	FD_SET(s,&fdSet);

	if(cmd.CompareNoCase("BLUHM_ON")==0){
		int len=0;
		CString cmd="";
		CString cmd_return="";
		//Ok<CR>
		CString cmd_return_soll="4F 6B 0D";

		FD_ZERO(&fdSet); // Inhalt leeren

		CString cmd_erg;
		// <ESC> + C1 + <CR>
		cmd_erg.Format("%cC1%c",ConvertStringHexToChar("1B"), ConvertStringHexToChar("0D"));

		len=cmd_erg.GetLength()+1;
		char * buf = new char[len];
		memcpy(buf,cmd_erg, len);

		ret=SendBlock(s, buf,len);

		char *recvbuffer=NULL;
		recvbuffer = new char[1024];
		rc=recv(s,recvbuffer,1024,0);
		if(rc==SOCKET_ERROR) {
			CString str;
			str.Format("%d",WSAGetLastError());
			AfxMessageBox("Fehler: recvfrom, fehler code: "+str);
		} else {
			recvbuffer[rc]='\0';
			CString tmp="", tmp2="";

			for(int xx=0; xx<rc; xx++){
				tmp.Format("%x", recvbuffer[xx]);
				if(tmp.GetLength()<2){
					tmp="0"+tmp;
				}
				if(tmp.GetLength()>2){
					CString tmp3="";
					for(int iy=tmp.GetLength()-2; iy<tmp.GetLength(); iy++){
						tmp3=tmp3+tmp.GetAt(iy);
					}
					tmp=tmp3;
				}
				if(xx<(rc-1)){
					tmp=tmp+" ";
				}
				tmp2=tmp2+tmp;
			}
			if(tmp2.CompareNoCase(cmd_return_soll)!=0){
				CString meldung;
				meldung.Format("Fehler: %s",tmp2);
				AfxMessageBox(meldung);
			}else{
				ret=TRUE;
				CString meldung;
				meldung.Format("Kein Fehler: %s",tmp2);
				AfxMessageBox(meldung);
			}
		}
		delete[] recvbuffer;
		delete[] buf;
	}

	if(cmd.CompareNoCase("BLUHM_SET_VTEXT")==0){
		int len=0;
		CString cmd="";
		// <ACK>
		CString cmd_return_soll_1="06";
		// <STX>1Ok<ETX>
		CString cmd_return_soll_2="02 31 4F 4B 03";

		CString lm=param[0];
		//CString lm="GS1.00I";

		CString cmd_erg;
		// <STX>TZTextname.00I;10<CR><ETX>
		cmd_erg.Format("%cTZ%s;10%c",ConvertStringHexToChar("02"), lm, ConvertStringHexToChar("0D"));

		for(int i=1; i<anz_param; i++){
			cmd_erg=cmd_erg+param[i]+ConvertStringHexToChar("0D");
		}

		cmd_erg = cmd_erg + ConvertStringHexToChar("03");

		len=cmd_erg.GetLength()+1;
		char * buf = new char[len];
		memcpy(buf,cmd_erg, len);

		ret=SendBlock(s, buf,len);

		char *recvbuffer=NULL; 
		recvbuffer = new char[1024];
		rc=recv(s,recvbuffer,1024,0);
		if(rc==SOCKET_ERROR) {
			CString str;
			str.Format("%d",WSAGetLastError());
			AfxMessageBox("Fehler: recvfrom, fehler code: "+str);
		} else {
			recvbuffer[rc]='\0';
			CString tmp="", tmp2="";

			for(int xx=0; xx<rc; xx++){
				tmp.Format("%x", recvbuffer[xx]);
				if(tmp.GetLength()<2){
					tmp="0"+tmp;
				}
				if(xx<(rc-1)){
					tmp=tmp+" ";
				}
				tmp2=tmp2+tmp;
			}
			if(tmp2.CompareNoCase(cmd_return_soll_1)==0){
				ret=TRUE;
				CString meldung;
				meldung.Format("Kein Fehler: %s",tmp2);
				AfxMessageBox(meldung);

			}else{
				CString meldung;
				meldung.Format("Fehler: %s",tmp2);
				AfxMessageBox(meldung);
			}
		}
		delete[] recvbuffer;
		delete[] buf;
	}

	if(cmd.CompareNoCase("BLUHM_ISPRINT")==0){
		int len=0;
		CString cmd="";
		CString cmd_return="";
		// <STX>1Ok<ETX>
		CString cmd_return_soll="02 31 4F 4B 03";
		CString tmp="", tmp2="";

		struct timeval tv;
		tv.tv_sec = 1;
		tv.tv_usec = 0;

		char *recvbuffer=NULL;
		rc=select(1,&fdSet,NULL,NULL, &tv); 
	    if(rc==SOCKET_ERROR) {
			CString ausg;
			ausg.Format("Fehler: select, fehler code: %i \n",WSAGetLastError());
			AfxMessageBox(ausg);
		}else{

			CString ausg;
			ausg.Format("%i",rc);
			AfxMessageBox("ausg:"+ausg);

			recvbuffer = new char[1024];
			rc=recv(s,recvbuffer,1024,0);

			if(rc==SOCKET_ERROR) {
				CString str;
				str.Format("%d",WSAGetLastError());
				AfxMessageBox("Fehler: recvfrom, fehler code: "+str);
			} else {
				recvbuffer[rc]='\0';

				for(int xx=0; xx<rc; xx++){
					tmp.Format("%x", recvbuffer[xx]);
					if(tmp.GetLength()<2){
						tmp="0"+tmp;
					}
					if(xx<(rc-1)){
						tmp=tmp+" ";
					}
					tmp2=tmp2+tmp;
				}
				//AfxMessageBox(tmp2);
				if(tmp2.CompareNoCase(cmd_return_soll)==0){
					ret=TRUE;
					CString meldung;
					meldung.Format("Kein Fehler: %s",tmp2);
					AfxMessageBox(meldung);
				}else{
					CString meldung;
					meldung.Format("Fehler: %s",tmp2);
					AfxMessageBox(meldung);
				}
			}
		}
		delete[] recvbuffer;
	}

	/////////////Befehle Ende////////////////

	/////////////Disconnect Anfang////////////////
	closesocket(s);
	WSACleanup();
	/////////////Disconnect Ende////////////////
	inUse = false;
	return ret;
}

mfg LT

versuchs mal mit rc = select(1, &fdSer, NULL,NULL, tv);

Gruss IchBinNichtSicher

versuchs mal mit rc = select(1, &fdSer, NULL,NULL, tv);

^^ was soll das bringen ? fdSer gibts bei mir net als variable und tv ohne & bringt fehelrmeldung

reihnfolge bei mir siet wie folgt aus:

-> Connect
-> FD_SET(soc,&fdSet);
-> FD_ZERO(&fdset)
-> Befehl: ON an Drucker (Bluhm_ON)
-> Disconnect

-> Connect
-> FD_SET(soc,&fdSet);
-> Befehl: SET_VTEXT an Drucker (Bluhm_SET_VTEXT) mit entsprechenden Parameter
-> Disconnect

-> Connect
-> FD_SET(soc,&fdSet);
-> Befehl: IsPrint an Drucker (Bluhm_ISPRINT)
-> rc=select(1,&fdSet,NULL,NULL, &tv);
-> <-- hier kommt fehlermeldung 10038 (Objekt sei kein socket)
-> Disconnect

^^ wieso kommt da das fdset kein socket sei oder was versteh ich da falsch?

ok frage zu deinem geschriebenen:

reihnfolge bei mir siet wie folgt aus:

-> Connect
-> FD_SET(soc,&fdSet);
-> FD_ZERO(&fdset)
-> Befehl: ON an Drucker (Bluhm_ON)
-> Disconnect

-> Connect
-> FD_SET(soc,&fdSet);
-> Befehl: SET_VTEXT an Drucker (Bluhm_SET_VTEXT) mit entsprechenden Parameter
-> Disconnect

-> Connect
-> FD_SET(soc,&fdSet);
-> Befehl: IsPrint an Drucker (Bluhm_ISPRINT)
-> rc=select(1,&fdSet,NULL,NULL, &tv);
-> <-- hier kommt fehlermeldung 10038 (Objekt sei kein socket)
-> Disconnect

^^ wieso kommt da das fdset kein socket sei oder was versteh ich da falsch?

FD_SET(s,&fdSet);

    if(cmd.CompareNoCase("BLUHM_ON")==0){
        int len=0;
        CString cmd="";
        CString cmd_return="";
        //Ok<CR>
        CString cmd_return_soll="4F 6B 0D";

        FD_ZERO(&fdSet); // Inhalt leeren

Du fügst den Socket der verbindungen annimmt zu, fdSet hinzu und leerst den Inhalt ein paar Zeilen wieder.

Gruss IchBinNichtSicher

also laut der webseite (sorry link net mehr da) hies es :
- man soll den socket aufmachen
- dann den fdset
- dann fdzero <--- nur beim 1. mal

daher hab ich das fdzero dort hingeschrieben wo ich das zwar 1 mal ausführe aber vor dem setvtext und vor isprint

hmmm jo mach ich weil ich gelesen hab das man das machen muss ...

also laut der webseite (sorry link net mehr da) hies es :
- man soll den socket aufmachen
- dann den fdset
- dann fdzero <--- nur beim 1. mal

daher hab ich das fdzero dort hingeschrieben wo ich das zwar 1 mal ausführe aber vor dem setvtext und vor isprint

fdzero must du vor fdset hinschreiben.

Gruss IchBinNichtSicher

http://msdn.microsoft.com/en-us/library/ms740141(VS.85).aspx

Ein fd_set ist nichts weiter, wie eine Datenstruktur, wo du Sockets eintragen kannst. FD_ZERO rufts du nur auf, um sicher zu stellen, dass das fd_set leer ist. Das fd_set selber macht gar nichts. Damit damit etwas geschieht, musst du es an select() übergeben, oder eine andere Funktion, die damit arbeiten kann.
Eigentlich ist das auch in der MSDN recht umfangreich und ausführlich beschrieben.

http://msdn.microsoft.com/en-us/library/ms740141(VS.85).aspx

So wie ich immer gedacht habe löscht FD_ZERO den Inhalt von fd_set, und das hat er getan nachdem er FD_SET aufruft.

Er übergibt fd_set leer an select().
Möglicherweise kommt daher die Fehlermeldung 10038 (Objekt ist kein socket).

Gruss IchBinNichtSicherr

Nick Unbekannt schrieb:

Ein fd_set ist nichts weiter, wie eine Datenstruktur, wo du Sockets eintragen kannst. FD_ZERO rufts du nur auf, um sicher zu stellen, dass das fd_set leer ist. Das fd_set selber macht gar nichts. Damit damit etwas geschieht, musst du es an select() übergeben, oder eine andere Funktion, die damit arbeiten kann.
Eigentlich ist das auch in der MSDN recht umfangreich und ausführlich beschrieben.

http://msdn.microsoft.com/en-us/library/ms740141(VS.85).aspx

So wie ich immer gedacht habe löscht FD_ZERO den Inhalt von fd_set, und das hat er getan nachdem er FD_SET aufruft.

Er übergibt fd_set leer an select().
Möglicherweise kommt daher die Fehlermeldung 10038 (Objekt ist kein socket).

Gruss IchBinNichtSicherr

^^ das problem hierbei ist aber :

ich rufe das conntect mehrfach auf !!!

also ich connecte mich führe nen start des druckers aus und disconnecte mich wieder...

dann concete ich mich um ein set text auszuführen (dieses liefert 2 rückgaben auf recv)

ich warte die 1. ab und disconnecte mich wieder

dann conntecte ich mich neu und warte auf das 2. recv

und ich glaube das geht nicht ... aber da bin ich mir net sicher -.-

mfg LT

So wie ich immer gedacht habe löscht FD_ZERO den Inhalt von fd_set,

Das ist auch richtig so. Ich habe auch nicht auf deinen Beitrag geantwortet. Mir ist nur aufgefallen, dass er im gleichen Kontext die Makros ohne select() benutzt und das ist Unsinn.

IchBinNichtSicher schrieb:

Er übergibt fd_set leer an select().
Möglicherweise kommt daher die Fehlermeldung 10038 (Objekt ist kein socket).

Eigentlich und da bin ich mir nicht sicher, dürfte er die Fehlermeldung nur bringen, wenn du versuchst einen Nicht-Socket in der Datenstruktur abzulegen. Ob man bei select() zwangsläufig mindestens einen Socket angeben muss weiß ich nicht, aber man kann über das Timeout eine Pause realisieren, welche auch ohne Socket nützlich sein kann.

jo es hat schon nen sinn das ich mich immer nur für ein kommando mit dem drucker verbinde ... das tool was ich schreib is nen überwachungstool und es soll von verschiedenen stellen möglich sein den drucker zu überwachen ... daher muss ich das so programmieren ...

zum thema unleserlichkeit ... hmmm ganz ehrlich ich dachte eigentlich das ich es eben schon gut gekapselt hab ... was soll ich denn deiner meinung nach noch auslagern ???

mfg lt

aber naja so schwierig find ich meinen quellcode gar net ...
erst kommt n bissi was zur socket-verbindung
dann kommen die befehle die in senden und empfangen aufgegliedert sind
und dann kommt n bissi dicsonncet kram also find den code nu net so unübersichtlich ... aber is sicher ansichtssache

aber problem besteht immer noch ...
also im mom hab ich das fd_zero vor den fd_set gesetzt
(wohlgemerkt bei jeder komunikation)

ergebnis= ich bekomm immer ne 0 als returnwert von select()

is also auch net das was ich will ...

Zum einen, wenn 0 zurückkommt bedeutet das nur das dein Timeout von 1 sec abgelaufen ist und mehr nicht. Also entweder erhöst du deinen Timeout oder du musst die Methode mehrfach in einer Schleife aufrufen. So in etwas (Vorsicht nur PseudoCode):

while (empfangen) {
  FD_ZERO(fdset);
  FD_SET(socket, &fdset);
  ret = select(0, &fdset, null, null, &tv);
  if (0 < ret) {
    if (FD_ISSET(socket, &fdset)) {
      // es kamm was an
      ret = recv(....)
      if (SOCKET_ERROR != ret) {
        // Daten auswerten
      }
      else {
        // Die Verbindung wurde getrennt
      }
      empfangen = true;
    }
  }
} // while (....

Nur so nebenbei, select ist ausgelegt um auf mehrere Sockets gleichzeitig zu hören, du könntest auch jederzeit denn Socket entsprechend umstellen das er Asynchron läuft, dann musst du die Wartezeit zwischen den Aufrufen nur selber mit Sleep einbauen. Aber um die While-Schleife wirst nicht rumkommen.

MfG Marco

die schleife hatte ich eh vor zu benutzen

mein programm ruft ja den cmdserver auf ...

so nach dem motto:

CString cmd="BLUHM_ON";
if(cmdServer(cmd, ip , port)){

  cmd="BLUHM_SET_VTEXT";
  if(cmdServer(cmd, ip , port)){

    do{
      cmd="BLUHM_ISPRINT";
      if(cmdServer(cmd, ip , port)){
        AfxMessageBox("Hat gedruckt mach noch was");
      }
    }while("bis ich sage stop");
  }
}

mfg kala