Передача свойств объектов между функциями

@netmare · Регистрация: 29.10.2013

Author24 — интернет-сервис помощи студентам

Народ, I need help!
Есть класс:

Кликните здесь для просмотра всего текста

C++

class Employees {
public:
    Employees();
    Employees(std::string _Surname, std::string _Name, std::string _Occupation, int _YearOfEmployment);
    int GetYear();
    
    std::string GetName();
    std::string GetSurname();
    std::string GetOccupation();
    void SetName (std::string _Name);
    void SetSurname (std::string _Surname);
    void SetOccupation (std::string _Occupation);
    void SetYear (int _YearOfEmployment);
private:
    std::string Surname;
    std::string Name;
    std::string Occupation;
    int YearOfEmployment;
};

Есть функция 1, которая задает свойства объекта:

Кликните здесь для просмотра всего текста

C++

void ManualInput () {
    setlocale(LC_ALL, "Russian");
    std::string nbuf,sbuf,obuf;
    int sName, dbuf;
    Employees Emp3[2];  
    int max = Emp3[0].GetYear();
    for (int i=0;i<2;i++) {
    std::cout << "Введите имя:";
    cin >> nbuf;
    Emp3[i].SetName(nbuf);
    std::cout << "Введите фамилию:";
    cin >> sbuf;
    Emp3[i].SetSurname(sbuf);
    std::cout << "Введите профессию:";
    cin >> obuf;
    Emp3[i].SetOccupation(obuf);
    std::cout << "Введите год приема:";
    cin >> dbuf;
    Emp3[i].SetYear(dbuf);
    std::cout << i << ". " << Emp3[i].GetName() << std::endl;
    }
    for(int i=0;i<2;i++) {
        if(max<Emp3[i].GetYear())
        max=Emp3[i].GetYear();
}
    std::cout << "Последний работник принят в " << max << " году" << std::endl;
}

Есть функция 2, которая по сути дела должна редактировать свойства выбранного объекта:

Кликните здесь для просмотра всего текста

C++

void EditEmployess () {
    setlocale(LC_ALL, "Russian");
    int sName, dbuf;
    std::string nbuf,sbuf,obuf;
    Employees Emp3[2];  
    
    std::cout << "Введите номер записи:";
    cin >> sName;
    for (int i=0;i<2;i++){
        if (i == sName) {
    std::cout << "Имя:" <<Emp3[i].GetName() << std::endl;
    std::cout << "Фамилия:" << Emp3[i].GetSurname() << std::endl;
    std::cout << "Должность:" << Emp3[i].GetOccupation() << std::endl;
    std::cout << "Год принятия:" << Emp3[i].GetYear() << std::endl;
    } else { 
        std::cout << "Записи не существует" << std::endl;
        break;
        }
    std::cout << "Введите  имя:";
    cin >> nbuf;
    Emp3[i].SetName(nbuf);
    std::cout << "Введите фамилию:";
    cin >> sbuf;
    Emp3[i].SetSurname(sbuf);
    std::cout << "Введите профессию:";
    cin >> obuf;
    Emp3[i].SetOccupation(obuf);
    std::cout << "Введите год приема:";
    cin >> dbuf;
    Emp3[i].SetYear(dbuf);
    }
}

Проблема в том, что при вызове объекта Emp3[i], почему то не передаются значения, которые мы задавали в функции ввода данных. То есть когда я ввожу номер записи, например 1 - выдает, что записи не существует.
Я никак не могу понять каким образом можно передавать объекты и их свойства между функциями

Заранее спасибо за помощь

@zvoronz · 13.11.2013, 00:39

Объявление массива

C++

1	Employees Emp3[2];

нужно сделать на глобальном уровне, либо добавить указатель на массив в параметры процедуры

C++

1	void ManualInput (Employees* Emp3)

и

C++

1	void EditEmployess (Employees* Emp3)

. Если добавить параметры то тогда не нужна глобальная переменная массив Emp3.
Из процедур

C++

1	Employees Emp3[2];

нужно будет убрать.

@netmare · 13.11.2013, 01:00 **[ТС]**

zvoronz, по Вашему совету сделал так:

Кликните здесь для просмотра всего текста

C++

void ManualInput (Employees* Emp3) {
    setlocale(LC_ALL, "Russian");
    std::string nbuf,sbuf,obuf;
    int sName, dbuf;
    //Employees Emp3[2];    
    int max = Emp3[0].GetYear();
    for (int i=0;i<2;i++) {
    std::cout << "Введите имя:";
    cin >> nbuf;
    Emp3[i].SetName(nbuf);
    std::cout << "Введите фамилию:";
    cin >> sbuf;
    Emp3[i].SetSurname(sbuf);
    std::cout << "Введите профессию:";
    cin >> obuf;
    Emp3[i].SetOccupation(obuf);
    std::cout << "Введите год приема:";
    cin >> dbuf;
    Emp3[i].SetYear(dbuf);
    std::cout << i << ". " << Emp3[i].GetName() << std::endl;
    }
    for(int i=0;i<2;i++) {
        if(max<Emp3[i].GetYear())
        max=Emp3[i].GetYear();
}
    std::cout << "Последний работник принят в " << max << " году" << std::endl;
}

И так:

Кликните здесь для просмотра всего текста

C++

void EditEmployess (Employees* Emp3) {
    setlocale(LC_ALL, "Russian");
    int sName, dbuf;
    std::string nbuf,sbuf,obuf;
    //Employees Emp3[2];    
    
    std::cout << "Введите номер записи:";
    cin >> sName;
    for (int i=0;i<2;i++){
        if (i == sName) {
    std::cout << "Имя:" <<Emp3[i].GetName() << std::endl;
    std::cout << "Фамилия:" << Emp3[i].GetSurname() << std::endl;
    std::cout << "Должность:" << Emp3[i].GetOccupation() << std::endl;
    std::cout << "Год принятия:" << Emp3[i].GetYear() << std::endl;
    } else { 
        std::cout << "Записи не существует" << std::endl;
        break;
        }
    std::cout << "Введите  имя:";
    cin >> nbuf;
    Emp3[i].SetName(nbuf);
    std::cout << "Введите фамилию:";
    cin >> sbuf;
    Emp3[i].SetSurname(sbuf);
    std::cout << "Введите профессию:";
    cin >> obuf;
    Emp3[i].SetOccupation(obuf);
    std::cout << "Введите год приема:";
    cin >> dbuf;
    Emp3[i].SetYear(dbuf);
    }
}

Теперь компилятор выдает ошибку:

C++

1
2

c:\users\netmare\documents\visual studio 2012\projects\lab2\lab2\lab2.cpp(142): error C2660: ManualInput: функция не принимает 0 аргументов
c:\users\netmare\documents\visual studio 2012\projects\lab2\lab2\lab2.cpp(144): error C2660: EditEmployess: функция не принимает 0 аргументов

@zvoronz · 13.11.2013, 01:20

Ну так в main теперь вот так должно быть

C++

int main()
{
...
   Employees Emp3[2];
   ManualInput(Emp3);
   EditEmployess(Emp3);
...
}

@netmare · 13.11.2013, 01:25 **[ТС]**

zvoronz, вроде бы с этим разобрался (видимо я писал ответ одновременно с Вами) - забыл передавать Emp3 при вызове функции. Но проблема в том, что отредактировать можно лишь запись с номером 0, при попытке выбрать другие - "Записи не существует"

@zvoronz · 13.11.2013, 01:34

C++

for (int i=0;i<2;i++){      /////////////////Тут не нужен цикл.
        if (i == sName) {
    std::cout << "Имя:" <<Emp3[i].GetName() << std::endl;
    std::cout << "Фамилия:" << Emp3[i].GetSurname() << std::endl;
    std::cout << "Должность:" << Emp3[i].GetOccupation() << std::endl;
    std::cout << "Год принятия:" << Emp3[i].GetYear() << std::endl;
    } else { 
        std::cout << "Записи не существует" << std::endl;
        break;
        }

Правильный код.

C++

void EditEmployess (Employees* Emp3) {
    setlocale(LC_ALL, "Russian");
    int sName, dbuf;
    std::string nbuf,sbuf,obuf;
    
    std::cout << "Введите номер записи:";
    cin >> sName;
    if (sName => 2) {        std::cout << "Записи не существует" << std::endl;
        return;}
    std::cout << "Имя:" <<Emp3[sName].GetName() << std::endl;
    std::cout << "Фамилия:" << Emp3[sName].GetSurname() << std::endl;
    std::cout << "Должность:" << Emp3[sName].GetOccupation() << std::endl;
    std::cout << "Год принятия:" << Emp3[sName].GetYear() << std::endl;
 
    std::cout << "Введите  имя:";
    cin >> nbuf;
    Emp3[sName].SetName(nbuf);
    std::cout << "Введите фамилию:";
    cin >> sbuf;
    Emp3[sName].SetSurname(sbuf);
    std::cout << "Введите профессию:";
    cin >> obuf;
    Emp3[sName].SetOccupation(obuf);
    std::cout << "Введите год приема:";
    cin >> dbuf;
    Emp3[sName].SetYear(dbuf);
    }
}

@netmare · 13.11.2013, 01:43 **[ТС]**

zvoronz, спасибо огромнейшее

Разобрался что к чему

@netmare 0 / 0 / 0 Регистрация: 29.10.2013 Сообщений: 7
	13.11.2013, 01:25 [ТС]	5
	zvoronz, вроде бы с этим разобрался (видимо я писал ответ одновременно с Вами) - забыл передавать Emp3 при вызове функции. Но проблема в том, что отредактировать можно лишь запись с номером 0, при попытке выбрать другие - "Записи не существует" 0

@netmare 0 / 0 / 0 Регистрация: 29.10.2013 Сообщений: 7
	13.11.2013, 01:43 [ТС]	7
	zvoronz, спасибо огромнейшее Разобрался что к чему 0