Nybörjarfråga C

Man kan också misstänka att kompilatorn ogillar att "kaffekopp" är konstant.

Det där funkar inte, du måste reversa en konstant sträng till ett nytt minnesområde, alt. kopiera den dit först.

const char* src = "kaffekopp"
char dst[512];
StringReverse(dst, src, 512); // tar stränglängd också så det inte overflowar.

eller

const char* src = "kaffekopp"
char dst[512];
strncpy(dst, src, 512);
StringReverse(dst);

Anledningen till detta är att kompilatorn lägger konstanter så som strängar i ett segment som är skrivskyddat. Det du sedan tillderar src är INTE strängen utan en pekare det, strängen i sig är fortfarande skrivskyddad.

Tog bort mitt inlägg som innehöll felaktiga råd. Lyssna jdv på istället.

Gör det verkligen det? Som det ser ut nu borde väl även den avslutande nollan behandlas av omkastningsfunktionen och följaktligen placeras först?

int len = strnLen(str) -1;

borde väl räcka?

Om man har CString:ar och vill skriva ännu mindre borde det väl räcka med att anropa

    csKaffekopp.MakeReverse();

Klart att dessa varianter är enklare att skriva och ger lite renare kod, men man lär sig inte så mycket av det (annat än att slå upp vilka funktioner som finns i standardbiblioteket). Och nu var det ju faktiskt C som efterfrågades.

Det där skriver dock bara ut textsträngen baklänges.

void StringReverse(char *Str){

  if(*Str){
    StringReverse(str+1);
      putchar(str);
  }
}

Ummm... snyggt? Absolut inte med tanke på att rekursionen troligen slår igenom stacken för strängar som är, säg, en miljon tecken långa (eller något i den stilen).

Sen skriver den som sagt bara ut strängen; vilket troligtvis kommer vara ett problem mycket oftare. Det är inte särskilt ofta man vill reversera strängar med flera miljoner tecken (även om det kan hända). En uppenbar lösning är ju någon form av loop:

function reverse(char* in, char* out){
  int l = strlen(in);
  for(int i=0;i<l;i++){
    out[i] = in[l-i-1];
  }
}