c++ - セットをどのように使用し、C ++で取得しますか？

Question

私はそれらをJavaで使用していて、それほど多くの問題はないようですが、C++ではあまりよく理解していません。割り当ては次のとおりです。

Write a class named Car that has the following member variables:

    year. An int that holds the car's model year.
    make. A string that holds the make of the car.
    speed. An int that holds the car's current speed.

In addition, the class should have the following member functions.

    Constructor.  The constructor should accept the car's year and make 
        as arguments and assign these values to the object's year and 
        make member variables.  The constructor should initialize the 
        speed member variable to 0.

    Accessors.  Appropriate accessor functions should be created to allow 
        values to be retrieved from an object's year, make, and speed 
        member variables.

    accelerate.  The accelerate function should add 5 to the speed member 
        variable each time it is called.

    brake.  The brake function should subtract 5 from the speed member 
        variable each time it is called.

Demonstrate the class in a program that creates a Car object, and then 
    calls the accelerate function five times.  After each call to the 
    accelerate function, get the current speed of the car and display 
    it.  Then, call the brake function five times.  After each call to 
    the brake function, get the current speed of the car and display it.

これまでのところ、これは私が持っているものですが、私は完全に間違っていると合理的に確信しています。誰かアドバイスがあれば本当にありがたいです、ありがとう！

#include<iostream>
#include<string>

using namespace std;

class Car
{
    public:
        int year;
        string make;
        int speed;
    Car()
    {
        setYear(newYear);
        setMake(newMake);
        setSpeed(0);
    }

    void setYear(int newYear)
    {
        year = newYear;
    }
    void setMake(string newMake)
    {
        make = newMake;
    }
    int getYear()
    {
        return year;
    }
    string getMake()
    {
        return make;
    }
    int accelerate(int speed)
    {
        return (speed+5);
    }
    int brake(int speed)
    {
        return (speed-5);
    }
};

int main()
{
    return 0;
}

PS：メインは0を返します。純粋に「取得して設定」すること全体を理解しようとするプレースホルダーとして。

score 4 · Accepted Answer

通常、get/set 関数は正常に動作するはずです。その他のコメント:

、および変数はおそらくプライベートにする必要があります。それ以外の場合はyear、変数を直接変更することもできるため、get/set 関数を使用する必要はまったくありません。makespeed
おそらく、集合関数はまったく存在しないはずです。yearまたはmakeまたはをspeed直接変更できるようにすることは意図されていないと思います。
コンストラクターはパラメーターとしてnewYearandを取る必要がありnewMakeます。
accelerate()とは異なる値を返すだけでなく、 car オブジェクトに保存されているbreak()を変更する必要があります。speedspeed
using namespace std;は多くの予期しない名前をグローバル名前空間にインポートする可能性があり、std::string代わりに明示的に修飾された名前のようなものを使用することをお勧めします。

score 1 · Accepted Answer

私が見るいくつかの問題：

コンストラクターに渡されなかったコンストラクター内の変数を参照しています（newYear、newMake）

accelerateand関数はdecelerate状態を変更しません。渡された速度から5を足したり引いたりするだけです-私は彼らがこのように振る舞うことになっているとは思いません。speed 問題の説明には、メンバー変数に加算/減算することが記載されていることに注意してください。

すべてのメンバー変数はパブリックです。これをJavaで行いますか？

score 1 · Accepted Answer

内部変数はクラス内のメソッドによってのみ更新される必要があるため、おそらく内部変数を非公開にする必要があります。次に、最初に年と作成を設定できるように、コンストラクターへのパラメーターが必要です。

元：

public: Car(int newYear, string newMake) {...}

class Car
{
private:
        int year;
        string make;
        int speed;
public:
    Car(int newYear, string newMake)
    {
        setYear(newYear);
        setMake(newMake);
        setSpeed(0);
    }
    ...
}

accelerateまた、brakeメソッドの速度の値を更新していません。試す：

return (speed -=5);

また

return (speed += 5);

score 0 · Accepted Answer

accelerate()andbrake()関数は、変更された値を返すだけでなく、メンバーを操作する必要がありspeedます。これは、必要に応じてに割り当てることを意味speedします。

また、アクセサを持っているメンバーは通常、公開されるprivateかprotected、公開される代わりに作成されます。

score 0 · Accepted Answer

いくつかのコメント:

メンバー変数は public ではなく private にする必要があります。publicにするとカプセル化が破られ、アクセサー (getter/setter) 関数を使用してそれらにアクセスする目的が無効になるためです。
関数名と関数パラメーターがメンバー変数の名前を覆い隠しているため、混乱が生じています。this.xメンバー変数 "x" をパラメーター "x" と区別するためにを使用する必要がある Javaと同様に、 this->x. ただし、メンバー変数に何らかの接頭辞を常に与えると、これを回避できます。2 つの一般的な規則は、メンバー変数の前にアンダースコアを付ける (たとえば、メンバー変数に名前を付け、パラメーターの名前として_speed使用speedする) か、'm' ("メンバー" の意味) に続けてアンダースコアを使用することです。
データを変更しない関数 (つまり、すべての「getter」関数) はキーワードconstで宣言して、このデータにconst Carorからアクセスできるようにする必要がありますconst Car&。たとえば、定数を宣言する int getSpeed()const代わりに使用します。int getSpeed()

score 0 · Accepted Answer

getter メソッドと setter メソッドを使用してデータのカプセル化を実現し、クラスメンバーのみがクラスのデータメンバーにのみアクセスできるようにします。

c++ - セットをどのように使用し、C ++で取得しますか？

6 に答える 6

Related

Reference