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

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

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

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

У меня есть класс 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
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
24.04.2016, 17:05
Ответы с готовыми решениями:

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

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() { setlocale(LC_ALL,...

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

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

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

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

Добавлено через 46 минут
Вобщем все работает.Спасибо огромное за помощь!!!
Правда так и не поняла зачем вообще здесь перегружать квадратные скобки.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
01.05.2016, 16:48
Помогаю со студенческими работами здесь

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

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 &lt; 1000; ++n) if (n / 100...

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

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

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


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

Или воспользуйтесь поиском по форуму:
32
Ответ Создать тему
Новые блоги и статьи
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru