0

Ну очень нужна помощь, я больше не могу, ну не понимаю и всё тут. Хочу циклом заполнять массив из структур. Создаю структуру, в ней два поля. Предположим, что первое поле - это int, а второе - массив char. В функции main сначала надо ввести количество структур в массиве, а потом циклом заполнять его. Моя функция заменяет все поля char на последнее введенное значение. Вот мой код.

struct Die {
    int sqrt = 0;
    char* str;
};

int main() {
    char* s = new char[10];
    Die* mas = new Die[10];
    int N;
    int j = 0;
    std::cin >> N;
    for (int i = 0; i < N;i++) {
        std::cin >> s;
        mas[i] = { j, s }; //заполняем структуру, где первое поле - индекс, а второе - строка
        std::cout << mas[i].sqrt << ' ' << mas[i].str << std::endl; //если выводим так, то все нормально
        j += 1;
    }
    std::cout << "_________" << std::endl;
    for (int i = 0; i < N; i++) {
        std::cout << mas[i].sqrt << ' ' << mas[i].str << std::endl; //а если вот так, то беда
    }
}

Вывод

3
aaa bbb ccc  
0 aaa 
1 bbb 
2 ccc                                                                                                                   _____________
 
0 ccc 
1 ccc
2 ccc 

Пожалуйста, помогите...

2
  • 1
    А почему бы не использовать string? C++ всё таки.
    – user361068
    31 окт 2020 в 9:40
  • Зачем вам этот char[10]?
    – vp_arth
    31 окт 2020 в 9:42

2 ответа 2

3

Ну Вы же один раз выделили память под строку, указатели во всех записях указывают на один адрес, и естественно, что там последнее назначенное лежит.

Делайте mas[i].str = new char[10]; для каждой структуры

8
  • Так тоже не работает, к сожалению.
    – Sergey
    31 окт 2020 в 9:35
  • Работает. Возможно, Вы по-прежнему назначаете одну строку полям всех структур. Читайте прямо в поле std::cin >>mas[i].str; или копируйте данные strcpy
    – MBo
    31 окт 2020 в 9:41
  • Исправил, но выводит совсем не то, что я вводил, а мусор какой-то:(
    – Sergey
    31 окт 2020 в 9:47
  • Отредактируйте свой пост, добавив в конце новый код
    – MBo
    31 окт 2020 в 9:49
  • Я все понял, надо было убрать строку s
    – Sergey
    31 окт 2020 в 9:54
0

у вас получается один указатель на массив букв (строку) для всех элементов массива, поскольку вы передаете в массив лишь указатель на строку

поэтому когда вы выводите содержимое массива уже после ввода всех значений, то вам выводится последний ввод

т.е. если образно - у вас куча народу и вместо того, чтобы каждому дать по коробке вы дали по планшету на который выводится видео из одной коробки и естественно все видят одно и тоже

вам надо каждый раз выделять память под строку и записывать строку

for (int i = 0; i < N;i++) {
    char* text= new char[10];
    std::cin >> text;
    mas[i] = { j, text}; //заполняем структуру, где первое поле - индекс, а второе - строка
    std::cout << mas[i].sqrt << ' ' << mas[i].str << std::endl; //если выводим так, то все нормально
    j += 1;
}

и вот тогда все будет работать

но лучше всего добавить в вашу структуру конструктор, чтобы наружи не делать лишних выделений памяти, а спрятать это всё вовнутрь структуры, например

struct Die {
    int sqrt = 0;
    char* str = nullptr;

    // конструктор
    Die(const int l_sqrt, const char* l_str) {
        sqrt = l_sqrt;
        str = new char[strlen(l_str) + 1];
        strcpy(str, l_str);
    }

    // деструктор
    ~Die() {
        delete [] str;
    }
};

и тогда можете использовать так:

for (int i = 0; i < N;i++) {
    std::cin >> s;
    mas[i] = Die(j, s); //заполняем структуру, где первое поле - индекс, а второе - строка
    std::cout << mas[i].sqrt << ' ' << mas[i].str << std::endl; //если выводим так, то все нормально
    j += 1;
}
4
  • А как в таком случае объявлять массив?
    – Sergey
    31 окт 2020 в 9:44
  • И куда же будет указывать mas[i].str после этой строчки: mas[i] = Die(j, s);?
    – wololo
    31 окт 2020 в 9:51
  • @wololo, на строку освобождённую после копирования деструктором временного объекта?
    – vp_arth
    31 окт 2020 в 10:08
  • @vp_arth, угу, свой оператор присваивания добавить в структуру не помешало бы.
    – wololo
    31 окт 2020 в 10:12

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.