String bearbeiten

DrakoXP

ich glaub zwar nicht, dass das der Absturzgrund ist,
aber das:

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.

sjBlack

Vielen Dank für deinen Performance-Tipp. Auf sowas leg ich besonders Wert
Die Variable temp war nach einer Überarbeitung nicht mehr nötig, ich habe
lediglich vergessen, sie wieder zu entfernen.

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

Jochen Kalmbach

Ach so...
Dir sollte bewusst sein, dass Compiler i.d.R. String-Literale in einen "Read-Onyl" Bereich legen...

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).

sjBlack

Es funkt. Also lag es daran, dass die Funktion als Parameter einen Pointer
zu einer char verlangt, aber es wird ein constantes string array der
Größe 11 übergeben (10 + \0). Und strdup legt einen string in den heap
und gibt den pointer zurück?

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;
}

Jochen Kalmbach

Wenn Du was mit "strdup" allokiert hast, musst Du es mit "free" freigeben!

sjBlack

Wieso'n das?

Jochen Kalmbach

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

Danke erst mal für die genazen Antworten ich hab mir jetzt meine eigene Funktion geschrieben.

#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;
}

CStoll

String Bearbeiten schrieb:

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.

Ja vorher stand das noch was anderes drinn^^ und ich habs vergessen rauszunehmen

sjBlack

Jochen Kalmbach schrieb:

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.

Jochen Kalmbach

http://msdn2.microsoft.com/en-us/library/y471khhc

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.

*D*Evil

Generell sollte man sich merken bei der Speicherallokation niemals C und C++ zu mischen.

#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]);
}

Meine Version

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