Форум программистов, компьютерный форум CyberForum.ru

Error C2106: '=' : left operand must be l-value - C++

Восстановить пароль Регистрация
 
 
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
24.04.2016, 17:05     Error C2106: '=' : left operand must be l-value #1
Добрый день! Пишу программу управления колледжом. И столкнулась с этой ошибкой.

У меня есть класс Department который имеет массив указателей на Course:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
class Department
{
    static Department* p_department;    
private:
    string name_department;
    long id_department; 
    Course** p_course;
    int count_course;
    Department(const Department& d);
    void operator = (const Department& d);
public: 
    Department();
    static Department* instance();
    ~Department();  
    void set(string name_department, long id_department);
    long get_id_department() const { return id_department; }
    int get_count_course() const { return count_course; }
    Course** get_p_course() { return p_course; }
    friend ostream& operator << (ostream &os, const Department* p_department);
    friend istream& operator >> (istream &is, Department* p_department);
    friend void operator += (Department* p_department,const Course& temp);
 
};
И в нем я пытаюсь перегрузить оператор += который добавляет Course в Department:
C++
1
2
3
4
5
6
7
8
9
10
11
void operator += (Department* p_department, const Course& temp)
{
    int count_course = p_department->get_count_course();
    if (!count_course)
    {
        p_department->get_p_course() = new Course*[count_course + 1]; //здесь выдает ошибку
        p_department->get_p_course()[0] = Course::instance();
        cin >> p_course[0];
        count_course++;
    }
...
Ошибка появляется тогда когда я хочу получить указатель на курсы и создать массив.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
28.04.2016, 10:29  [ТС]     Error C2106: '=' : left operand must be l-value #21
Тогда static здесь не нужен?
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
nmcf
4260 / 3691 / 1243
Регистрация: 14.04.2014
Сообщений: 14,458
28.04.2016, 10:33     Error C2106: '=' : left operand must be l-value #22
И учитывая, что массив двумерный, поиск индекса, наверное, нужно не вычитанием искать, а просмотром и сравнением.

Добавлено через 2 минуты
Цитата Сообщение от Alonka Посмотреть сообщение
Тогда static здесь не нужен?
Для таких целей нет. Я же говорю: static - это как глобальная переменная. Одна на всех, и значение одно.

У тебя массив указателей и в нём каждый указатель только на один объект, так? Т. е. двумерный массив с одним столбцом?
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
28.04.2016, 10:43  [ТС]     Error C2106: '=' : left operand must be l-value #23
Да! Тоесть убрать static из Student и просто сделать так:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
class Student
{
    Student* p_student;
private:
    string name_student;
    long id_student;
    char sex_student;
    int age_student;
    Course** p_course;
    int count_course;
    Department* p_department;
    Student(const Student& s);
    void operator = (const Student& s);
public:
    Student();
    Student* instance();
    ~Student(); 
    Student* get_p_student() const { return p_student; }}
Добавлено через 2 минуты
C++
1
Student* get_p_student() const { return p_student; }
И это тогда здесь тоже не нужно

Добавлено через 6 минут
И функция
C++
1
Student* instance();
и
C++
1
Student* p_student;
тоже не нужно?
nmcf
4260 / 3691 / 1243
Регистрация: 14.04.2014
Сообщений: 14,458
28.04.2016, 10:44     Error C2106: '=' : left operand must be l-value #24
Цитата Сообщение от Alonka Посмотреть сообщение
Тоесть убрать static из Student и просто сделать так
Эта функция вообще не нужна. Ты можешь просто взять указатель от s с помощью &:
C++
1
&s
Массив так устроен как я описал? Тогда нужен сначала поиск, а не вычитание.
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
28.04.2016, 10:46  [ТС]     Error C2106: '=' : left operand must be l-value #25
Да,я поняла. Спасибо большое! Буду разбираться. Просто до этого разбиралась с шаблоном Singlton и что-то видно из-за него запуталась.
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
30.04.2016, 14:18  [ТС]     Error C2106: '=' : left operand must be l-value #26
Появился вопрос. Я добавляю новый Department в College. В College.h есть Department** p_department.
Допустим name_department физика и id_department 123.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
case 1: //Add department
            {               
                if (!count_department)
                {
                    p_department = new Department*[count_department + 1];
                    Department temp_department;
                    cin >> temp_department;
                    p_department[0] = &temp_department;                 
                    count_department++;
                }
                else
                {                   
                    Department** temp_p_department = new Department*[count_department + 1];
                    for (int i = 0; i < count_department; i++)
                    {
                        temp_p_department[i] = p_department[i];
                    }                   
                    Department temp_department;
                    cin >> temp_department;
                    p_department[count_department] = &temp_department;
                    for (int i = 0; i < count_department; i++)
                        if (temp_p_department[count_department]->get_id_department() == p_department[i]->get_id_department())
                        {
                            cout << "Error! This department already exists." << endl;
                            goto end;
                        }
                    p_department = new Department*[count_department + 1];
                    for (int i = 0; i < count_department + 1; i++)
                    {
                        p_department[i] = temp_p_department[i];
                    }
                    count_department++;
                    delete[] temp_p_department;
                }               
                cout << endl;               
            end:break;
            }
А потом когда хочу вывести на экран данные о добавленом Department. Он выводит только id_department 123 а вместо name_department просто пустота:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
case 8: //Print department information
            {
                long id_department;
                bool flag = false;
                cout << "Please enter code of department:";
                cin >> id_department;
                for (int i = 0; i < count_department; i++)
                    if (id_department == p_department[i]->get_id_department())
                    {
                        cout << *p_department[i];
                        flag = true;
                    }
                if (!flag)
                    cout << "Error! This department does not exist.";
                cout << endl;
                break;
            }
Используются перегруженные операторы >> <<:

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
ostream& operator << (ostream &os, const Department& p_department) //operator cout
{
    os << p_department.name_department << endl << p_department.id_department << endl;
    return os;
}
 
istream& operator >> (istream &is, Department& p_department) //operator cin
{
    cout << "Please enter department details." << endl;
    cout << "Name:";
    is >> p_department.name_department;
    cout << "Code:";
    is >> p_department.id_department;
    return is;
}
nmcf
4260 / 3691 / 1243
Регистрация: 14.04.2014
Сообщений: 14,458
30.04.2016, 18:41     Error C2106: '=' : left operand must be l-value #27
Так нельзя. Всё нужно динамически создавать. А ты указатель на локальную переменную просто сохраняешь.
C++
1
2
3
4
5
                    p_department = new Department*[count_department + 1];
                    Department *temp_department = new Department;
                    cin >> *temp_department;
                    p_department[0] = temp_department;                 
                    count_department++;
В других случаях аналогично. Я ведь спрашивал, зачем двумерный массив нужен. Лишние new только от этого.
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
01.05.2016, 13:56  [ТС]     Error C2106: '=' : left operand must be l-value #28
Спасибо большое за помощь!!Все получилось кроме удаления Student по его индексу из Course. И для этого как-то нужно использовать перегруженные квадратные скобки.

C++
1
Student* Course::operator [] (int index_student) { return p_student[index_student]; }
И само удаление студента по индексу:
Я переделала поиск индекса студента через цикл

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
void Course::operator -= ( const Student & s)
{
    Student** temp = new Student*[count_student - 1];   
    int index;
    for (int i = 0; i < count_student; i++)
    {
        if (p_student[i] == &s)     
            index = i;      
    }
    for (int i = 0; i < index; i++)
    {
        temp[i] = p_student[i];
    }
    for (int i = 0; i < count_student - 1; i++)
    {
        temp[i] = p_student[i + 1];
    }       
    delete p_student[index]; //здесь ошибка.все никак не могу понять как удалить
    p_student=temp;     
}
Добавлено через 13 часов 53 минуты
Уже все перепробывала. Все никак не могу удалить. Выдает или ошибка доступа к памяти или вызов abort().
nmcf
4260 / 3691 / 1243
Регистрация: 14.04.2014
Сообщений: 14,458
01.05.2016, 14:01     Error C2106: '=' : left operand must be l-value #29
А как заполняется p_student? Так примерно должно быть:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
void Course::operator -= ( const Student & s)
{
    Student** temp = new Student*[count_student - 1];   
 
    for (int i = 0, j = 0; i < count_student; i++)
    {
        if (p_student[i] == &s) continue;
        temp[j] = p_student[i];
        ++j;
    }
 
    delete &s;
    delete[] p_student;
    p_student = temp;     
}
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
01.05.2016, 14:08  [ТС]     Error C2106: '=' : left operand must be l-value #30
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
void Course::operator += ( Student & s) //operator += add Student to Course
{
    if (!count_student)
    {
        p_student = new Student*[count_student + 1];        
        p_student[0]=&s;        
        count_student++;        
    }
    else
    {
        for (int i = 0; i < count_student; i++)
            if (s == p_student[i]->get_id_student())
            {
                cout << "Error! This student already exists." << endl;
                return;
            }
        Student** temp_student = new Student*[count_student + 1];           
        temp_student[count_student] = &s;               
        p_student = new Student*[count_student + 1];        
        for (int i = 0; i < count_student + 1; i++)
        {
            p_student[i] = temp_student[i];         
        }
        count_student++;        
        delete[] temp_student;      
    }   
}
Добавлено через 2 минуты
И как бы я не пойму зачем мне в задании нужна перегрузка [].
Я думала что как раз чтобы это как-то использовать чтоб удалить Student по его индексу.
nmcf
4260 / 3691 / 1243
Регистрация: 14.04.2014
Сообщений: 14,458
01.05.2016, 15:20     Error C2106: '=' : left operand must be l-value #31
А добавляемый студент через new был создан? Если нет, то надо нового создавать при добавлении.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
void Course::operator += ( Student & s) //operator += add Student to Course
{
    if (!count_student)
    {
        p_student = new Student*[++count_student];        
        p_student[0] = new Student(s);        
    }
    else
    {
        for (int i = 0; i < count_student; i++)
            if (s == p_student[i]->get_id_student())
            {
                cout << "Error! This student already exists." << endl;
                return;
            }
 
        Student** temp_student = new Student*[count_student + 1];
        temp_student[count_student] = new Student(s);               
 
        for (int i = 0; i < count_student; i++) temp_student[i] = p_student[i];         
 
        count_student++;        
 
        delete[] p_student;
        p_student = temp_student;      
    }   
}
Удалить по индексу где? Есть такая функция для удаления по индексу?
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
01.05.2016, 16:48     Error C2106: '=' : left operand must be l-value
Еще ссылки по теме:

Ошибка: lvalue required as left operand of assignment C++
Error C2106 или error C2440 C++
C++ Error: lvalue required as left operand of assignment|

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
Alonka
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 17
01.05.2016, 16:48  [ТС]     Error C2106: '=' : left operand must be l-value #32
Да, новый студент создается через new
C++
1
2
Student *temp_student = new Student;
cin >> *temp_student;
И функции по удалению по индексу нету. Но в задании дано что нужно кроме всех операторов еще перегрузить оператор []. Который получает индекс и возвращает студента по этому индексу.
И просто я уже все сделала и так мне эта перегрузка оператора и нигде и не понадобилась.
И я решила что возможно с помощью [] скобок нужно как-то удалять студента.

Добавлено через 46 минут
Вобщем все работает.Спасибо огромное за помощь!!!
Правда так и не поняла зачем вообще здесь перегружать квадратные скобки.
Yandex
Объявления
01.05.2016, 16:48     Error C2106: '=' : left operand must be l-value
Ответ Создать тему

Метки
c++
Опции темы

Текущее время: 20:17. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru