Viết lại một số hàm cơ bản về xâu ký tự lỗi Null pointer assignment?

[khucnam]

Mình viết lại một số hàm cơ bản như chèn, xóa trong xâu ký tự.
+ Hàm int POS(char *st1, char *st2) trả về vị trí xuất hiện của st2 trong st1, nếu không có thì trả về -1
+ Hàm char *copy(char *st, int pos, int n) để trích một chuỗi con có n kí tự của chuỗi st bắt đầu từ vị trí pos
+ Hàm char *insert(char *s, char *st, int pó) để chèn chuỗi s vào chuỗi st tại vị trí pos
+ Hàm char *del(char *st, int pos, int n) để xóa n ký tự trong chuỗi s bắt đầu tại ví trí pos

Dưới đây là code của mình (mình chạy trên Turbo C). Nhờ các bạn xem giúp vì sao
+ hàm copy bị chèn trước một vài ký tự gì đó trước chuỗi kết quả
+ hàm insert của mình ko bị lỗi khi dịch nhưng khi chạy lại cho ra kết quả Null pointer assignment

#include <stdio.h>#include <conio.h>#include <iostream.h>#include <string.h> int POS(char *st1, char *st2){     if (strstr(st1,st2)==NULL) return -1;     else    {        char *s=strstr(st1,st2);        return strlen(st1)-strlen(s);    } } char *copy(char *st, int pos, int n){    char *temp;    char *result;    for(int i=pos; i<pos+n; i++)    {        *temp=st[i];        result = strcat(result,temp);    }    return result;} char *insert(char *s, char *st, int pos){     int len = strlen(s);    int len1 = strlen(st);    int i, j=0;    char *temp;      if ((pos<0) || (pos>len1))        {            cout<<"\n Vi tri chen khong phu hop ";            return NULL;        }    else       if (pos==len1)        {         cout<<"\n Chen cuoi ";         for(i=len1;i<len1+len;i++)            st[i] = s[j++];         for(i=0;i<len+len1;i++) cout<<st[i];         st[i] = '\0';         }        else       {         cout<<"\n Chen vao giua ";        for(i=len1+len-1; i>=pos+len;i--)        st[i] =st[i-len];         j=0;        for(i=pos; i<pos+len;i++)        st[i] = s[j++];         st[len1+len]='\0';       }    return st;} char *del(char *st, int pos, int n){    int len = strlen(st);    char *temp;     if ((pos<0) || (pos>=len))    {        cout<<"\nVi tri xoa khong hop le ";        return NULL;    }    else    if (n<=0)    {        cout<<"\nSo ki tu xoa khong hop le ";        return NULL;    }    else      {        int i;        for(i=0;i<pos;i++)            temp[i] = st[i];        for(i=pos;i<=len-n-1;i++)            temp[i] = st[i+n];        temp[i]='\0';      }      return temp; } main(){    clrscr();    /*    // Minh hoa ham POS    char *s1,*s2;    cout<<"Nhap s1 \n";    gets(s1);    cout<<"Nhap s2 \n";    gets(s2);    cout<<"\n Vi tri la :"<<POS(s1,s2);   */       //Minh hoa ham copy    char *chuoicopy;    int vitritrich, sokitutrich;    cout<<"Nhap vao chuoi can trich ";    gets(chuoicopy);    cout<<"\n Nhap vao so ki tu trich ";    cin>>sokitutrich;    cout<<"\n Nhap vao vi tri trich ";    cin>>vitritrich;     cout<<"\n"<<copy(chuoicopy,vitritrich,sokitutrich);      /*    // Minh hoa ham INSERT    char *chen, *chuoigoc;    int vitri;     cout<<"\nNhap chuoi goc : ";    gets(chuoigoc);    cout<<"\nNhap chuoi muon chen ";    gets(chen);     cout<<"\nNhap vi tri can chen ";    cin>>vitri;     cout<<"\nChuoi sau khi chen la: "<<insert(chen,chuoigoc,vitri); */      // minh hoa ham DEL    /*    char *xauxoa, *ketqua;    int vitrixoa, sokituxoa;    cout<<"Nhap vao xau can xoa :\n";    gets(xauxoa);     cout<<"Nhap vao vi tri xoa : \n";    cin>>vitrixoa;    cout<<"Nhap vao so ki tu can xoa :\n";    cin>>sokituxoa;     ketqua = del(xauxoa,vitrixoa,sokituxoa);    if (ketqua!=NULL) cout<<"\Sau khi da xoa :"<<ketqua;    */     getch();}

[fbchicken]

Chưa đọc code nhưng lướt qua thấy thấy 2 vấn đề:
- Bạn chưa cấp phát cho chuỗi đầu vào (chuoicopy)
- Tương tự như vậy, hàm insert bạn cũng phải cấp phát (động) 1 chuỗi kết quả để trả về. Vì không thể đảm bảo chuỗi đầu vào s hoặc st đã được cấp phát đủ bộ nhớ để chứa chuỗi kết quả.

[khucnam]

Bạn sửa giùm mình một chút được ko? Như thế nào là cấp phát động?

[bachhaiduong]

Ở đây nè :P
Cấp phát, giải phóng vùng nhớ trong C++.