| Цитата(xvr @ 15.12.2009, 11:34) | В коде КУЧА ошибок (но не с tapes++)
| Код |
int main(int argc, char *argv[])
{
char *str = new char[1024]; <<< *2
// Счетчик цикла
int i = 0;
// Цикл "пока в потоке есть данные"
while(file.good())
{
// Читаем из потока одну строку и записываем ее в str.
// Если в строке больше чем 1024 символа, она обрезается до 1024
file.getline(str, 1024);
tapes[i].parse(str);
}
!!! Баг №1 - i в цикле не инкрементируется - все данные будут сложенны в tapes[0]
tapes++; // <<< *1
// Освобождаем память, выделенную под массив из n объектов
delete [] tapes;
!!! Баг №2 - Указатель tapes передвинут на 2й элемент массива, который вернул new. Делать к нему delete нельзя (см *1)
return 0;
}
Videotape::Videotape()
{
// Выделяем память под поля класса
nazvanie = new char[255];
regisser = new char[50];
kinokompaniya = new char[50];
producer = new char[50];
annotation = new char[255];
}
!!! Баг №3 - Поле nomer не проинициализированно, а вы его пытаетесь печатать (после tapes++)
void Videotape::parse(char *str)
{
// Разбиваем переданную сюда строку по разделителю
// и записываем полученные подстроки в соответствующие поля класса
nomer = atoi(strtok(str, "::"));
nazvanie = strtok(NULL, "::");
regisser = strtok(NULL, "::");
kinokompaniya = strtok(NULL, "::");
year = atoi(strtok(NULL, "::"));
producer = strtok(NULL, "::");
annotation = strtok(NULL, "::");
}
!!! Баг № 4 и 5 - Все поля, выделенные через new char[] переписанны значениями из strtok, которые в свою очередь указывают на один и тот же буфер, выделенные с помощью new char [] в main (*2).
Итого - memory leak + все экземпляры Videotape ссылаются на одну и ту же строку (которую многократно переписали в main)
Videotape::~Videotape()
{
// Освобождаем память
delete nazvanie;
delete regisser;
delete kinokompaniya;
delete producer;
delete annotation;
}
!!! Баг №6 (следствие бага №5) - делать delete для этих указателей нельзя - они все ссылаются на внутренности *2 из main
|
|
Как тогда сделать правильно, подскажите, пожалуйста. Что еще нужно тут изменить?
| Код |
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <iostream>
#include <fstream>
#include <math.h>
#include "main.h"
using namespace std;
int main(int argc, char *argv[])
{
// По заданию число сортируемых элементов, т.е. строк в файле, заранее известно.
// Будем хранить это значение в переменной n
int n = 7;
// Создаем новую переменную, которая будет хранить текущую прочитанную из файла строку
// и выделяем под нее 1024 байта
char *str = new char[1024];
// Создаем указатель на массив объектов типа Videotape, и выделяем память под n элементов
Videotape *tapes = new Videotape[n];
// Создаем новый поток, который открывает файл на чтение
ifstream file("data.txt", ios::in);
// Если файла нет, или его не удалось открыть, выводим ошибку
if (!file)
cout << "Can't open file!\n";
// Счетчик цикла
int i = 0;
// Цикл "пока в потоке есть данные"
while(file.good())
{
// Читаем из потока одну строку и записываем ее в str.
// Если в строке больше чем 1024 символа, она обрезается до 1024
file.getline(str, 1024);
tapes[i].parse(str);
i++;
}
//int y = sizeof(tapes[0]);
//tapes += sizeof(tapes[0]);
//tapes++;
//int mynum = tapes[0].getNomer();
//MergeSort(tapes, n);
// Закрываем поток и освобождаем выделенные ресурсы
file.close();
getch();
// Освобождаем память, выделенную под массив из n объектов
delete [] tapes;
return 0;
}
Videotape::Videotape()
{
nomer = 0;
year = 0;
// Выделяем память под поля класса
nazvanie = NULL;//new char[255];
regisser = NULL;//new char[50];
kinokompaniya = NULL;//new char[50];
producer = NULL;//new char[50];
annotation = NULL;//new char[255];
}
Videotape::~Videotape()
{
// Освобождаем память
delete nazvanie;
delete regisser;
delete kinokompaniya;
delete producer;
delete annotation;
}
void Videotape::parse(char *str)
{
// Разбиваем переданную сюда строку по разделителю
// и записываем полученные подстроки в соответствующие поля класса
nomer = atoi(strtok(str, "::"));
nazvanie = strtok(NULL, "::");
regisser = strtok(NULL, "::");
kinokompaniya = strtok(NULL, "::");
year = atoi(strtok(NULL, "::"));
producer = strtok(NULL, "::");
annotation = strtok(NULL, "::");
}
const int Videotape::getNomer()
{
return this->nomer;
}
|
|
)
