void Reverse(char *text[])
{
  unsigned short int anz = 5;  // Wie bekomm ich das dynamisch ?
  char *a = *text, *e = *text + anz, *tmp;
  while (a <= e)
  {
    *tmp = *a;
    *a = *e;
    *e = *tmp;
    a++;
    e--;
  }
}
int main(int argc, char* argv[])
{
  char *text = "Ronald";
  Reverse(&text);
  cout << text;
  return 0;
}

Ich bin noch neu im Pointerkram, leider hat sizeof text / sizeof text[0] nichts gebracht und auch strlen war hier nicht zu gebrauchen oder ich weis nicht wie wegen den unterschiedlichen typen. Bitte helft mir. Danke

1. Wenn du die Adresse text an die Funktion Reverse() uebergeben moechtest,dann brauchst du da keine Referenz mehr, d.h. du kannst das & Zeichen einfach weglassen, denn die Variable ohne den * ist schon die Adresse.

2. genuegt es, wenn du im Parameter auch nur char *text schreibst... die eckigen Klammern kannst du dir sparen.

3. Wuerde ich dafuer nicht char * benutzen sondern mir mal die STL string anschauen. ich glaube die beinhaltet schon eine Methode reverse();

hier ein kleines beispiel:

string a;
cin>>a;
a.reverse();
cout<<a;

@shadowmasta: mach aus "char *text[]" einfach "char* text". Und "text + anz" ist falsch, das muss "text + anz - 1" sein -- zumindest so wie du in der Schleife weitermachst muss es "text + anz - 1" sein, sonst würdest du das Nullbyte welches den String hinten abschliesst auch mit an den Anfang tauschen.

...
...leider hat sizeof text / sizeof text[0] nichts gebracht und auch strlen war hier nicht zu gebrauchen ...

Hi,

also das hat eigentlich nichts mit C++ zu tun, sondern ist reines C ... aber ich versuche mal zu helfen:
1.) Der "sizeof-"-Trick funktioniert nur bei echten Arrays, nicht mehr, wenn sie "zu Zeigern degeneriert sind" (wie es bei der Übergabe an Funktionen immer passiert).
2.) Wieso "ist strlen() nicht zu gebrauchen" ? Ist hier das Mittel der Wahl ... und außerdem das letzte, was Dir bleibt, wenn Du die Länge nicht explizit übergeben darfst (weswegen std::string auch besser wäre ).

Mal eine Frage vorab: Bist Du sicher, dass Du C einsetzen musst und nicht C++ verwenden darfst ? Andererseits steht da noch, dass man "2 Pointer" verwenden soll .... und dafür hätte man mit Verwendung eines std::strings irgendwie keinen Bedarf....hmmm.

Zum Rest hat der hustbaer schon Wesentliches gesagt:

hustbaer schrieb:

...
@shadowmasta: mach aus "char *text[]" einfach "char* text". Und "text + anz" ist falsch, das muss "text + anz - 1" sein -- zumindest so wie du in der Schleife weitermachst muss es "text + anz - 1" sein, sonst würdest du das Nullbyte welches den String hinten abschliesst auch mit an den Anfang tauschen.

(nur zur weiteren Erläuterung: "text + anz" zeigt auf das Byte hinter dem letzten Zeichen: text+0 -> 1. Zeichen, text+1 -> 2. Zeichen, .... )

Gruß,

Simon2.

Wieso:

char *a = *text, *e = *text + anz, *tmp;

und nicht

char *a = text, *e = text + anz, *tmp;

Er will doch damit vergleichen, ob er am Ende vom char-array angekommen ist, oder nicht? Und da will er doch den Wert von text (also die Adresse) an die pointer a und e übergeben, und nicht etwa das, was an der Stelle im Array steht.

Was habe ich hier bez. Pointer missverstanden?

@Simon2: da steht "text string", also nicht einfach string. Von daher würde ich darauf tippen, dass, wenn sie std::string in der Schule noch nicht gesehen haben, damit ein char array gemeint ist.

Ich hätte da noch eine kleine Verständnisfrage:

Wieso:

*text

...

Naja, sein Reverse bekommt ja keinen char*, sondern einen char** ... da muss man schon einmal dereferenzieren, um einen char* in die Finger zu bekommen.
Was entspricht einem "text string" besser als ein std::basic_string<char> , der std::string nunmal ist ? :p

Gruß,

Simon2.

void Reverse(char* text)
{
  unsigned short int anz = 5;  // Statisch is doch dooof :/
  char *a = &text[0], *e = &text[anz];
  char tmp;
  while (a <= e)
  {
    tmp = *a;
    *a = *e;
    *e = tmp;
    a++;
    e--;
  }
}

Aber davon mal abgesehen war die Aufgabe diese Übung zu machen und meine Frage, ob es eine Möglichkeit gibt die Länge des übergegebenen char-Pointers auszulesen. Warum das mit meinen Methoden nicht ging ist mir nun klar geworden.
Alternativ müsste ich den char bis zum Nullbyte durchgehn.
Danke an alle Erklärbären das soll C++ sein und ich übe halt mit den Aufgaben.

PS: Wenn es C++ sein soll, vergiss char-Pointer für den Moment und nimm dir einen std::string.

In dem Buch wurde nur char behandelt und damit geht ja auch alles, wenn auch umständlicher :p
Ok ich danke euch allen nun ist diese Aufgabe gelöst.

unsigned short int anz = strlen(text)-1;
char *a = text, *e = text + anz;

mfg

Weil: Die letzte Stelle eines 1 Zeichen langen Strings ist 0

seeehr wichtig finde ich noch diesen Hinweis von CStoll, den Du unbedingt beherzigen solltest:

CStoll schrieb:

Und noch eine Ergänzung: char* tmp; wird dir das Programm recht schnell abrauchen lassen (SegFault) ...

Hier nochmal der Kontext:

shadowmasta schrieb:

...

char *text = "Ronald";

Hier lässt Du text auf readonly-Speicher zeigen und hier:

shadowmasta schrieb:

...

...  char *a = *text
... *a = *e;

schreibst Du trotzdem rein !!
Das führt zu "undefinierten Ergebnissen", was üblicherweise verglichen wird mit dem Sprung ohne Fallschirm aus einem Flugzeug: Auch wenn beim erstem Mal ein zufällig vorbeifliegender Wildentenschwarm Deinen Fall abgebremst hat, würde ich es
a) nicht nocheinmal tun und
b) auf keinen Fall unter "Klappt doch - mach ich jetzt immer so !" verbuchen.

Ferner ist das hier:

CStoll schrieb:

...
- und als Zwischenvariable für den Ringtausch reicht ein normaler char völlig aus...

auch noch eine schöne Aufgabe, falls Du noch Lust auf etwas Lernen hast.

Gruß,

Simon2.

@CStoll: bezieht sich das noch auf den aktuellen Code ?

void Reverse(char* text)
{
  unsigned short int anz = strlen(text);
  char *a = text, *e = text + anz-1;
  char tmp;
  while (a <= e)
  {
    tmp = *a;
    *a = *e;
    *e = tmp;
    a++;
    e--;
  }
}

PS. So tief bin ich auch nicht in der Materie aber ich lasse mich gerne berichtigen