Klausuraufgabe

Danke für die Hilfe.
Ich hab jetzt strlen ersetzt:

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
int strlaenge=0;
    while(ziel[strlaenge]!='\0')
    {
        strlaenge++;
    }

    if(ziel[strlaenge]!=''){
        ziel[strlaenge+1]='';
        ziel[strlaenge+2]=trenn;
    }

    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge+2+i]=quelle[i];
    }

    return ziel;

}

Badestrand

Leerzeichen ist ' ' und nicht '' . Am Schluss sollte der String auch wieder ein '\0' hinten haben. Und du hast noch einen weiteren kleinen Fehler drin.

_matze

Nimm [cpp] statt [code], dann sieht das auch schöner aus.

EDIT: Du berücksichtigst den Fall 'kein Leerzeichen am Ende des Eingabestrings' nicht. Da musst du auch das Trennzeichen anfügen. Und wenn du ein Leerzeichen anhängst, ändern sich die Länge, wie gesagt...

Ok,muss ich bei der for Schleife +3 anstatt +2 nehmen? Ansonsten würde im ersten Durchgang ja mein Trennzeichen überschrieben werden:

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
int strlaenge=0;
int komplettStrLaenge=0;
    while(ziel[strlaenge]!='\0')
    {
        strlaenge++;
    }

    if(ziel[strlaenge]!=' '){
        ziel[strlaenge+1]=' ';
        ziel[strlaenge+2]=trenn;
    }

    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge+3+i]=quelle[i];
    }

    komplettStrLaenge=strlaenge+3+i;
    ziel[komplettStrLaenge+1]='\0';

    return ziel;

}

Sorry für Doppelpost, werd mich bei der nächsten frage wohl vorher registrieren um editieren zu können.

Ok,muss ich bei der for Schleife +3 anstatt +2 nehmen? Ansonsten würde im ersten Durchgang ja mein Trennzeichen überschrieben werden:

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
int strlaenge=0;
int komplettStrLaenge=0;
    while(ziel[strlaenge]!='\0')
    {
        strlaenge++;
    }

    if(ziel[strlaenge]!=' '){
        ziel[strlaenge+1]=' ';
        ziel[strlaenge+2]=trenn;
    }

    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge+3+i]=quelle[i];
    }

    komplettStrLaenge=strlaenge+3+i;
    ziel[komplettStrLaenge+1]='\0';

    return ziel;

}

_matze

Du musst erst ermitteln, ob am Ende ein Leerzeichen ist. Abhängig davon hängst du eins an oder nicht. Hierbei ermittlest du die neue Länge (ist ja eventuell um eins größer geworden). Und das Trennzeichen musst du in jedem Fall anhängen, auch wenn im Ursprungsstring kein Leerzeichen am Ende war (denn das machst du im Moment nicht). Die Position für das Trennzeichen hängt von der neuen Länge ab (wie auch die Folgeschritte).

Hab nochmal deinen Vorschlag mit eingebracht:

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
int strlaenge=0;

    while(ziel[strlaenge]!='\0')
    {
        strlaenge++;
    }

    if(ziel[strlaenge]!=' '){
        ziel[strlaenge]=' ';
        strlaenge++;
    }
    ziel[strlaenge]=trenn;   
    strlaenge++;

    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge]=quelle[i];
    strlaenge++;
    }

    ziel[strlaenge+1]='\0';

    return ziel;

}

_matze

Und, klappt jetzt alles?

Martin Richter

_matze schrieb:

Und, klappt jetzt alles?

Nö! Sieht man doch...

_matze

Martin Richter schrieb:

_matze schrieb:

Und, klappt jetzt alles?

Nö! Sieht man doch...

Hehe, ich hatte keinen Bock, mir das alles wieder anzusehen. Er sollte eigentlich mal so vorgehen, dass er sein Programm ausprobiert und dann die Fehler postet, anstatt den Code zur allgemeinen Überprüfung bereitzustellen...

Ok, habs nun mal in XCode probiert. Fehler gibt er mir keine aus, allerdings auch keine Ausgabe:

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

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
int strlaenge=0;

    while(ziel[strlaenge]!='\0')
    {
        strlaenge++;
    }

    if(ziel[strlaenge]!=' '){
        ziel[strlaenge]=' ';
        strlaenge++;
    }
    ziel[strlaenge]=trenn;  
    strlaenge++;

    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge]=quelle[i];
    strlaenge++;
    }

    ziel[strlaenge+1]='\0';

    return ziel;

} 

int main () {
cout << strcatTrenn("test","string",'*');
    return 0;
}

_matze

So darfst du das sowieso nicht aufrufen! Du musst schon ein char-Array (kein Literal) mit genügend Platz definieren. Zum Beispiel so:

char str[256];
strcpy(str,"test");

Ich bin zu blöd...

int main () {
char str[256];
str=strcatTrenn("test ","string",'-');
cout << str;
    return 0;
}

führt zu
incompatible types .. of char* to char[256]

Martin Richter

Dein Code hängt immer ein Leerzeichen an! Auch wenn bereits eines am Ende steht!

Echt? Dieser Code hängt doch nur eins an wenn am Ende kein Leerzeichen steht:

if(ziel[strlaenge]!=' '){
        ziel[strlaenge]=' ';
        strlaenge++;
    }

_matze

c++-user schrieb:

int main () {
char str[256];
str=strcatTrenn("test ","string",'-');
cout << str;
    return 0;
}

Du sollst ein char-Array anlegen (hast du) und dann den String deiner Wahl hineinkopieren (hast du nicht). Ich habe es doch schon vorgemacht, nimm strcpy. Und dann übergibst du str an deine Funktion...

_matze

c++-user schrieb:

Echt? Dieser Code hängt doch nur eins an wenn am Ende kein Leerzeichen steht:
if(ziel[strlaenge]!=' '){
        ziel[strlaenge]=' ';
        strlaenge++;
    }

Sorry, aber hast du deinen Code auch mal richtig durchdacht? Wenn dein String "H " ('H', ' ' und '\0') ist, dann ist strlaenge=2. Du vergleichst hier also ziel[2]==' '. Und was ist ziel[2], hm? Kleiner (bzw. riesengroßer) Tipp: der Index fängt bei 0 an.

Das Problem ist ja das ich keine Bibliothelsfunktionen wie z.b. strcpy verwenden darf.

Zu deiner zweiten Anmerkung:
strlaenge wird ja in einer while Schleife ermittelt und wenn ich das mit deinem Beispiel "H " durchspiele müsste als Wert 1 rauskommen.Und ziel[2]=' ' ist ja eine Zuweisung und kein Vergleich.

Oder habe ich dich falsch verstanden?

Danke

_matze

c++-user schrieb:

Das Problem ist ja das ich keine Bibliothelsfunktionen wie z.b. strcpy verwenden darf.

Du darfst strcpy nicht in deiner Funktion benutzen, aber außerhalb ist das schon in Ordnung (es geht ja nur darum, eine Umgebung zu schaffen, in der du deine Funktion ausführen und testen kannst).

c++-user schrieb:

strlaenge wird ja in einer while Schleife ermittelt und wenn ich das mit deinem Beispiel "H " durchspiele müsste als Wert 1 rauskommen.Und ziel[2]=' ' ist ja eine Zuweisung und kein Vergleich.

Da hast du dich wohl verzählt.

Durchgang 1: strlaenge ist 0 und du zeigst auf 'H'.
Durchgang 2: strlaenge ist 1 und du zeigst auf ' '.
Durchgang 3: strlaenge ist 2 und du zeigst auf '\0', die Schleife wird abgebrochen.

strlaenge ist also 2. Und dann vergleichst du:

if(ziel[strlaenge]!=' '){    //ziel[2] ist '\0' und ungleich ' ', also ist der Vergleich war
  ziel[strlaenge]=' ';       //du überschreibst '\0' (ziel[2]) mit ' '
  strlaenge++;
}

Alles klar?

Ich versteh deine Punkte, kann das aber nicht wirklich in neuen Code umsetzten