Как создать конструктор, который инициализирует координаты случайными значениями?

Question

У меня есть этот класс «Точка», который принимает значения x и y в качестве аргументов. Однако мне нужно создать конструктор, который инициализирует их случайными значениями. Я точно не знаю, как это делается. Вот мой код: Я создал конструкторы, но я получаю значения, которые абсурдны, даже когда я устанавливаю x и y.

#include <iostream> 
#include <cmath> 
#include <ctime> 

    using namespace std; 
    class Point 
    { 
    private: 
     double x; 
     double y; 

    public: 

     double get_x() 
     { 
      return x; 
     } 
     void set_x (double x) 
     { 
      this->x = x; 
     } 
     double get_y() 
     { 
      return y; 
     } 
     void set_y(double y) 
     { 
      this->y = y; 
     } 
     double distanceTo(Point p) 
     { 
      double x2 = p.get_x(); 
      double y2 = p.get_y(); 
      return sqrt(pow(x-x2,2) + pow(y-y2,2)); 
     } 
     Point(double x, double y) 
     { 
      x = rand()*1.0/RAND_MAX * 100; 
      y = rand()*1.0/RAND_MAX * 100; 
     } 
     Point(){}; 


    }; 

    void main() 
    { 
     Point a(1.2,0.5); 
     Point b; 
     b.set_x(1); 
     b.set_y(1); 
     cout << a.distanceTo(b); 
     system ("Pause"); 
    } 

Answer 1

Ошибка: Вы меняете локальные копии переменных, переданных конструктору по значению (они имеют одно и то же имя как переменные-члены), а переменные-члены остаются неинициализированными, поэтому ваша программа имеет неопределенное поведение.

Как исправить: Вы должны явно указать, какую переменную вы назначаете. Кроме того, перегрузите конструктор, поэтому у вас будет один (по умолчанию) для случайных значений и один для пользовательского предопределенного.

также:

• это лучше передать ценности, не собирается менять в функции, как постоянная
• вы можете пройти doubleпо ссылке, не значение
• использование списки инициаторов вместо присваивания в теле конструктора
• при использовании постоянных ссылок здесь вам не нужно для разрешения имен, потому что вы не можете изменить постоянные значения, поэтому компилятор будет изменять переменный-член

Код:

class Point 
{ 
public: 
    Point() : 
     x(rand()*1.0/RAND_MAX * 100), 
     y(rand()*1.0/RAND_MAX * 100) 
    { 

    } 

    Point(const double& x, const double& y) : 
     x(x), 
     y(x) 
    { 

    } 

private: 
    double x, y; 

}; 


int main() 
{ 
    Point pt(42, 3.14); 
    Point pt_random; 
} 

же в функциях аксессоры:

double GetX() const { return x; } 
    void SetX(const double& x) { Point::x = x; } 

Answer 2

Поскольку вы изменяете временные переменные в конструкторе (столкновение имен в той же области). Попытка:

Point(double x, double y) 
{ 
    Point::x = rand()*1.0/RAND_MAX * 100; 
    Point::y = rand()*1.0/RAND_MAX * 100; 
} 

Но это полностью игнорирует аргументы, данные конструктору. Но поскольку теперь вы знаете, как различать различные области переменных, вы можете продолжать здесь, я полагаю.

Answer 3

Это потому, что вы не инициализируете переменные-члены, а изменяете копию переменных, которые передаются в конструктор. Следовательно, вы видите значения мусора, поскольку x и y (версии классов) никогда не инициализируются. Вы должны изменить это:

Point() 
{ 
    x = rand()*1.0/RAND_MAX * 100; 
    y = rand()*1.0/RAND_MAX * 100; 
} 

Кроме того, вы никогда не называйте srand() в любом месте - вам нужно сделать, это в какой-то момент, чтобы правильно засеять генератор случайных чисел.

Answer 4

Ваш конструктор по умолчанию ничего не делает. Он даже не инициализирует значения, что означает, что они будут иметь неопределенное значение.

Ваш конструктор, имеющий два аргумента, просто присваивает аргументы, потому что их имена затеняют имена членов.

Answer 5

изменение

Point(){}; 

в

Point() 
{ 
    x = rand()*1.0/RAND_MAX * 100; 
    y = rand()*1.0/RAND_MAX * 100; 
} 

также я бы посоветовал изменить свои аргументы \ члены имя, чтобы избежать ошибок. Я лично хотел бы использовать m_ для участников:

private: 
    double m_x; 
    double m_y; 

Answer 6

Вы получаете «имя столкновения». При этой функции

Point(double x, double y) 
    { 
     x = rand()*1.0/RAND_MAX * 100; 
     y = rand()*1.0/RAND_MAX * 100; 
    } 

Компилятор не знает, какой x вы имеете в виду при назначении. В языковых правилах указывается, что входные аргументы должны быть назначены - но вы не должны полагаться на это - потому что это бесполезно запутывает.

Хорошей практикой является принятие соглашения об именах для переменных-членов. Два общих - это префикс членов с «m» или «_». Мне нравится «м» лично.

Тогда ваш код становится:

class Point 
{ 
    public: 
     double mX; 
     double mY; 

    Point(double x, double y) 
    { 
     mX = rand()*1.0/RAND_MAX * 100; 
     mY = rand()*1.0/RAND_MAX * 100; 
    } 
} 

Также аргументы конструктора в данном случае являются избыточными и могут быть удалены.

Некоторые из приведенных выше ответов также являются правильными, но это признак плохого дизайна класса, если у вас есть имена областей видимости (например, Point ::) в простых функциях.