Ошибка сегментирования

Привет всем! Хочу отсортировать в алфавитном порядке строку badc, компиляция успешна, при запуске программы Ошибка сегментирования...Вот мой код:

include<stdio.h>
#include<stdlib.h>
#define SIZE 4
char Bubble(char *s, int len);
void main()
{
int i;
char *s="badc";
Bubble(s,SIZE);
for (i=0; i<SIZE; i++) printf("%s  ", s[i]);
}
char Bubble(char *s, int len)
{
        int i,j,buf;
        for(i=1; i<len-1; i++)
        {
                for(j=0; j<len-i; j++)
                if (s[j]>s[j+1])
                {
                        buf=s[j];
                        s[j]=s[j+1];
                        s[j+1]=buf;
                }
        }
}

Сделайте так:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define SIZE 4
void Bubble(char *s, int len);
int main(){
int i;
char buf[10],s[8]="badc";
Bubble(s,SIZE);*buf=0;
for (i=0; i<SIZE; i++)sprintf(buf+strlen(buf),"%c  ", s[i]);
fprintf(stdout,"%s\n",buf);
return 0;}
void Bubble(char *s, int len)
......

Пётр.

1

printf("%s  ", s[i]);
надо поменять на
printf("%с  ", s[i]);

2

int i,j,buf;
надо поменять на
int i,j;
char buf;

Петр, нафига s[8]?

В дополнение полезная статья по сортировке:
http://www.cyberforum.ru/cpp-beginners/thread27084.html

Drayzer написал:

char *s="badc";

"badc" это константа. Константа, это неизменяемая НИКОГДА переменная.
ну а вы её меняете. (потому-что s это вовсе не строка, а адрес константной строки).

пишите так:

int main()
{
 char *cs = "badc";
 size_t len = strlen(cs);
 char *s = malloc(len + 1);
 strcpy(s, cs);
 bubble(s, len);
 for (i=0; i<len; i++)
   printf("%c  ", s[i]);
 free(s);
 return 0;
}

voral написал:

char s[SIZE+1]="badc";

Peter написал:

s[8]="badc";

facepalm.jpg

кстати, что у вас за компилятор? Что void main проглатывает?

и да, в сортировке ошибка.

drBatty написал:

facepalm.jpg

Успел.... заснять плюху :lol:

voral написал:

кстати, что у вас за компилятор?

вы лучше скажите, что У ВАС за компилятор, что s[8]="xzxz" проглатывает? Это питонотред?

drBatty написал:

"badc" это константа. Константа, это неизменяемая НИКОГДА переменная.

черт...... точно фйеспалм..... пойду ка я отдыхать..........

drBatty написал:

вы лучше скажите, что У ВАС за компилятор, что s[8]="xzxz" проглатывает? Это питонотред?

Как бы я это даже не пытался писать, не то что компилировать

Drayzer, макрос SIZE не нужен. Подумайте, как вы его зададите для строки

"jkssssssssjkjdddddddddddddddddddddddddddddddddddddddddddddddddddddddeeeeeeeeeeeeeeeeeeejjjkejejwjkjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjjdjdjdjddkkkkaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa"

voral написал:

пойду ка я отдыхать..........

бывает. Я тоже не сразу заметил.

Вариант

char s[SIZE+1]="abvf";

для учебных целей вполне годный.

voral написал:

Вариант
char s[SIZE+1]="abvf";

да? пробовали?

ага вот только что

voral написал:

для учебных целей вполне годный.

вот не надо самому себе грабли подкладывать, тем более в учебных целях.

drBatty написал:

вот не надо самому себе грабли подкладывать, тем более в учебных целях.

Ну вы взгляните в букварь. В какой последовательности обычно обучают.... Динамическое выделение памяти идет позже.

voral написал:

ага вот только что

на инициализации сработает. Теперь объясняйте ТСу разницу между инициализацией и присваиванием.