文字列の文字を読み取る際にエラーが発生しました。メモリを割り当てる

Question

私はこのコードを使用しています。

多分それは簡単ですが、今はできません。これについて私を助けてください。この関数では常にNULLを参照しています。

どうすればこの問題を解決できますか？私はそれを行うことはできません。

ありがとうございます。

コード：

int my_len(const char* p) { 

     int c = 0; 
     while (*p != '\0') 
     { 
      c++; 
      *p++; 
     } 
     return c; 
    } 


String::String() 
    :m_str(NULL) 
{ 
} 


String::String(char * other_str) 
{ 
} 
{ 

    int mystrlen = my_len(m_str); 
    int myrhslen = my_len(other_str.m_str); 

    if (mystrlen != myrhslen) 
    { 
     return false; 
    } 
    else 
    { 
     for (int i = 0; i < mystrlen; i++) 
     { 
      if (m_str[i] != other_str.m_str[i]) 
      { 
       return false; 
      } 
     } 
     return true; 
    } 
} 
} 

Answer 1

あなたのデフォルト以外のコンストラクタは空の実装を持っています

String::String(char * other_str) 
{ 
} 

だからここm_strが初期化されていない残っています。それはそうのようなあなたの意図である場合 あなたは多分、文字列をコピーすることができます：

String::String(char * other_str) 
{ 
    m_str = strdup(other_str); 
} 

しかし、その後、あなたは、例えば、自分をSTRDUPによって割り当てられたメモリを管理する必要があります