#include <iostream.h>
#include <string.h>
void sort( char **p,int n)
{
    char p1[30];        //这里怎么改成指针？是不是不能改成指针
    for(int i=0;i<n-1;i++)
    {
        for(int j=i+1;j<n;j++)
        {
            if(strcmp(*(p+i),*(p+j))>0)
            {
                strcpy(p1,*(p+i));
                strcpy(*(p+i),*(p+j));
                strcpy(*(p+j),p1);
            }
        }
    }
}
int main()
{
    int n;
    cout<<"你要输入多个字符串";
    cin>>n;
    char **p= new char*[n];
    for(int i=0; i<n; i++)
    {   
        *(p+i)=new char;
        cin>>*(p+i);
    }
//    for(int j=0;j<n;j++)
//    {
//        cout<<*(p+j)<<endl;
//    }
    sort(p,n);
    cout<<"排序后为："<<endl;
    for(int k=0; k<n; k++)
    {
        cout<<*(p+k)<<endl;
    }
    return 0;
}

你的main()里是要想输入多个字符还是输入多个字符串？从sort()里看有字符串操作，但在main()里看，*(p+i)=new char;虽然能得到一个char*,但是系统只分配给它一个字节，如果你用这个char* 来存放string的话，就不可能是你想要的效果。

你应该改成p[i]=new char[128]; 首先，p是char**，它在32位系统里是一个32位指针，占4字节，所以p+1不可能是第二个char**的位置，p+2不可能是第三个char**的位置。使用数组的话，系统会跟据p的类型指向相应的单元地址。如果你要运行程序的机器是单片机，且内存较少的话，char[128]的分配量就自己看着用吧。另外提醒一下，如果你要运行程序的机器是多任务系统，特别是移动设备智能系统，使用new分配内存后，要用delete释放内存。

但是我编译器没报错而且也能正常运行