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

#define MAX_COMMAND_LENGTH	50

using namespace std;

char szCurrentCommand[MAX_COMMAND_LENGTH];

void EvaluateCommand(char* pszText);

int main()
{
	EvaluateCommand("Test");
	cout << szCurrentCommand << '\n';
	_getch();
	return 0;
}

void EvaluateCommand(char* pszText)
{
	int iIndex		 = 0;
	int iIterator	  = 0;
	char szTemp		= 'a';
	char szCommand[MAX_COMMAND_LENGTH];

	//Iterate from the first letter to the last one and save those letters to a buffer
	while((szTemp != '\n')||(szTemp != ' ')||(szTemp != '\0')||(iIterator <	MAX_COMMAND_LENGTH)||(iIndex < strlen(pszText)))
	{
		szTemp = pszText[iIndex];
		szCommand[iIterator] = pszText[iIndex];
		iIterator++;
		iIndex++;
	}

	szCommand[iIterator] = '\0';	//Zero-Terminate the Command-String

	strncpy(szCurrentCommand, szCommand, sizeof(szCommand));
}

Ich bin mir eigentlich fast sicher, dass ich wieder einmal etwas nicht
korrekt angewendet oder verstanden habe. Allerdings komme ich einfach nicht
auf die Lösung und hoffe ihr könnt mir helfen.

Gruß Honky

Für C++:

#include <iostream>
#include <string>

using namespace std;

string EvaluateCommand(const string &str)
{
	return str.substr(0, str.find(' '));
}

int main()
{
    cout << EvaluateCommand("Test") << '\n';
    return 0;
}

Was mir grad noch beim ueberfliegen stilistisch aufgefallen ist: Ungarische Notation ist allgemein verpoehnt, aber wenn man sie schon verwendet, dann richtig (sz steht fuer zeroterminated string, du benutzt es aber fuer ein einzelnes Char).

Entscheide dich bitte: Willst du C oder C++ programmieren. Dann kann man dir auch besser helfen.

string EvaluateCommand(const string &str)
{
	return str.substr(0, str.find(' '));
}

Ich hätte jetzt eher an das gedacht:

string str = "Test";
string copy;
istringstream(str) >> copy; // Schön, dass es in C++11 mit einem Temporary funktioniert. :-)

Aber was nun wirklich die effizienteste Methode ist, das erste Wort einer Zeichenkette zu extrahieren, keine Ahnung?

Ich weiß nicht. Mir gefallen C-Strings.

Das ist keine Frage des Geschmacks, sondern eine Frage der SPrache, die du benutzen möchtest. Ich behaupte auch mal, du kennst weder std::string noch C-Strings so richtig gut, wenn dir C-Strings gefallen .

Und die WinApi arbeitet ja auch konstant
mit C-Strings. Oder nicht?

Ja, aber es ist ja nicht so, dass man in C++ am rohen Winapi-Interface rumfummeln würde und selbst wenn, kann man zur Not auch mit std::string aus C++ einen C-String erzeugen.

Aber nochmal: Die Frage ist, welche Sprache du lernen möchtest, C oder C++. Im Moment lernst du (wahrscheinlich aus einer sehr schlechten Quelle) irgendwie beides so halb. Das wird nicht lange gut gehen. Du bist gerade an dem Punkt, an dem es das erste Mal krachen wird, danach wird's nur schlimmer.

Aber in vielen C++-Büchern werden ja auch C-Strings benutzt.

Ich äußere mal Zweifel an der Qualität deiner Literatur. Dirk Louis? Jürgen Wolf? Galileo-Verlag?

const

Warum verwendest du ein #define als Konstante statt

const

Hat er schon erklärt:

Honky schrieb:

Galileo Verlag

IrgendeinName schrieb:

Warum verwendest du ein #define als Konstante statt

const

Hat er schon erklärt:

Honky schrieb:

Galileo Verlag

LOL
Ja ok, ich habs verstanden. Ich werde mir einen besseren Stil aneignen.

Ich werde mir einen besseren Stil aneignen.

Eigne Dir lieber ein gutes Buch an. In Sepps Signatur gibt es einen Link mit Buchempfehlungen.

(Skalar-Literal) Makros.

Woher nimmst du nur die Wörter.

if( 1==sscanf(" blafasel\t\n","%49s",chararray50) )
  puts(chararray50);
else
  puts("EOF oder Lesefehler");

Und der Ausgangsstring darf sogar const sein, bzw. implizit const als Stringliteral.

Für das Separieren eines durch Whitespaces getrennten Teilstrings aus einem String nimmt man sscanf und einen Einzeiler wie

if( 1==sscanf(" blafasel\t\n","%49s",chararray50) )
  puts(chararray50);
else
  puts("EOF oder Lesefehler");

Und der Ausgangsstring darf sogar const sein, bzw. implizit const als Stringliteral.

tinnef.
ich nehme an, das wort ist vorher nicht bekannt. dann nimmt man funktionen wie find, copy, etc. aus der string-bibliothek.
http://www.cplusplus.com/reference/string/string/

Für das Separieren eines durch Whitespaces getrennten Teilstrings aus einem String nimmt man sscanf und einen Einzeiler wie

if( 1==sscanf(" blafasel\t\n","%49s",chararray50) )
  puts(chararray50);
else
  puts("EOF oder Lesefehler");

Und der Ausgangsstring darf sogar const sein, bzw. implizit const als Stringliteral.

Sind wir im C-Forum? Geh bloss weg mit diesem Dreck...

std::string hw = "Hallo Welt!",
            erstesWort;

std::istringstream(hw) >> erstesWort;

Oder dein Beispiel (wenn ich es verstanden habe):

std::string hw = " blafasel\t\n",
            erstesWort;

std::istringstream(hw) >> std::ws >> erstesWort;

@Eisflamme: Obiges ist einfacher und braucht kein boost :p

Sorry fuer die ganzen edits.

Sind wir im C-Forum? Geh bloss weg mit diesem Dreck...

armes deutschland ...

Sone schrieb:

Sind wir im C-Forum? Geh bloss weg mit diesem Dreck...

armes deutschland ...

Nicht annaehrend so arm wie dein Post an Argumentation und Sinn.

Diese Variante von Dir geht aber erst mit C++11 so kurz, richtig?

Für ein Element ist das sicher am Besten so. Für mehrere packt split das aber alles schon brav in einen vector. Gut, ein kleines while und push_back darauf hin geht wohl auch.

O ja:

std::vector<std::string> strings;
    std::istringstream stream(" das ist die geschischte\t von\nbla");

    while( stream )
    {
        strings.push_back( std::string() );
        stream >> std::ws >> strings.back();
    }

Diese Variante von Dir geht aber erst mit C++11 so kurz, richtig?

Nope.

Eisflamme schrieb:

Diese Variante von Dir geht aber erst mit C++11 so kurz, richtig?

Nope.

Doch. Er meinte:

std::string hw = "Hallo Welt!",
erstesWort;
std::istringstream(hw) >> erstesWort; // Error: no match for 'operator>>' in 'std::basic_istringstream<char>((*(const __string_type*)(& hw)), (std::_Ios_Openmode)8u) >> erstesWort

Und das hatte ich bereits ganz oben geschrieben.

Gugelmoser schrieb:

Sone schrieb:

Eisflamme schrieb:

Diese Variante von Dir geht aber erst mit C++11 so kurz, richtig?

Nope.

Doch. Er meinte:

std::string hw = "Hallo Welt!",
erstesWort;
std::istringstream(hw) >> erstesWort; // Error: no match for 'operator>>' in 'std::basic_istringstream<char>((*(const __string_type*)(& hw)), (std::_Ios_Openmode)8u) >> erstesWort

Und das hatte ich bereits ganz oben geschrieben.

Ah shit, jetzt sehe ich es auch. Klar, hast Recht.
Dachte, ging schon immer mit rtemporaeren Objekten.