Блог

Главная - Вопросы по программированию - векторный указатель c не имеет правильного размера vector* ptr, ptr-> размер неправильный

векторный указатель c не имеет правильного размера vector* ptr, ptr-> размер неправильный

#c #pointers #vector #garbage

#c #указатели #вектор #мусор

Вопрос:

Вот мой код

 #include <iostream>
#include <vector>
#include <string>

using namespace std;

struct Student // EdgeContainer
{
    string name = "";
    vector<string> *addresses = nullptr;
    Student()
    {
        cout << __FILE__ << " " << __LINE__ << " Student()" << endl;
    }

    Student(const Student amp;stu)
    {
        cout << __FILE__ << " " << __LINE__ << " Student(const Student amp;stu)" << endl;
        name = stu.name;
        addresses = stu.addresses;
    }

    void print()
    {
        cout << name << " " << addresses << " size=" << addresses->size() << endl;
        if (addresses != nullptr)
        {
            for (int i = 0; i < addresses->size(); i  )
            {
                cout << i << " " << (*addresses)[i] << endl;
            }
        }
        cout << "----------------n";
    }
};

struct Printer //ToyView
{
    vector<Student *> students;
    vector<vector<string>> stu_addresses;
    void add_student()
    {
        for (int i = 0; i < 10; i  )
        {
            vector<string> address;
            address.push_back("addr1");
            address.push_back("addr2");
            stu_addresses.push_back(address);
            Student *stu = new Student;
            students.push_back(stu);
            students[i]->name = "stu"   to_string(i);
            students[i]->addresses = amp;stu_addresses[i];
            cout << i << " stu->address=" << stu->addresses << " size=" << stu->addresses->size() << endl;
        }
    }

    void print()
    {
        cout << " students size=" << students.size() << endl;
        for (int i = 0; i < students.size(); i  )
        {
            cout << i << " stu->address=" << students[i]->addresses << endl;
            // students[i]->addresses = amp;stu_addresses[i];
            students[i]->print();
        }
    }
};

int main()
{
    Printer printer;
    printer.add_student();
    printer.print();
    return 0;
}

Вот некоторый результат:

 ../vector_cp.cpp 13 Student()
0 stu->address=0x563888ac6f30 size=2
../vector_cp.cpp 13 Student()
1 stu->address=0x563888ac7418 size=2
../vector_cp.cpp 13 Student()
2 stu->address=0x563888ac74f0 size=2
../vector_cp.cpp 13 Student()
3 stu->address=0x563888ac7508 size=2
../vector_cp.cpp 13 Student()
4 stu->address=0x563888ac76c0 size=2
../vector_cp.cpp 13 Student()
5 stu->address=0x563888ac76d8 size=2
../vector_cp.cpp 13 Student()
6 stu->address=0x563888ac76f0 size=2
../vector_cp.cpp 13 Student()
7 stu->address=0x563888ac7708 size=2
../vector_cp.cpp 13 Student()
8 stu->address=0x563888ac7a10 size=2
../vector_cp.cpp 13 Student()
9 stu->address=0x563888ac7a28 size=2
 students size=10
0 stu->address=0x563888ac6f30
stu0 0x563888ac6f30 size=18446744073709549283

Я ожидаю, что students[0]-> адреса.size() должен быть 2, но он показывает номер мусора.
После раскомментирования этой строки
// students[i]->адреса = amp;stu_addresses[i];
Код на самом деле работает, то есть students[0]->addresses.size() .
Я все еще не могу понять, почему это работает. Кто-нибудь может помочь?

Комментарии:

1. почему это vector<string> *addresses не std::vector<string> так ?

2. Я не знаю, является ли этот указатель причиной вашей проблемы, но определенно это делает код намного сложнее, чем необходимо. Например addresses = stu.addresses; , в конструкторе копирования есть рецепт для удаления (вы просто не замечаете этого, потому что у вас происходит утечка памяти)

3. Вы сохраняете адреса векторных элементов. Если вектор будет изменен, он может перераспределиться, и сохраненные указатели будут признаны недействительными. В современном C вы хотите максимально избегать необработанных указателей.

Ответ №1:

 stu_addresses.push_back(address);
// ...
students[i]->addresses = amp;stu_addresses[i];

Возврат элементов в stu_addresses может перераспределить ваш вектор, и ранее действительные указатели / итераторы на stu_addresses элементах будут признаны недействительными.

Ср. https://en.cppreference.com/w/cpp/container#Iterator_invalidation