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 ,求指点

我的意思是重新为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);
这一段代码根本没执行。

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