#include<iostream>
#include<string>
using namespace std;
void fun(string a)
{ int i,j,n;
  char c;
  n=sizeof(a);
  for(j=n;j>=0;j--)
  for(i=0;a[i]!='\0';i++)
     { if(a[i]<a[i+1])
      {c=a[i+1];
       a[i+1]=a[i];
       a[i]=c;
      }
  }
  cout<<a<<endl;
}
int main()
{ string a,b,c;
  cout<<"input string"<<endl;
  cin>>a>>b>>c;
  fun(a);
  fun(b);
  fun(c);
  return 0;
}

不明白为什么，总之就是不让对a[i+1]赋值

#include<iostream>
#include<string>
using namespace std;
void fun(string a)
{ int i,j,n;
  char c;
  n=sizeof(a);
  for(j=n;j>=0;j--)
  for(i=1;a[i]!='\0';i++)
     { if(a[i]<a[i-1])
      {c=a[i-1];
       a[i-1]=a[i];
       a[i]=c;
      }
  }
  cout<<a<<endl;
}
int main()
{ string a,b,c;
  cout<<"input string"<<endl;
  cin>>a>>b>>c;
  fun(a);
  fun(b);
  fun(c);
  return 0;
}
这样可以运行

明白了，楼主的那个代码会让'\0'和前一个字符交换，这是不允许的

有道理

但是输入一个字母的ASCII肯定大于‘\0'  下面的赋值就不会执行呀

你这个  挺好用的

好吧 我晕了，这好像就解释了，那个问题。

数据本身就是以ASCII码形式存储在电脑中的，直接大于零就可以了吧，好像不需要加引号和转义符