sitze momentan an der Erstellung eines IRC-Bots. Da ich ein C++-Anfänger bin, habe ich dieses Tutorial als Grundlage genommen. Zur Übersicht im folgenden der Code:

#include "IrcBot.h"
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <errno.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <netdb.h>
#include <arpa/inet.h>
#include <sys/wait.h>
#include <signal.h>
#include <time.h>

using namespace std;

#define MAXDATASIZE 100

IrcBot::IrcBot(char * _nick, char * _usr)
{
	nick = _nick;
	usr = _usr;
}

IrcBot::~IrcBot()
{
	close (s);
}

void IrcBot::start()
{
	string connectserver;

	cout<<"Type in Server:\n";
	cin>>connectserver;
	cout<<"Connecting to server: "<<connectserver<<"\n";

	struct addrinfo hints, *servinfo;

	setup = true;

	port = "6667";

	memset(&hints, 0, sizeof hints);

	hints.ai_family = AF_UNSPEC;
	hints.ai_socktype = SOCK_STREAM;

	int res;
	if ((res = getaddrinfo(connectserver.c_str(),port,&hints,&servinfo)) != 0)
	{
		setup = false;
		fprintf(stderr,"getaddrinfo: %s\n", gai_strerror(res));
	}

	if ((s = socket(servinfo->ai_family,servinfo->ai_socktype,servinfo->ai_protocol)) == -1)
	{
		perror("client: socket");
	}

	if (connect(s,servinfo->ai_addr, servinfo->ai_addrlen) == -1)
	{
		close (s);
		perror("Client Connect");
	}

	freeaddrinfo(servinfo);

	int numbytes;
	char buf[MAXDATASIZE];

	int count = 0;
	while (1)
	{
		count++;

		switch (count) {
			case 3:
					sendData(nick);
					sendData(usr);
				break;
			case 4:
				sendData("JOIN #ircbottest\r\n");
			default:
				break;
	}

		numbytes = recv(s,buf,MAXDATASIZE-1,0);
		buf[numbytes]='\0';
		cout << buf;

		msgHandel(buf);

		if (charSearch(buf,"PING"))
		{
			sendPong(buf);
		}

		if (numbytes==0)
		{
			cout << "----------------------CONNECTION CLOSED---------------------------"<< endl;
			cout << timeNow() << endl;

			break;
		}
	}
}

bool IrcBot::charSearch(char *toSearch, char *searchFor)
{
	int len = strlen(toSearch);
	int forLen = strlen(searchFor);

	for (int i = 0; i < len;i++)
	{
		if (searchFor[0] == toSearch[i])
		{
			bool found = true;
			for (int x = 1; x < forLen; x++)
			{
				if (toSearch[i+x]!=searchFor[x])
				{
					found = false;
				}
			}

			if (found == true)
				return true;
		}
	}

	return 0;
}

bool IrcBot::isConnected(char *buf)
{
	if (charSearch(buf,"/MOTD") == true)
		return true;
	else
		return false;
}

char * IrcBot::timeNow()
{
	time_t rawtime;
	struct tm * timeinfo;

	time ( &rawtime );
	timeinfo = localtime ( &rawtime );

	return asctime (timeinfo);
}

bool IrcBot::sendData(char *msg)
{
	int len = strlen(msg);
	int bytes_sent = send(s,msg,len,0);

	if (bytes_sent == 0)
		return false;
	else
		return true;
}

void IrcBot::sendPong(char *buf)
{

	char * toSearch = "PING ";

	for (int i = 0; i < strlen(buf);i++)
		{
			if (buf[i] == toSearch[0])
			{
				bool found = true;
				for (int x = 1; x < 4; x++)
				{
					if (buf[i+x]!=toSearch[x])
					{
						found = false;
					}
				}

				if (found == true)
				{
					int count = 0;
					for (int x = (i+strlen(toSearch)); x < strlen(buf);x++)
					{
						count++;
					}

					char returnHost[count + 5];
					returnHost[0]='P';
					returnHost[1]='O';
					returnHost[2]='N';
					returnHost[3]='G';
					returnHost[4]=' ';

					count = 0;
					for (int x = (i+strlen(toSearch)); x < strlen(buf);x++)
					{
						returnHost[count+5]=buf[x];
						count++;
					}

					if (sendData(returnHost))
					{
						cout << timeNow() <<"  Ping Pong" << endl;
					}

					return;
				}
			}
		}

}

void IrcBot::msgHandel(char * buf)
{

	if (charSearch(buf,"hi bot"))
	{
		sendData("PRIVMSG #ircbottest :hi :)\r\n");
	}
}

Als erstes möchte ich natürlich sowohl Server als auch Channel variabel machen. Ersteres ist mir bereits gelungen. Allerdings schaffe ich es nicht, selbiges für den channel hinzubekommen. Folgende Zeile macht Probleme:

sendData("JOIN #ircbottest\r\n");

Wie zu sehen, habe ich für das Wechseln des Servers per cout/cin die Variable connectserver erstellt, welche soweit auch funktioniert. Versuche ich das selbe mit einer connectchannel, welche ich in die aufgeführte Zeile einbinde, funktioniert mein Programm nicht mehr, da sendData vermutlich einen char anstatt string benötigt und darüber hinaus über zu viele Argumente meckert, da ich es mit %s einbinde.
Würde mich über Hilfe zur Lösung meines Problemes freuen

MfG
VB_94

Such bei Google mal nach "const correctness". Wenn du das durchgelesen hast, änder den Parameter von sendData von char * zu const char *. Anschließend kannst du mit der c_str()-Methode der std::string-Klasse den String an die Funktion übergeben. Sieht dann ungefähr so aus:

bool sendData(const char *buff)
{
    ...
}

...

std::string foo = "irgendwas";

sendData(foo.c_str());

int count = 0;
    for (int x = (i+strlen(toSearch)); x < strlen(buf);x++)
    {
	    count++;
    }

Nicht zu fassen, das steht so tatsaechlich in dem Tutorial drin.

#include "IrcBot.h"
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <errno.h>
#include <string.h>
#include <sys/types.h>
#include <sys/socket.h>
#include <netinet/in.h>
#include <netdb.h>
#include <arpa/inet.h>
#include <sys/wait.h>
#include <signal.h>
#include <time.h>

using namespace std;

#define MAXDATASIZE 100

IrcBot::IrcBot(char * _nick, char * _usr)
{
	nick = _nick;
	usr = _usr;
}

IrcBot::~IrcBot()
{
	close (s);
}

void IrcBot::start()
{
	string connectserver;

	cout<<"Type in Server:\n";
	cin>>connectserver;
	cout<<"Type in Channel/s:\n";
	cin>>connectchannel;
	cout<<"Connecting to server: "<<connectserver<<"\n";

	struct addrinfo hints, *servinfo;

	setup = true;

	port = "6667";

	memset(&hints, 0, sizeof hints);

	hints.ai_family = AF_UNSPEC;
	hints.ai_socktype = SOCK_STREAM;

	int res;
	if ((res = getaddrinfo(connectserver.c_str(),port,&hints,&servinfo)) != 0)
	{
		setup = false;
		fprintf(stderr,"getaddrinfo: %s\n", gai_strerror(res));
	}

	if ((s = socket(servinfo->ai_family,servinfo->ai_socktype,servinfo->ai_protocol)) == -1)
	{
		perror("client: socket");
	}

	if (connect(s,servinfo->ai_addr, servinfo->ai_addrlen) == -1)
	{
		close (s);
		perror("Client Connect");
	}

	freeaddrinfo(servinfo);

	int numbytes;
	char buf[MAXDATASIZE];

	int count = 0;
	while (1)
	{
		count++;

		switch (count) {
			case 3:
					sendData(nick);
					sendData(usr);
				break;
			case 4:
				sendData("JOIN %s\r\n", connectserver.c_str());
			default:
				break;
	}

		numbytes = recv(s,buf,MAXDATASIZE-1,0);
		buf[numbytes]='\0';
		cout << buf;

		msgHandel(buf);

		if (charSearch(buf,"PING"))
		{
			sendPong(buf);
		}

		if (numbytes==0)
		{
			cout << "----------------------CONNECTION CLOSED---------------------------"<< endl;
			cout << timeNow() << endl;

			break;
		}
	}
}

bool IrcBot::charSearch(char *toSearch, char *searchFor)
{
	int len = strlen(toSearch);
	int forLen = strlen(searchFor);

	for (int i = 0; i < len;i++)
	{
		if (searchFor[0] == toSearch[i])
		{
			bool found = true;
			for (int x = 1; x < forLen; x++)
			{
				if (toSearch[i+x]!=searchFor[x])
				{
					found = false;
				}
			}

			if (found == true)
				return true;
		}
	}

	return 0;
}

bool IrcBot::isConnected(char *buf)
{
	if (charSearch(buf,"/MOTD") == true)
		return true;
	else
		return false;
}

char * IrcBot::timeNow()
{
	time_t rawtime;
	struct tm * timeinfo;

	time ( &rawtime );
	timeinfo = localtime ( &rawtime );

	return asctime (timeinfo);
}

bool IrcBot::sendData(char *msg)
{
	int len = strlen(msg);
	int bytes_sent = send(s,msg,len,0);

	if (bytes_sent == 0)
		return false;
	else
		return true;
}

void IrcBot::sendPong(char *buf)
{

	char * toSearch = "PING ";

	for (int i = 0; i < strlen(buf);i++)
		{
			if (buf[i] == toSearch[0])
			{
				bool found = true;
				for (int x = 1; x < 4; x++)
				{
					if (buf[i+x]!=toSearch[x])
					{
						found = false;
					}
				}

				if (found == true)
				{
					int count = 0;
					for (int x = (i+strlen(toSearch)); x < strlen(buf);x++)
					{
						count++;
					}

					char returnHost[count + 5];
					returnHost[0]='P';
					returnHost[1]='O';
					returnHost[2]='N';
					returnHost[3]='G';
					returnHost[4]=' ';

					count = 0;
					for (int x = (i+strlen(toSearch)); x < strlen(buf);x++)
					{
						returnHost[count+5]=buf[x];
						count++;
					}

					if (sendData(returnHost))
					{
						cout << timeNow() <<"  Ping Pong" << endl;
					}

					return;
				}
			}
		}

}

void IrcBot::msgHandel(char * buf)
{

	if (charSearch(buf,"hi bot"))
	{
		sendData("PRIVMSG #ircbottest :hi :)\r\n");
	}
}

Bei der Abfrage nach Server/Channel wird nun die Variable connectchannel erstellt, welche ich in der folgenden Zeile verwenden möchte:

sendData("JOIN %s\r\n", connectserver.c_str());

Genau für diese Zeile bekomme ich auch eine Fehlermeldung. Wie kann ich die Variable hier richtig einbinden?

PS: Ich weiß nicht, ob es nicht genügend rüberkam, dass ich ANFÄNGER!!! bin. Seit ich mich mit C++ beschäftige, bekomme ich, sobald ich online Hilfe suche, zu 90% Antworten im Sinne von "dein code ist schlecht", "würde ich viel besser machen" etc. Wenn ihr nicht helfen wollt, ignoriert den Thread doch einfach. Sind alle C++-Entwickler so elitär?

Sind alle C++-Entwickler so elitär?

Sind denn alle C++ Anfänger so überheblich? ._.

Ich weiß nicht, ob es nicht genügend rüberkam, dass ich ANFÄNGER!!! bin.

...und da ist es natürlich genau das richtige, wenn du mit einem IRC-Bot anfängst!

Leg dir ein gutes! Buch zu, direkt hier im Unterforum einer der ersten gepinnten Threads mit Buchtipps.

Außerdem ist das wohl mal wieder so ein Typisches "C++ ist C mit cout"-Tutorial. Ein Tutorial bzw. so ein Beispiel solltest du erst zu verstehen versuchen, wenn du es verstehen kannst und bei dir hapert es wohl schon an Grundbegriffen, denn ein "Tutorial" über einen IRC-Bot dazu gedacht ist, dir die Grundsematik von C++ beizubringen. Mal ganz davon abgesehen, dass das Beispiel Rotz ist.
Für die Zukunft hier im Forum: Wenn ein Fehler kommt, ist das natürlich schön, wenn du uns das sagst, nützt allerdings nichts, wenn du net sagst, WAS für ein Fehler kommt.

Einen schönen Nachmittag.
Tobi