Inverser une chaîne en C en utilisant la boucle while

La valeur de i et j ne sont qu'un nombre qui représente l'emplacement dans la chaîne où la valeur qui vous intéresse existe.

Mais quand vous le faites l'échange réel, vous semblez avoir manqué cela, même si votre commentaire indique la bonne solution.

Donc au lieu de:

temp = s[i];
s[i] = s[j];
s[j] = temp;

Essayez: p>

temp = i;
i = j;
j = temp;

également comme mentionné dans les commentaires, si une fonction est déclarée comme renvoyant void, vous ne devriez pas renvoyer de valeur.

Votre code semble mélanger les concepts d'utilisation d'un index (comme 0 ou strlen (s) -1 ) ou l'utilisation de pointeurs. Même dans un commentaire, vous avez écrit "permutez les caractères s [i] et s [j] " mais vous avez déclaré i et j en tant que variables char * .

La deuxième erreur est que vous permutez les valeurs du pointeur, pas les caractères vers lesquels pointent les pointeurs.

Vous devez décider si vous voulez utiliser des pointeurs ou un index pour accéder aux caractères.

Solution utilisant des pointeurs:

void stringReverse(char* s){

    size_t i = 0;
    size_t j = strlen(s)-1;
    //in order to swap the characters s[i] and s[j] make a temp
    char temp;

    while(i < j){
        temp = s[i];
        s[i] = s[j];
        s[j] = temp;

        i++;
        j--;
    }
    //end of the while loop means that it reversed everything (no need for if/else)
}

Solution utilisant l'index:

void stringReverse(char* s){

    //in order to swap the characters s[i] and s[j] make a temp
    char temp;
    char* i = s;
    /* according to the standard, the behavior is undefined if the result
     * would be one before the first array element, so we cannot rely on
     * char* j = s + strlen(s) - 1;
     * to work correct. */
    char* j = s + strlen(s); 
    if(j > i) j--; // subtract only if defined by the C standard.

    while(i < j){
        temp = *i;
        *i = *j;
        *j = temp;

        i++;
        j--;
    }
    //end of the while loop means that it reversed everything (no need for if/else)
}

Le compilateur devrait avertir de certains des problèmes dans le code source d'origine si suffisamment d'avertissements sont activés. Je suggère de toujours activer autant d'avertissements que possible.

Vous le voulez. Explication dans les commentaires.

void stringReverse(char* s) {

  int i = 0;                // int instead of char*
  int j = strlen(s) - 1;    // int instead of char*
  char temp;                // char instead of char*

  while (i < j) {
    temp = s[i];            // actually exchange the two characters
    s[i] = s[j];
    s[j] = temp;

    i++;
    j--;
  }
}

Ici, vous allez sans utiliser de variable temporaire, je n'ai pas testé ce code, voici comment je l'ai fait quand j'avais rencontré une question similaire dans une interview

    void stringReverse(char* str){

        char* i = 0;
        char* j = strlen(str);


    while(i < (j/2)){
        strl[j]= str[j-i-1];
        str[i+1] = str[j];
        str[i]=str[j];
        i++;       
    }
   str[j]="\0";

    }

Essayez ceci fonctionne également

#include<stdio.h> 
#include<string.h>  
int main() 
{ 
   char str[10] = "abc"; 

   printf("The given string is =%s\n",str); 

   printf("After reversing string is =%s",strrev(str)); 

   return 0; 
}