Moving repeated characters in string to the end [closed]

Question

Closed. This question does not meet Stack Overflow guidelines. It is not currently accepting answers.

---

Closed 8 years ago.

• This question appears to be off-topic because it lacks sufficient information to diagnose the problem. Describe your problem in more detail or include a minimal example in the question itself.
• Questions asking for code must demonstrate a minimal understanding of the problem being solved. Include attempted solutions, why they didn't work, and the expected results. See also: Stack Overflow question checklist

Improve this question

I tried to move the repeated characters in a string to its end positions but my code not working for more than two repeated characters. I tried to solve it but not getting it. Here is my code

main () {
char string[100];
char *s, *p;
char c;
scanf("%s", string);
s = string;
c = *s++;
while (*s){
    if(*s == c){
        for(p = s; *p ; p++)
            *p = *(p + 1);
        *--p = c;
    }
    c = *s;
    s++;
}
printf ( "%s\n", string);
}

Answer 1

Hope you like the test code

#include <string.h>
#include <stdio.h>


void rep2end(char *string) {
char *s, *p, *e, *stop;
char c;
s = string;
e = s+strlen(string)-1; /* find end of string */
stop = e;               /* place to stop processing */
while (stop > s){        
    c = *s;             /* char to look for */
    while(*(s+1) == c){ /* repeated char */
        for(p = s+1; *p ; p++){ /* shuffle left to overwrite current pos *s */
            *(p-1) = *p;
        }
        *e = c; /* set end char to be the repeat we just found */
        stop--; /* bump the stop position left to prevent reprocessing */
    }
    s++;
    }
}


main () {
char *in[]={"aabbccefghi", "uglyfruit", "highbbbbchair"};
char *out[]={"abcefghiabc", "uglyfruit", "highbchairbbb"};
char string[100];
int i;

for (i=0; i<3; i++) {
strcpy(string, in[i]);
rep2end(string);
if (!strcmp(string,out[i])) { 
   printf("ok\n");
   }else {
   printf("fail %s should be %s\n", string, out[i]);
   }


}
return 0;
}