"hAllO wElT" -> "Hallo Welt"

Befor ich nun dafür eine Funktion schreibe wollte ich wissen ob es sowas schon gibt. Wäre gut wenn mir das jemand sagen könnte.

Hab dazu was rumprobiert.
Kann mir wer sagen, wieos das hier zum Absturz führt?

#include <cstdlib>
#include <iostream>
using namespace std;

char* up_first_char(char* sentence)
{
	bool upped = false;

	for (unsigned short int i = 0; i < strlen(sentence); i++)
	{
		if (sentence[i] == ' ')
			// Nächstes Zeichen gilt als erstes Zeichen des nächsten Wortes
			upped = false;
		else if (!upped)
		{
			// Dieses Zeichen gilt als erster Buchstabe eines aktuellen Wortes
			switch (sentence[i])
			{
				case 'a': sentence[i] = 'A'; break;
				case 'b': sentence[i] = 'B'; break;
				case 'c': sentence[i] = 'C'; break;
				case 'd': sentence[i] = 'D'; break;
				case 'e': sentence[i] = 'E'; break;
				case 'f': sentence[i] = 'F'; break;
				case 'g': sentence[i] = 'G'; break;
				case 'h': sentence[i] = 'H'; break;
				case 'i': sentence[i] = 'I'; break;
				case 'j': sentence[i] = 'J'; break;
				case 'k': sentence[i] = 'K'; break;
				case 'l': sentence[i] = 'L'; break;
				case 'm': sentence[i] = 'M'; break;
				case 'n': sentence[i] = 'N'; break;
				case 'o': sentence[i] = 'O'; break;
				case 'p': sentence[i] = 'P'; break;
			}

			// Das erste Zeichen des aktuellen Wortes wurde geupt
			upped = true;
		}
	}

	return sentence;
}

int main(int argc, char *argv[])
{
	cout << up_first_char("hallo welt") << endl;

    system("PAUSE");
    return 0;
}

PS: Einfacher geht es mit _toupper und _tolower ...

und natürlich hast Du vergessen die grossbuchstaben im Wort klein zu machen...

Ich habe seinen Wunsch als Übung genommen und wollte
eigentlich nur wissen, woran der Absturz liegt...
Meine Funktion soll den ersten Buchstaben jedes Wortes uppen.
Sry, wenn ich einfach mal sein Thema benutzt habe.

// edit
Das gibt's doch nicht!

#include <cstdlib>
#include <iostream>
using namespace std;

char* up_first_chars(char* sentence)
{
	bool checked = false;
	int temp;

	for (unsigned short int i = 0; i < strlen(sentence); i++)
	{
		if (sentence[i] == ' ')
			// Nächstes Zeichen gilt als erstes Zeichen des nächsten Wortes
			checked = false;
		else if (!checked)
		{
			// Dieses Zeichen gilt als erster Buchstabe eines aktuellen Wortes
			sentence[i] = _toupper(sentence[i]);

			// Für das aktuelle Wort erledigt.
			checked = true;
		}
	}

	return sentence;
}

int main(int argc, char *argv[])
{
	cout << up_first_chars("hallo welt") << endl;

    system("PAUSE");
    return 0;
}

Woran liegt der Programmabsturz!?!?
Kompilieren tut Dev-C++ es ja!

for (unsigned short int i = 0; i < strlen(sentence); i++)

sollte man besser so:

for (unsigned short int i = 0, iLen = strlen(sentence); i < iLen; i++)

machen, da der Teil bei for zwischen den beiden ; ; bei jeder Wiederholung
erneut ausgeführt wird.
Bei längeren Zeichenketten sind das große Performanceeinbußen!

außerdem frage ich mich, wozu diese

int temp;

gut sein soll ???
schließlich verwendest du es ja nicht.

Es wäre vllt auch hilfreich, wenn du uns verraten würdest, an welcher Stelle der Absturz passiert.

Hier der ganze source

#include <cstdlib>
#include <iostream>
using namespace std;

char* up_first_char(char* sentence)
{
	bool checked = false;

	for (unsigned short int i = 0, iLen = strlen(sentence); i < iLen; i++)
	{
		if (sentence[i] == ' ')
			// Nächstes Zeichen gilt als erstes Zeichen des nächsten Wortes
			checked = false;
		else if (!checked)
		{
			// Dieses Zeichen gilt als erster Buchstabe eines aktuellen Wortes
			sentence[i] = _toupper(sentence[i]);

			// Für das aktuelle Wort erledigt.
			checked = true;
		}
	}

	return sentence;
}

int main(int argc, char *argv[])
{
	cout << up_first_char("hallo welt") << endl;

    system("PAUSE");
    return 0;
}

Und hier der screenshot vom Übel
http://img3.imagebanana.com/view/kcgxdnl6/prob.png
Keine compile errors

Du übergibst hier direkt ein String-Literal einer Funktion, welche ein "char*" erwartet. Dies sollte IMHO eine Compiler-Warnung erzeugen...

Verwende:

cout << up_first_char(strdup("hallo welt")) << endl;

und es sollte gehen... (PS: der SPeicher wird hier dann nicht mehr freigegen...; verwende am besten std::string).

Jochen Kalmbach schrieb:

und es sollte gehen... (PS: der SPeicher wird hier dann nicht mehr freigegen...; verwende am besten std::string).

Du meinst, das auf dem heap liegene char array, dessen Pointer
der Funktion als Parameter übergeben wird, wird nach dem
return nicht wieder aus dem Speicher entfernt?
Wie wäre es denn damit?

int main(int argc, char *argv[])
{
	char * bla;
	cout << up_first_char(bla = strdup("hallo welt")) << endl;
	delete[] bla;

    system("PAUSE");
    return 0;
}

#include <iostream>
#include <stdio.h>

using namespace std;

string a;

void down(int i,int L,bool inside)
{
    if(inside==true)
    {i++;}
    else
    {i++;}

    for(int x=i;x<=L;x++)
    {
        if(a[x]!=' ')
        {
            if(a[x]<=90 && a[x]>=65)
            {
                a[x]=a[x]+32;
            }
        }
    }
}

void charwork(char *letter)
{
    a =letter;
    int L= a.length(),Zusatz=0;
    for(int i=0;i<=L;i++)
    {
        Zusatz=i;
        if(a[i]==' ')
        {
            Zusatz++;
            if(a[Zusatz]<=122 && a[Zusatz]>=97)
            {
               a[Zusatz]=a[Zusatz]-32;
               down(Zusatz,L,true);
            }
        }
        if(i==0 && a[i]<=122 && a[i]>=97)
        {
             a[i]=a[i]-32;
             down(i,L,false);
        }
    }
    cout <<a<<endl;
}
int main(int argc, char *argv[])
{
     charwork("  hAllO du AAa");

    system("PAUSE");
    return 0;
}

if(inside==true)
{i++;}
else
{i++;}

Sehr sinnreich - eine if-else macht nur dann Sinn, wenn in beiden Blöcken etwas unterschiedliches steht. Und anstatt dir etwas eigenes zusammenzuschustern, solltest du lieber die vorgegebenen Funktionen toupper() und tolower() verwenden.

Hmm... die einfachste Erklärung ist immer:
WEIL ES IN DER DOKU STEHT!

Hättest du mir einen Link oder Tip gegeben, wo sich diese Doku befindet,
wäre folgende Bitte jetzt nicht extra gepostet worden: Gib mal Link pls.

Because _strdup calls malloc to allocate storage space for the copy of strSource, it is good practice always to release this memory by calling the free routine on the pointer returned by the call to _strdup.

#include <cctype>
#include <algorithm>
#include <string>

void string_toupper(std::string& str) { std::transform(str.begin(), str.end(), str.begin(), static_cast<int (*)(int)>(std::toupper)); }
void string_tolower(std::string& str) { std::transform(str.begin(), str.end(), str.begin(), static_cast<int (*)(int)>(std::tolower)); }

soll man so nicht machen ... aber nja was solls
So ... das direkt dann in unserer neuen, tollen Funktion nutzen um jeden Buchstaben im Satzanfang groß zu machen

void string_first_upper(std::string& str)
{
    string_tolower(str);
    for (std::size_t i(0); i < str.length(); ++i)
        if ((i == 0 || str[i - 1] == ' ') && str[i] >= 'a' && str[i] <= 'z') str[i] = 'A' + str[i] - 'a';
}

oh da war ich aber plemplem Die Funktion kann natürlich anders lauten:

void string_first_upper(std::string& str)
{
    string_tolower(str);
    for (std::size_t i(0); i < str.length(); ++i)
        if (i == 0 || str[i - 1] == ' ') std::toupper(str[i]);
}

void GeileFunktion(char *String)
{
    bool bJetzt;
    String[0]=toupper(String[0]); //Sätze fangen ja groß an
    for(int i=0;i<(int)(strlen(String));i++)
    {
        if(String[i]==' ' || String[i]=='\n' || String[i]=='\t' || String[i]=='.' || String[i]=='?' || String[i]=='!')
            bJetzt=true;
        else
            bJetzt=false;
        String[i+1]= bJetzt?toupper(String[i+1]):tolower(String[i+1]);
    }
}

Ob die Wörter wirklich Substabtive sind ist eine andere Sache