c++ - 文字列クラスの実装で立ち往生

Question

私の目標は、呼び出された文字列の i 番目の要素 (配列のようにゼロベース) を返すメソッドを作成することです。要求された場所が文字列の範囲外である場合、このメソッドはエラー メッセージを出力し、null 文字 (\0) を返します。

char String::element(int i) const
{
    if (i < m_str1) 
    {
        cout << s[i]; // also the s here is undefined
    }
    else 
    {
        cout << "Error" << endl;
    }
    return (0);
}

それで、これで問題ないか、さらに追加する必要があるか、未定義の変数を修正するにはどうすればよいかを知りたいだけです。