c ++ массив для присвоения массива

Question

c ++ массив для присвоения массива

0

Я написал следующий класс employee:

#include<iostream>
#include<string>
using namespace std;

class employee
{
    private:
        int id;
        int salaries[12];
        int annualS;
        string name;

    public:
        employee(int id2, string name2, int array[12])
        {
            id = id2;
            name=name2;
            salaries = array; //here where the error occurred.
        }
        ~employee()
        {
            cout<<"Object Destructed";
        }

        employee()
        {
            id = 0;
            name="Mhammad";
        }

        int annulalSalary()
        {
            for( int i=0; i<12; i++)
            {
                annualS+=salaries[i];
            }
            return annualS;
        }
        int tax()
        {
            return (annualS*10/100);
        }
};

void main()
{
    int salaries[12];
    for(int i=0; i<12; i++)
    {
        cin>>salaries[i];
    }

    employee Mohammad(10,"Mohammad",salaries);

    cout<< Mohammad.annulalSalary();
    cout<< Mohammad.tax();
}

... но когда я его компилирую, компилятор возвращает следующую ошибку:

cannot convert from 'int []' to 'int [12]'

Может ли кто-нибудь помочь мне решить эту проблему?

zein alsalman 17 окт. 2014, в 22:03

Источник

5

Используйте std::array<T> или std::vector<T> , у них есть operator= перегружен (среди прочего), что означает, что вам не нужно писать код присваивания самостоятельно.
Borgleader 17 окт. 2014, в 19:57
0

проблема в аргументе вашего конструктора. зарплаты не могут быть объявлены размером 12. Вместо этого используйте int* salaries . Но да, вы должны использовать вектор <T>, гораздо лучше и безопаснее
WindowsMaker 17 окт. 2014, в 20:06

Теги:

c++

arrays

4 ответа

Ещё вопросы

Используйте std::array<T> или std::vector<T> , у них есть operator= перегружен (среди прочего), что означает, что вам не нужно писать код присваивания самостоятельно.
проблема в аргументе вашего конструктора. зарплаты не могут быть объявлены размером 12. Вместо этого используйте int* salaries . Но да, вы должны использовать вектор <T>, гораздо лучше и безопаснее

yzt · Answer 1 · 2014-10-17T18-41-00.000Z

Вместо массива C используйте [ std::array<> ] C++, например:

class employee {
    //...
    std::array<int, 12> salaries;
    //...
};

и, конечно же, вы должны включить <array>. И объявите конструктор следующим образом:

employee(int id2, string name2, std::array<int, 12> const & array)
{
    //...
}

(или отбросить const & если вы не уверены, что они или не нуждаются в них).

Mahesh · Answer 2 · 2014-10-17T18-23-00.000Z

Вы не можете копировать массивы по заданию. Вам нужно скопировать каждый элемент по отдельности. Используйте std::copy.

std::copy(array, array+12, salaries);

Или используйте std::vector<int> или std::array<int, 12> как это предлагает Borgleader, который копирует по заданию.

@ Jarod42 Спасибо. Забыли параметр std::size_t .

WindowsMaker · Answer 3 · 2014-10-17T17-34-00.000Z

Используйте векторный класс!

Но для решения вашей проблемы:

int salaries[12] должен быть int* salaries employee(int id2, string name2, int array[12]) должен быть employee(int id2, string name2, int* array)

но тогда у вас могут возникнуть проблемы со ссылкой на вещи вне выделенной памяти и segfault. ИСПОЛЬЗУЙТЕ ВЕКТОРЫ!

SouvikMaji · Answer 4 · 2014-10-17T17-15-00.000Z

вы не можете скопировать весь массив, используя оператор = в c++. у вас есть два варианта.

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

for (int я = 0; я <12; i++) зарплаты [i] = array [i];

на другой ноте не используйте магические числа, такие как 12 в вашем коде.