Klausuraufgabe

_matze

c++-user schrieb:

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

1. strcpy kannst du so (in deiner main-Funktion) benutzen, wie von mir im Beispiel gezeigt.

2. Beim Anfügen eines Leerzeichens (und auch bei den folgenden Schritten, wie z.B. Anhängen des Trennzeichens) nimmst du nicht strlaenge als Array-Index, sondern strlaenge-1. Eine Ausnahme sollte hier strlange==0 sein, aber ignoriere das erst mal.

Jetzt solltest du aber weiterkommen, oder? Schließlich habe ich mit dem Zaunpfahl, ja sogar mit dem ganzen Zaun gewunken!

Hab mal wieder angepasst, aus irgend einen Grund hängt der in folgender Zeile:

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

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

    if(ziel[strlaenge-1]!=' '){
        ziel[strlaenge-1]=' ';
        strlaenge++;
    }
    ziel[strlaenge-1]=trenn;               //Mit dieser Zeile hängt der Compiler
    strlaenge++;
cout<<strlaenge;
    for(int i=0;quelle[i]=='\0';i++){
    ziel[strlaenge-1]=quelle[i];
    strlaenge++;
    }
	 ziel[strlaenge]='\0';

    return ziel;

}

_matze

'Der Compiler hängt' ist keine Fehlerbeschreibung!

Also poste den Fehlertext. Übrigens sieht die Zeile unverdächtig aus. Hast du vielleicht eher einen Laufzeitfehler statt eines Compiler-Fehlers?

EDIT: Und arbeite doch mal an deiner Einrückung, die ist echt grauenhaft...
http://de.wikipedia.org/wiki/Einrückungsstil

EDIT2: Und übrigens ist die Bedingung deiner for-Schleife ziemlich sinnfrei. Schau dir doch bitte mal an, wie eine for-Schleife funktioniert.

könnte so gehen (ungetestet)

char *my_special_strcat (char *d, char *s, char c)
{
   char *ret = d;      // rückgabewert speichern
   while (*d)          // ende finden
      d++;
   if (d[-1] != ' ')   // wenn nicht space -> space
      *d++ = ' ';
   *d++ = c;           // trennzeichen dran
   while (*s)          // den rest anhaengen
      *d++ = *s++;
   return ret;         // altes 'd' zurückgeben
}

_matze

Super fricky, so lernt er bestimmt was dabei...

_matze schrieb:

Super fricky, so lernt er bestimmt was dabei...

keine angst, mein vorschlag tut vielleicht nicht genau das, was er will. und wenn er sich damit auseinandersetzt, lernt er nebenbei doch was.

volkard

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

Bin nicht sicher, daß Du da triffst. Vielleicht eher

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

ziel[strlaenge-1]=trenn;               //Mit dieser Zeile hängt der Compiler
    strlaenge++;
cout<<strlaenge;

Das cout<< da geht gar nicht. Es will ausgeben bis zum '\0', aber Du hast noch keins geschrieben, um den String abzuschließen.

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

Solange quelle[i] gleich '\0' ist?
Nee, grad falschrum. In C++ gibt es keine Abbruchbedingungen, sondern nur Laufbedingungen.

_matze

;fricky schrieb:

_matze schrieb:

Super fricky, so lernt er bestimmt was dabei...

keine angst, mein vorschlag tut vielleicht nicht genau das, was er will. und wenn er sich damit auseinandersetzt, lernt er nebenbei doch was.

Na ich bezweifle, dass er das Zeiger-inkrementieren überhaupt kennt (und somit versteht). Er ist sicher besser mit seiner Zähler-Variante bedient. Aber einen (für ihn fiesen) Stolperstein hast du wirklich dringelassen.

volkard

_matze schrieb:

Aber einen (für ihn fiesen) Stolperstein hast du wirklich dringelassen.

Der kam sicher beim Zurückfrickeln von

while (*d++ = *s++);// den rest anhaengen

_matze schrieb:

Na ich bezweifle, dass er das Zeiger-inkrementieren überhaupt kennt (und somit versteht). Er ist sicher besser mit seiner Zähler-Variante bedient.

ach mist. egal, kann ja nicht verkehrt sein, sich sowas auch mal anzugucken. eine musterlösung für seine hausaufgabe wollt ich nun auch garnicht posten.

_matze schrieb:

Aber einen (für ihn fiesen) Stolperstein hast du wirklich dringelassen.

ja, da fehlt noch was.

Das Problem ist er gibt mir nicht wirklich einen Fehler im Code aus. Der Code von ;fricky verhält sich allerdings genau wie meiner, es kommt also auch zu keiner Ausgabe.

Was der Compiler mir anzeigt ist:
The Debugger has exited due to signal 10 (SIGBUS).The Debugger has exited due to signal 10 (SIGBUS).

c++-user schrieb:

Das Problem ist er gibt mir nicht wirklich einen Fehler im Code aus. Der Code von ;fricky verhält sich allerdings genau wie meiner, es kommt also auch zu keiner Ausgabe.

mach mal in meinen code vor das 'return ret' ein '*d = 0' oder bau die zeile von volkard ein. und falls es dann gehen sollte, denk dran: nicht als hausaufgabe abgeben bevor du nicht verstanden hast, was der code macht.

_matze

1. Lies dir doch mal alle Beiträge seit gestern ganz genau durch, da findest du schon mal einige Hinweise.

2. Nutze den Debugger! Der bietet dir so tolle Sachen wie Breakpoints zu setzen, schrittweise durch das Programm zu gehen, Variableninhalte auszugeben usw. So kommt man normalerweise ganz schnell auf den Fehler.

3. Poste danach noch mal deinen aktuellen Code plus Fehlerbeschreibung (wenn noch Fehler da sind).

Martin Richter

Ich sehe noch ein gravierendes problem in ;frickys Code.
Oder sagen wir anders. Es gibt zwei Probleme.
1. Problem betrifft Zeile 6
2. Problem betrifft Zeile 9+10

_matze

Zeile 6 ist klar, aber in Zeile 9-10 sehe ich kein Problem...

EDIT: Du meinst nicht die fehlende, terminierende Null, oder?

Martin Richter

_matze schrieb:

EDIT: Du meinst nicht die fehlende, terminierende Null, oder?

Ja!

Eine grundlegende Frage noch, dann sollte es eigentlich klappen:
Ich rufe die Funktion auf:

strcpy(str,strcatTrenn("h","string",'*'));

Nun soll zu Testzwecken mein "Ziel" einfach einen anderen Wert bekommen und zurückgeben:

char *strcatTrenn(char *ziel,const char*quelle,char trenn){
    ziel[0]='D';
    return ziel;

Dieses führt wieder zum Fehler:

The Debugger has exited due to signal 10 (SIGBUS).The Debugger has exited due to signal 10 (SIGBUS).

_matze

c++-user schrieb:

Ich rufe die Funktion auf:
strcpy(str,strcatTrenn("h","string",'*'));

Wer hat dir denn gesagt, dass du das so machen sollst??

Nochmal: du willst etwas an einen String anhängen. Daher muss auch genügend Platz für diesen String bereitstehen. Du kannst also nicht einfach "h" (Stringliteral mit 2 Zeichen Länge) an deine Funktion übergeben. Du musst den String vorher erzeugen und dabei genug Speicher reservieren (ansonsten schreibst du ins Nirwana). Das machst du am einfachsten, indem du dich an mein strcpy-Beispiel auf Seite 2 hältst (ich verstehe nicht, warum du das ignorierst). Dein Code hier ist vollkommen sinnlos und zeigt, dass du leider nicht verstehst, was du da machst.

Kleiner Tipp: du brauchst die Rückgabe deiner Funktion nicht zu verwenden! Mach dich mal schlau, was eine Parameterübergabe per Wert (by-value) und per Referenz (by-reference) bedeutet. Wikipedia ist eine Möglichkeit.

Vielen Dank für die Hilfe, habs jetzt hinbekommen. Für die Klausur hätte mein bereits geposteter Kram sicher ein paar Punkte gebracht, wird ja alles schriftlich gemacht, hätt also meine Funktion nicht aufrufen müssen.

Danke nochmal

#include <iostream>
using namespace std;

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

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

	while(ziel[strlaenge-1]==' '){
        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]='\0';	

    return ziel;

} 

int main () {
char str[256];
strcpy(str,"h");
cout<<strcatTrenn(str,"string",'*');
return 0;
}

Martin Richter

Dein Code ist falsch wenn, der Eingabestring leer ist!

Bufferunderflow!