lỗi Null pointer assignmemt

[phthinh]

#include<stdio.h>#include<conio.h>#include<iostream.h>#include<string.h>typedef char *string;char *chinhstr(string ch)  {   int i;   while(!(((ch[0] >= 'a')&&(ch[0] <= 'z'))||((ch[0] >= 'A')&&(ch[0] <= 'Z'))))     {    for(i=0;i<strlen(ch);i++)      ch[i]=ch[i+1];     }   while(ch[strlen(ch)-1] == ' ')     {    ch[strlen(ch)-1]='\0';     }   return ch;  }int findstr(string cha,string chb,int vitri)  {    int flag,poss;    flag=0;    poss=-1;    for(int i=vitri;i<strlen(cha);i++)       {     if(cha[i]==chb[0])       {         poss = i;         for(int j=1;j<(strlen(chb));j++)        if(cha[i+j]!=chb[j])           {             poss=-1;             flag+=1;             }           else            {             flag+=0;            }         if(flag==0)          {           poss=poss;           break;          }       }    }    return poss;  }char *deletestr(string chs,string cha,int solan,int vitri)  {    int i;    while(findstr(chs,cha,vitri)!=-1)      {     for(i=findstr(chs,cha,vitri);i<strlen(chs)-strlen(cha)-1;i++)        chs[i]=chs[i+strlen(cha)+1];     chs[strlen(chs)-strlen(cha)-1]='\0';     if(solan==1)       break;       }    chinhstr(chs);    return chs;  }char *cutbefore(string chs)  {    int i;    string cha;    for(i=0;i<strlen(chs);i++)      {    if(chs[i]==' ')      break;     else cha[i]=chs[i];      }    strcpy(chs,cha);    chs[i]='\0';    chinhstr(chs);    return chs;  }  void main(){  string s,a;  cout<<"cho chuooi vao"<<endl;  gets(s);  strcpy(a,s);  cout<<"\n xau vao :"<<a<<endl;  cutbefore(a);  cout<<"\n vi tri chuoi hello trong s"<<findstr(s,"hello",0)<<endl;  cout<<a<<endl;  cout<<strlen(a)<<endl;  deletestr(s,a,1,0);  cout<<"chuoi sau khi delete :"<<s<<endl;  getch();}

-------------------------------------------------------------------------------
đến khi run thì nó vẫn cho kết quả đúng ,nhưng kết thúc quá trình run thì nó báo Null pointer assignmemt và thoát khỏi DOS.
Có ai biết chỉ dùm lỗi và cách giải quyết nó.
thanks.!

Vui lòng để code vào trong tag code. Đọc Nội quy để biết thêm chi tiết

[rox_rook]

Đọc sơ vào thì tình hình là

typedef char *string;

......

void main(){  string s,a;  cout<<"cho chuooi vao"<<endl;  gets(s);...

char* có dữ liệu gì ở đây không vậy cậu ?
Và 1 câu hỏi nữa là cậu dùng C++ thì tui nghĩ cậu nên dùng #include <string>.
Còn sai chỗ nào nữa thì tui chưa có thời gian coi hết. Cậu coi lại chỗ đó đã !
- Với lại tui nghĩ cậu nên biết poor-performance của gets(), dùng cin.getline().
- Pass đối vào hàm nếu không có ý định modify, thêm keyword *const* vào.
- Trong C++ không có headfile .h.
chỗ này :

 while (ch[strlen(ch)-1] == ' ')

Trong hàm cậu dùng strlen( ) rất nhiều, vậy cứ mỗi lần xài là mỗi lần compiler phải tính lại giá trị này à ? Tại sao không dùng 1 biến giữ lại giá trị đó ?

[rox_rook]

Code này lỗi đầu tiên là con trỏ chưa hề được cấp phát mà lấy dùng như không !
Còn cái hàm này :

char *chinhstr(string ch)  {   int i;   while(!(((ch[0] >= 'a')&&(ch[0] <= 'z'))||((ch[0] >= 'A')&&(ch[0] <= 'Z'))))     {    for(i=0;i<strlen(ch);i++)      ch[i]=ch[i+1];     }   while(ch[strlen(ch)-1] == ' ')     {    ch[strlen(ch)-1]='\0';     }   return ch;  }

Tui chẳng hiểu cậu muốn edit gì ở đây ? Cả dòng này tui đoán chắc là muốn nó phải là chữ ?

 while(!(((ch[0] >= 'a')&&(ch[0] <= 'z'))||((ch[0] >= 'A')&&(ch[0] <= 'Z'))))

Vậy sao không xài isalpha() ?
Code viết quá tối nghĩa, hàm dưới thì cậu lại dùng :

 flag+=0;

Tại sao phải cộng += 0 ?
Hàm đó là tìm vị trí chuỗi con trong vị trí chuỗi cha, tại sao nó có 1 biến vitri = 0 vậy ?

cout<<"\n vi tri chuoi hello trong s"<<findstr(s,"hello",0)<<endl;

0 mang ý nghĩa gì chỗ này ?
Hàm xóa chuỗi thì nên có check range vị trí cần xóa, giả nếu vị trí quá độ dài của chuỗi thì không biết chuyện gì xảy ra. Tui check hàm đầu tiên và dừng tại đây vì khi tui chưa hiểu ý tưởng của cậu thì có tiếp tục sữa cũng vô nghĩa ! Tui demo cho cậu cách input string, cậu post idea lên đây tui sẽ coi tiếp.

#include <iostream>#include <cstring> char* editString( char* str ){    int s_len = strlen( str );        while(  !(  ( ( str[ 0 ] >= 'a' ) && ( str[ 0 ] <= 'z' ) )                ||                ( ( str[ 0 ] >= 'A' ) && ( str[ 0 ] <= 'Z' ) )             )         )    {        for( int x = 0; x < s_len; ++x)            str[ x ] = str[ x + 1 ];    }     while( str[ s_len - 1 ] == ' ')    {        str[ s_len - 1 ] = '\0';    }        return str;} int main(){    char* s;    int   str_size;        std::cout << "Enter size of string :\n";    std::cin >> str_size;     std::cin.ignore( 80, '\n');        s = new char[ str_size ];        std::cout << "Enter a string : \n";    std::cin.getline( s, str_size );        editString( s );     std::cout << "After edit :\n";    std::cout << s << "\n";     delete[] s;}

[phthinh]

mấy cái hàm này tui viết để nhằm xử lý xâu chuỗi, nó liên quan đến cái bài tập mà tui đang làm.
trong đó yêu cầu phải xử lý cặn kẽ về xâu:
ví dụ xâu vào:
phthinh thanks everybody.
thì phải thực hiện được các công việc:
- cắt được phthinh, thanks , và everybody ra khỏi xâu
- tìm xem trong xâu vào đó có chuổi con a nào đó không (mà chuổi con này được nhập vào bất kỳ ,ví dụ như chuổi :phthinh chẳng hạn), ngoài ra còn cho phép tìm một ký tự mà ,bỏ qua một số ký tự nhất định (bắt đầu từ một vị trí định trước, như tìm chữ 'h' mà chữ này không thuộc xâu con là phthinh chẳng hạn...), bởi vậy mình mới viết hàm findstr(string a, string b,int vitri), cái này có thể kết hợp tìm cả chuổi con lẫn cả một ký tự (ký tự được xem như chuổi có độ dài là 1).còn 0 chính là tìm từ vị trí đầu chuổi!
xâu vào là nhập bất kỳ từ bàn phím , xâu đó bao gồm cả những ký tự đặc biệt.và độ dài xâu vào là không xác định.(chương trình yêu cầu xử lý nhiều xâu vào cùng một lúc nên mình định nghĩa kiểu string cho tiện thôi)
- còn cái vụ mà code tối nghĩa đó chắc do bác chưa quen thôi chứ tui đưa cho mấy đứa bạn , bọn chúng đều biết được đoạn code trên tui muốn làm gì.
- còn cái hàm isalpha() thì quả thực tui không biết nhiều hàm của C và C++ lắm, với lại tui không quen dùng những thứ có sẵn, tui thích dùng những thú mà mình tạo ra, khi đó mình có thể tùy biến nó theo ý mình chứ không nhất thiết phải đi theo nhưng đường mà họ đã vạch ra. với lại cứ mỗi lần muốn dùng hàm nào đó lại phải đi tìm xem trong ngôn ngữ đó có hàm đó hay không à? phải tự viết lấy mấy cái mà dùng riêng chứ!
và còn nhiều chức năng khác nữa nhưng mình chỉ đưa lên chừng này thôi !
còn tại sao không dùng một biến toàn cục để lưu giá trị strlen(a) ư, cái này mình cũng đã làm, nhưng do số lượng biến toàn cục trong chương trình mình nhiều quá rồi(đoạn code này chỉ chiếm chưa đầy 1/10 cái chương trình thôi )
cảm ơn mọi người đã góp ý nha!,nếu ai có ý kiến nào hay hơn thì xin chỉ dùm tiếp

[phthinh]

cảm ơn các bác nghe, code trên đã có cao nhân chỉ điểm rồi, chạy ok luôn ,không mắc lỗi nữa!
chỉ cần thêm hàm malloc() vào là chạy ngon ơ

[rox_rook]

cảm ơn các bác nghe, code trên đã có cao nhân chỉ điểm rồi, chạy ok luôn ,không mắc lỗi nữa!
chỉ cần thêm hàm malloc() vào là chạy ngon ơ

Nếu cậu nói bỏ malloc vào chạy ngon ơ, mà cao nhân này đọc code này mà cũng làm thinh bỏ vào malloc thì tui cũng chẳng nghĩ người này là cao nhân đâu ! Vấn đề không phải là tui bắt bẻ cậu mà là tui phân tích code cho cậu hiểu, cậu dùng C++ viết code, mà cậu nói bỏ malloc vào thì tui đầu hàng.
- Còn tạo 1 biến cần gì phải là global. Xét đoạn code của cậu xem. Hàm strlen nó đại loại thế này

 unsigned strlen( char const* s ){    int count = 0;    for(; *s != '\0'; ++s )        count++;    return count;}

Mỗi lần cậu gọi hàm strlen( s ) thì tất nhiên sẽ có 1 vòng for được gọi để tính độ dài, vậy chuỗi cậu dài chừng 80 kí tự, gọi hàm chừng 1 chục lần thì cậu nhân lên thử xem.

char *chinhstr(string ch)  {   int i;   while(!(((ch[0] >= 'a')&&(ch[0] <= 'z'))||((ch[0] >= 'A')&&(ch[0] <= 'Z'))))     {    for(i=0;i<strlen(ch);i++)      ch[i]=ch[i+1];     }   while(ch[strlen(ch)-1] == ' ')     {    ch[strlen(ch)-1]='\0';     }   return ch;  }

Tạo 1 biến glocal không được sao ?

- còn cái hàm isalpha() thì quả thực tui không biết nhiều hàm của C và C++ lắm, với lại tui không quen dùng những thứ có sẵn, tui thích dùng những thú mà mình tạo ra, khi đó mình có thể tùy biến nó theo ý mình chứ không nhất thiết phải đi theo nhưng đường mà họ đã vạch ra. với lại cứ mỗi lần muốn dùng hàm nào đó lại phải đi tìm xem trong ngôn ngữ đó có hàm đó hay không à? phải tự viết lấy mấy cái mà dùng riêng chứ!
và còn nhiều chức năng khác nữa nhưng mình chỉ đưa lên chừng này thôi !

Vậy sao cậu không viết cả hàm strlen và strcpy luôn để xài cho tiện ? Cậu có hiểu thế nào là code tối nghĩa và code cần viết không ?
Tui nói thì cậu nghĩ tui bắt bẻ vặt vãnh, khi học đến secure code, design pattern rùi sẽ hiểu.
Mà thôi, tui cảm thấy tui không có sức để cãi nữa, vì cãi rùi lại nói tui dạy đời. Mấy câu hay đem cao nhân ra hù doạ tui sợ lắm. Good luck !

[phthinh]

có lẽ bác nói đúng . vì tuy không thích xài của có sẵn ,nên tui đã viết riêng cho mình một số thư viện để dùng khi cần rồi