STRNODE *str_con(STRNODE *s1,STRNODE *s2)   
{ //将字符串S2连接到S1串的后面，两个串合并成一个串，放到S1串中/
        STRNODE *p,*q,*s;
        int i;
    p=s1->next;
    q=s2->next;
   
    do{
        p=p->next;
    }while(p->next==NULL);
//    p=q;

    do{
   s=(STRNODE *)malloc(sizeof(STRNODE));
     s->ch=q->ch;
     p->next=s;
     p=s;
//     p->next=NULL;
     q=q->next;
    }while(q->next=NULL);

    return p;
我实在分析不通我的问题在哪里，我在草稿纸上验证了无数遍 ，都没发现问题在哪里，但是在机器上只返回原始的P ,求指点

需要重新开辟空间，复制s2中的字符，还仅仅是连接两个串？

STRNODE *str_con(STRNODE *s1,STRNODE *s2)   
{
        STRNODE *p,*q,*s;
        int i;
    p=s1->next;
    q=s2->next;
   
    do{
        p=p->next;
    }while(p->next==NULL);

    do{
    s=(STRNODE *)malloc(sizeof(STRNODE));
     s->ch=q->ch;
     p->next=s;
     p=s;
     q=q->next;
    }while(q!=NULL); //你的条件显然错了，不该是赋值
    p->next=NULL;  //串尾加上结束标致
    return s1;        //注意到p在变，返回应该是s1
}

[ 本帖最后由 kwxx 于 2014-4-25 23:34 编辑 ]

我的意思是重新为p 开辟节点，然后朝里面塞入q的值

楼主试试上边改的。

试过了  ，    STRNODE *p,*q,*s;
//    int i;
    p=s1->next;
    q=s2->next;
   
    while(p->next!=NULL)
    {
        p=p->next;
    }
//    p=q;

    do{
   s=(STRNODE *)malloc(sizeof(STRNODE));
     s->ch=q->ch;
     p->next=s;
     p=s;
//     p->next=NULL;
     q=q->next;
    }while(p!=NULL);
p->next=NULL;
    return s1;

其中，感觉应该是while(p!=NULL); 还是没用，还只是返回原始的P的串。
while(p==NULL)我也试过，一样的结果 ，我感觉
do{
   s=(STRNODE *)malloc(sizeof(STRNODE));
     s->ch=q->ch;
     p->next=s;
     p=s;
//     p->next=NULL;
     q=q->next;
    }while(p!=NULL);
这一段代码根本没执行。

不好意思，刚才蒙了。循环条件应该是 q!=NULL  你别改成p!

不管是p还是q都对最后的结果没有任何影响，返回的结果依旧是没有连接成功的P ,我有点无奈了

有点看不懂，你这是C++ 还是C呀

你单独输出一下，你的s1,和s2,看它们是否有误？