11

char 配列の宣言と初期化に問題があります。常にランダムな文字が表示されます。大規模なプログラムで何をしようとしているのかを示すために、小さなコードを作成しました。

class test
{
    private:
        char name[40];
        int x;
    public:
        test();
        void display()
        {
            std::cout<<name<<std::endl;
            std::cin>>x;
        }
};
test::test()
{
    char name [] = "Standard";
}

int main()
{   test *test1 = new test;
    test1->display();
}

私のフォーマットが悪い場合は申し訳ありませんが、コードを修正する方法は言うまでもなく、このWebサイトをほとんど理解できません:(

4

5 に答える 5

10

使用しない特別な理由がない場合はstd::string、を使用してくださいstd::string。

しかし、本当にその文字配列メンバーを初期化する必要がある場合は、次のようにします。

#include <assert.h>
#include <iostream>
#include <string.h>
using namespace std;

class test
{
    private:
        char name[40];
        int x;
    public:
        test();
        void display() const
        {
            std::cout<<name<<std::endl;
        }
};

test::test()
{
    static char const nameData[] = "Standard";

    assert( strlen( nameData ) < sizeof( name ) );
    strcpy( name, nameData );
}

int main()
{
    test().display();
}
于 2012-05-02T22:30:47.837 に答える
6

コンストラクターはメンバー変数を設定していませんname。ローカル変数を宣言しています。コンストラクターの最後でローカル変数がスコープから外れると、ローカル変数は消えます。その間、メンバー変数はまだ初期化されておらず、ランダムなゴミでいっぱいです。

strcpy昔ながらの文字配列を使用する場合は、メンバー変数にコピーするような昔ながらの関数も使用する必要があります。空の文字列に設定するだけの場合は、で初期化できますname[0] = 0。

于 2012-05-02T22:28:45.940 に答える
5

質問に C++ のタグを付けたことを考えると、次を使用する必要がありますstd::string。

#include <string>

class test
{
    private:
        std::string name;
        int x;
    public:
        test();
        void display()
        {
            std::cout<<name<<std::endl;
            std::cin>>x;
        }
};
test::test() : name("Standard")
{

}
于 2012-05-02T22:23:28.717 に答える
4

C++ を使用しているため、char 配列の代わりに文字列を使用することをお勧めします。それ以外の場合は、strcpy (または友人) を使用する必要があります。

また、test1 インスタンスを削除するのを忘れました。

#include <iostream>
#include <string>

class test
{
    private:
        std::string name;
        int x;
    public:
        test();
        void display()
        {
            std::cout<<name<<std::endl;
        }
};

test::test()
{
    name = "Standard";
}

int main()
{   
    test test1;
    test1.display();

    std::cin>>x;
}
于 2012-05-02T22:23:21.743 に答える