0

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

private:
static  std::string Name;  //
static  float Price;         // обьявил приватные переменные
static  int Quantity;     //   

Потом в файле класса

float SmartPhones::Price=0;
int SmartPhones::Quantity=0;
std::string SmartPhones::Name = "";

Далее определил конструкторы ,которые меняют их значение

SmartPhones::SmartPhones(int quantity)
{
    Quantity = quantity;
}
SmartPhones::SmartPhones(std::string name)
{
    Name = name;
}
SmartPhones::SmartPhones(float price)
{
    Price = price;
}

И конструктор который должен выводить их значения

SmartPhones::SmartPhones()
{
    printf("Class SmartPhones opened");
    if ( Price != 0 && Quantity != 0)
    {

        std::cout << "Name:" << SmartPhones::Name;        // получил доступ к значению приватной переменной используя публичную функцию
        printf("\n Price: %f", SmartPhones::Price);      // получил доступ к значению приватной переменной используя публичную функцию
        printf("\n Quantity: %i", SmartPhones::Quantity);  // получил доступ к значению приватной переменной используя публичную функцию
        printf("\n--------------------------");

    }
}

Проблема в том,что в итоге выводятся рандомные значения вроде -89081789798,причем при отладке видно что переменные Name,Price,Quantity имеют нужные значения. В чем проблема? Вызов перегрузок конструктора

case 1: { cin >> Name;  SmartPhones(Name); break; }



  case 2: { cin >> price; SmartPhones(price); break; }                          

case 3: { cin >> quantity; SmartPhones(quantity); break; }  
16
  • Я не знаю, что вам нужно, но описанное выше вам точно не нужно. 17 мая 2019 в 19:14
  • @ARHovsepyan По умолчанию да,но я последовательно вызываю его перегрузки до того,как выводить значения
    – Fate
    17 мая 2019 в 19:30
  • @ARHovsepyan Добавил код в конец вопроса
    – Fate
    17 мая 2019 в 19:38
  • 1
    Какой-то странный у вас билдер :)
    – IR42
    17 мая 2019 в 20:43
  • 1
    С чего вы вообще взяли, что нужны статические переменные? "сохраняли свое значение для каждого объекта класса" Для меня звучит так, что нужны наоборот нестатические. Тогда требование использовать перегруженный конструктор становится логичным. 17 мая 2019 в 21:11

2 ответа 2

0

Наверное вы хотели что то такого (обратите внимание, что один из конструкторов имеет два аргумента):

class SmartPhones {
private:
    static  std::string Name;
    static  float Price;
    static  int Quantity;
public:
    SmartPhones(int quantity, float p)
    {
        Quantity = quantity;
        Price = p;
    }
    SmartPhones(std::string name)
    {
        Name = name;
    }
    SmartPhones(float price )
    {
        Price = price;
    }
    SmartPhones() {
        std::cout << "Class SmartPhones opened\n"
                  << "Name:" << Name
                  << "\n Price: " << Price
                  << "\n Quantity: " << Quantity
                  << "\n--------------------------";
    }

};

float SmartPhones::Price=0;
int SmartPhones::Quantity=0;
std::string SmartPhones::Name = "";

int main()
{ 
    std::string Name;
    float price;
    int quantity;
    std::cin >> price >> Name >> quantity;
    SmartPhones s1(price), s2(Name), s3(quantity, price), s ;
    // вот это последовательный вызов, а не создание временных обьектов
    return 0;
}

Ваши ошибки в том, что вы создаете временные(безимянные) обьекты, которые после выражения уничтожаются, и данные не меняются, но еще и вы перегружаете конструкторы с типами, которые неявно преобразются в друг друга. Например, как может компилятор понять, какой из конструкторов вы вызывать хотите этим: SmartPhones(5)? SmartPhones::SmartPhones(int) или SmartPhones::SmartPhones(float)?. То, что вы аргумент называете price ом, это ничего не значит... Вот почему при выводе вы получаете не то, что ожидали(хотя выводить куда то прямо из конструктора целесообразно разве что только в целях экспериментов) И так как нет полного кода, невозможно точно говорить о всех ваших ошибках

0

Скорее всего, имеется в виду инициализация нестатических полей, которые у каждого экземпляра свои. Для этого конструкторы и применяют на практике. Если требуется определить несколько конструкторов, это не значит, что какие-то поля не надо инициализировать. Задача, аналогичная вашей, делается так:

#include <iostream>
#include <cmath>

using namespace std;


class xy {
public:
    float x, y;

    xy () : x(0), y(0) {}

    explicit xy (float angle) {
        sincosf (angle, &y, &x);
    }

    xy (float x, float y) : x(x), y(y) {}

    void print () {
        cout << "(" << x << ", " << y << ")" << endl;
    }
};


int main () {
    xy a;
    xy b(M_PI / 3);
    xy c(3, 4);

    a.print ();
    b.print ();
    c.print ();
}

Вывод:

(0, 0)
(0.5, 0.866025)
(3, 4)

Условия задачи вроде бы соблюдены: есть несколько конструкторов и переменные там изменяются. Вывод тоже есть. Вам осталось только сделать решение для своего класса на основе этого.


Если все же интересно про статические переменные, то они не привязаны к экземплярам класса, и обращаться к ним можно вообще не имея экземпляра: имя_класса::имя_переменной. Очень редко когда возникает необходимость обращаться к ним из конструктора. Из практики помню только один такой случай. Объекты в игре должны были иметь уникальный номер, причем чтобы можно было по нему сравнивать время их создания. Реализация:

entity.h

#ifndef UNTITLED1_ENTITY_H
#define UNTITLED1_ENTITY_H


class entity {
    static int next_id;
public:
    const int id;
    entity () : id (next_id++) {}
};


#endif //UNTITLED1_ENTITY_H

entity.cpp

#include "entity.h"

int entity::next_id = 0;

main.cpp

#include <iostream>
#include "entity.h"

using namespace std;


int main () {
    entity player, mob1, mob2;
    cout << player.id << " " << mob1.id << " " << mob2.id << endl;
}

Вывод:

0 1 2
1
  • Почему вы решили, что делается именно так? Вы просто предложили свой вариант, но делается совсем по разному. И вообше, любую задачу можно решать многими способами (особенно такую задачу ) 17 мая 2019 в 22:01

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.