Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Карта форума Темы раздела Блоги Сообщество Поиск Заказать работу  
 
 
Рейтинг 4.55/29: Рейтинг темы: голосов - 29, средняя оценка - 4.55
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
1

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

24.04.2016, 17:05. Показов 5486. Ответов 31
Метки c++ (Все метки)

Author24 — интернет-сервис помощи студентам
Добрый день! Пишу программу управления колледжом. И столкнулась с этой ошибкой.

У меня есть класс 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++;
    }
...
Ошибка появляется тогда когда я хочу получить указатель на курсы и создать массив.
0
Programming
Эксперт
94731 / 64177 / 26122
Регистрация: 12.04.2006
Сообщений: 116,782
24.04.2016, 17:05
Ответы с готовыми решениями:

Error C2106: '=' : left operand must be l-value
Всем привет. Вот у меня есть такой хедер: #ifndef MATRIXX_H #define MATRIXX_H #include...

error C2106: '=' : left operand must be l-value выдаёт ошибку на сортировку
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; #include &lt;math.h&gt; using namespace std; int main() {...

Error: lvalue required as left operand of assignment|
Программа перевода с 10-ой СИ в римскую.В строках начиная с 19 пишет &quot;error: lvalue required as...

[Error] lvalue required as left operand of assignment
Задание Ошибка при компиляции. 28 14 lvalue required as left operand of assignment...

31
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
28.04.2016, 10:29  [ТС] 21
Author24 — интернет-сервис помощи студентам
Тогда static здесь не нужен?
0
7792 / 6559 / 2984
Регистрация: 14.04.2014
Сообщений: 28,668
28.04.2016, 10:33 22
И учитывая, что массив двумерный, поиск индекса, наверное, нужно не вычитанием искать, а просмотром и сравнением.

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

У тебя массив указателей и в нём каждый указатель только на один объект, так? Т. е. двумерный массив с одним столбцом?
1
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
28.04.2016, 10:43  [ТС] 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;
тоже не нужно?
0
7792 / 6559 / 2984
Регистрация: 14.04.2014
Сообщений: 28,668
28.04.2016, 10:44 24
Цитата Сообщение от Alonka Посмотреть сообщение
Тоесть убрать static из Student и просто сделать так
Эта функция вообще не нужна. Ты можешь просто взять указатель от s с помощью &:
C++
1
&s
Массив так устроен как я описал? Тогда нужен сначала поиск, а не вычитание.
1
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
28.04.2016, 10:46  [ТС] 25
Да,я поняла. Спасибо большое! Буду разбираться. Просто до этого разбиралась с шаблоном Singlton и что-то видно из-за него запуталась.
0
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
30.04.2016, 14:18  [ТС] 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;
}
0
7792 / 6559 / 2984
Регистрация: 14.04.2014
Сообщений: 28,668
30.04.2016, 18:41 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 только от этого.
1
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
01.05.2016, 13:56  [ТС] 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().
0
7792 / 6559 / 2984
Регистрация: 14.04.2014
Сообщений: 28,668
01.05.2016, 14:01 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;     
}
1
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
01.05.2016, 14:08  [ТС] 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 по его индексу.
0
7792 / 6559 / 2984
Регистрация: 14.04.2014
Сообщений: 28,668
01.05.2016, 15:20 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;      
    }   
}
Удалить по индексу где? Есть такая функция для удаления по индексу?
1
0 / 0 / 0
Регистрация: 17.04.2016
Сообщений: 18
01.05.2016, 16:48  [ТС] 32
Да, новый студент создается через new
C++
1
2
Student *temp_student = new Student;
cin >> *temp_student;
И функции по удалению по индексу нету. Но в задании дано что нужно кроме всех операторов еще перегрузить оператор []. Который получает индекс и возвращает студента по этому индексу.
И просто я уже все сделала и так мне эта перегрузка оператора и нигде и не понадобилась.
И я решила что возможно с помощью [] скобок нужно как-то удалять студента.

Добавлено через 46 минут
Вобщем все работает.Спасибо огромное за помощь!!!
Правда так и не поняла зачем вообще здесь перегружать квадратные скобки.
0
01.05.2016, 16:48
IT_Exp
Эксперт
87844 / 49110 / 22898
Регистрация: 17.06.2006
Сообщений: 92,604
01.05.2016, 16:48
Помогаю со студенческими работами здесь

Error: lvalue required as left operand of assignment. Не могу разобраться где ошибка
#include &lt;stdio.h&gt; #include &lt;stdlib.h&gt; int main() { int n, count = 0; for (n = 111; n...

Error C2106 или error C2440
Необходимо создать односвязный список из слов строки.Ошибка в 28 и 35 строке. Код: #include...

Lvalue required as left operand of assignment
Подскажите, пожалуйста. Можно ли так (точнее что-то подобное) сделать? template &lt;class T&gt; ...

Ошибка в методе, left operand maust be I-value
#include &quot;PrintClass.h&quot; #include &quot;Class.h&quot; #include &lt;iostream&gt; using namespace std; ...


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

Или воспользуйтесь поиском по форуму:
32
Ответ Создать тему
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2024, CyberForum.ru