#include <iostream>

void dstrcpy(char *s1, char *s2) {

  int i = 0;
  while(*s1 != '\0') {
    *s2++ = *s1++;
  }
}

int main() {
  char s1[] = "123456789123456789123456789123456789";
  char s2[] = "cdef";

  dstrcpy(s1, s2);

  return 0;
}

Nur leider tritt bei diesem Programm ein Speicherzugriffsfehler auf, da s2[] nicht die gleiche größe hat wie s1[]. Wie schaffe ich es nun diesen Fehler zu beheben?! Es darf natürlich nur etwas in der Funktion geändert werden.

Vielen Dank schon Mal für die Hilfe

An der Größe der Strings kannst du ja nichts ändern, daher wäre eine Möglichkeit den Überlauf zu verhindern auf das Ende beider Strings zu prüfen und dann mit dem Kopieren aufzuhören.

Dann wäre die gewünschte Funktion aber nicht mehr gegeben. Problemstellung sieht ja so aus, dass s2 nur 4bit groß ist, aber nun durch einen 36bit langen string ersetzt werden soll. Das funktioniert auch soweit, das einzige Problem ist, dass dabei s1 beschädigt wird. Es wird aufgrund des Speicherüberlaufs überschrieben.

Du hast somit auch keine Möglichkeit ein größeres Array vollständig in ein kleineres zu kopieren. Das ist unmöglich. Den Überlauf verhinderst du nur damit rechtzeitig mit dem Kopieren aufzuhören.

...
int main() {
  char s1[] = "123456789123456789123456789123456789";
  char s2[sizeof(s1)] = "cdef";

...

?

Noch besser: std::string nutzen und sich mit wirklich Interessantem beschäftigen.

Gruß,

Simon2.

Nur leider tritt bei diesem Programm ein Speicherzugriffsfehler auf, da s2[] nicht die gleiche größe hat wie s1[].

Genauer:
s1 ist vom Typ char[37]
s2 ist vom Typ char[5]
Dass Du hier char[] schreiben kannst, liegt an Initialisierungs-Sonderregeln. Der Compiler vervollständigt den Typ zu char[37] bzw char[5] indem er zählt, wieviele Zeichen Deine Initialisierer haben.

Javun schrieb:

Wie schaffe ich es nun diesen Fehler zu beheben?!
Es darf natürlich nur etwas in der Funktion geändert werden.

Ganz einfach:

void dstrcpy(char *s1, char *s2)
{
}

Problem behoben.
Du hast nicht gesagt, was dstrcpy eigentlich machen soll. Da man die Größe von solchen Arrays wie s1 und s2 nicht nachträglich ändern kann, kann ich mir auch nicht zusammenreimen, was ein "korrektes" dstrcpy hier eigentlich machen soll.

kk

Es geht bei dieser Funktion einfach darum einen beliebig langen String in einen beliebigen kurzen zu kopieren.

Sofern du mit dynamischen Strings arbeitest (wie std::string) ist dies auch möglich, nicht aber mit statischen C-Arrays deren Größe du schlicht und ergreifend nicht mehr zur Laufzeit ändern kannst. Das ist auch völlig unabhängig davon wie die Aufgabenstellung ist, es ist schlicht und ergreifend nicht möglich.

Eine andere Alternative ist es, wenn diese Arrays dynamisch alloziert werden (Sprich Zeiger und new[]/delete[] bzw. falls das eine C-Frage und nicht C++ per malloc/free), und du den Speicherbereich des kleineren löschst und entsprechend neu allozierst.

Javun schrieb:

An der main() soll daher auch nichts geändert werden.

Dann ist die Lösung nicht möglich. Ich gehe dann eher davon aus das der Inhalt des kleinen Arrays in das große Array übertragen werden soll. Oder: das nur der Teil kopiert werden soll, der in das Zielarray wirklich hinein passt.

dann gibts auch ne einfache antwort ;o)

bb

Möglich ist dies aufjedenfall

nö. nicht mit den von dir dargestellten rahmenbedingungen.

die einfachste lösung heisst jedenfalls "std::string verwenden".

ru,
cirion

Kopiere solang bis entweder s1 oder s2 "zu Ende" ist. D.h. für Dein Beispiel das es nur die ersten 4 Zeichen von s1 nach s2 kopiert und gut ist. Funktioniert btw. auch nur solang die Strings null-terminiert sind (bei konstant-langen Strings wie in Deinem Beispiel durch den Compiler gegeben, bei dynamisch alloziierten Strings muss man dies selber tun).

Gut Schuß
VuuRWerK

Es geht bei dieser Funktion einfach darum einen beliebig langen String in einen beliebigen kurzen zu kopieren. ...

garantiert nicht.

Machen wir das einfach mal ganz ohne Programmiersprache:
"Schaufele einen Kubikmeter Sand in eine 1-Literflasche" ist das Äquivalent zu Deiner Aufgabe.

Ich vermute, dass
a) die Lösung von VuuRWerK das Gewünschte ist oder
b) die Anlage eines neuen Arrays (z.B. mit new )
gemeint ist.
So wie Du die Aufgabe beschrieben hast, ist sie nicht lösbar.
Vermutlich hast Du die Vorgabe "... mit beliebig langen Strings ..." mißverstanden (gemeint: "Die Funktion soll keine feste Stringlänge voraussetzen"; verstanden: "Die Funktion soll mit jeder Stringlänge alles kopieren").

Gruß,

Simon2.

Es geht bei dieser Funktion einfach darum einen beliebig langen String in einen beliebigen kurzen zu kopieren.

Bestimmt nicht.

Sicher, dass Du Quelle und Ziel nicht verwechselt hast?

Hat das d in dstrcpy noch eine besondere Bedeutung?

Bei der Standard-C-String-Funktion (strcpy) kommt der Ziel-Parameter auch vor dem Quell-Parameter.

kk