構造体またはクラスとパブリックデータはどうですか？

Question

クラスの作成に関しては、パブリックデータメンバーはいません。それは問題ありませんが、私は理由を理解していますが、ここに私の問題があります。私はクラスで使用している構造体を持っている（リニアリンクリスト）。私は実際に構造体フィールド（配列）の1つをNULLに初期化して、クライアントが有効な構造体を渡していることを確認できるようにしたいと思います。私はC++が構造体宣言のデフォルト値を設定させないことを理解しています。どうすれば私のゴミをクライアントから守ることができますか？

struct task { 
    char *description; 
    char *name; 
    float time_remaining; 
    float time_spent; 
}; 

その後、私のクラスで私のような何かをしようとしている：何...

//I am making a copy of "to_add" and storing it in the LLL 
int list::add_to_list(const task & to_add) 
{ /.../ } 

私は、ユーザーがリンクリストに初期化されていない「タスクの」の束を追加する必要はありませんが何をする？その構造体をクラスに変換してデータメンバーをプライベートに移動させると、データメンバーにアクセスしてそれらのコピーを作成しようとする問題が多かったです。私はvarの価値をもたらすために何もしないように非常に注意を払っていましたが、コンパイラから離れて "エラー：" const taskを 'this'の引数 'char * task :: get_comp_name' （）は修飾子を捨てる[-fpermissive] "（" get_comp_name "）は私がコピーを渡すだけで値を編集していないと確信していたゲッターの1つでした）顔に（表情に。

Answer 1

C++では、structとclassはアクセス制御を除いて同じです。したがって、構造体のメンバーと継承へのデフォルトのアクセスはpublicですが、クラスのprivateはprivateです。したがって、構造体にデフルトコンストラクタなどを与えて初期化することができます。

struct task { 
    task() : desctiption(0), name(0), time_remaining(0.), time_spent(0.) {} 
    char *description; 
    char *name; 
    float time_remaining; 
    float time_spent; 
}; 

コンストラクタを追加の一副作用は、構造体がもはや凝集しないということです。これはあなたにとって問題かもしれません。 C++ 11では

、あなたはまた、宣言の時点でメンバーを初期化することができます：

struct task { 
    char *description{nullptr}; 
    char *name{nullptr}; 
    float time_remaining{0}; 
    float time_spent{0}; 
}; 

このインプレース初期化があまりにもtype x = y構文を受け入れ、値の引数レス{}初期結果の初期化では、のプリミティブ型の初期化はであるため、この例の引数を省略することができます。

Answer 2

構造体のコンストラクタを実装します。

struct task { 
    task() : description(0), name(0), time_remaining(0), time_spent(0) {} 
    char *description; 
    char *name; 
    float time_remaining; 
    float time_spent; 
}; 

C++でクラスと構造体の唯一の違いは、それのメ​​ンバーのデフォルトアクセシビリティです。

Answer 3

ここにいくつかの問題があります。

パブリックまたはパブリックではありませんか？

パブリック属性は便利なように見えるかもしれませんが、ほとんどの場合、あなたが少なくともそれを期待しているときにあなたを噛ませるように戻ってきます。私はすでにtime_remainingとtime_spentの問題を疑っています。どちらも同じ時刻にかなり修正されていると思いますが、そうではありませんか？クラスは、time_remaining + time_spentとして不変を強制できるようにデフォルトで

、可変属性は、privateなければならないタスクの寿命を通して一定です。

定数属性がpublicであれば問題ありませんが、インバリアントでの役割はコンストラクタ内で一度だけ決まります。

しかし、奇妙なエラーメッセージ？

これは良いチュートリアルや書籍がないためです。手元の問題は非常に単純です：constの問題です。

constオブジェクトは、const-referenceまたはvalueによって関数/メソッドに渡され、constメソッドだけが呼び出されます。あなたの場合、name()メソッドの適切な宣言は、メソッド名の後に修飾子constを持つ必要があります。

完全

それを入れて、それはそんなに簡単操作することですので、std::stringに投げます。

class Task { 
public: 
    Task(): time_remaining(0), time_spent(0) {} 
    Task(std::string name, std::string desc, float duration): 
     _name(name), _desc(desc), _time_remaining(duration), _time_spent(0) {} 

    // Accessors 
    std::string const& name() const { return _name; } 
    std::string const& description() const { return _desc; } 
    float time_remaining() const { return _time_remaining; } 
    float time_spent() const { return _time_spent; } 

    // Modifiers 
    void spend(float duration) { 
     _time_remaining -= duration; 
     _time_spent += duration; 
    } 

private: 
    std::string _name, _desc; 
    float _time_remaining, _time_spent; 
}; // class Task 

注：おそらくspendメソッドに渡されたdurationがそうでなければ、あなたが持っているよりも多くを費やし、_time_remaining属性に優れていないことを確認するために有益であり得る...