Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > C/C++: Общие вопросы > Ошибка внутри класса


Автор: Voldemar2004 25.1.2010, 17:24
Есть класс:

Код

#include <string.h>
#include <fstream.h>
#include <windows.h>

//---------------------------------------------------------------------------

class SelectTable {

public:

signed long int StrCount (string FileName);

// Метод считывает весь файл в массив.
void ReadFileToArray(string FileName);

// Конструктор класса.
SelectTable (void);

// Деструктор класса.
~SelectTable (void);

private:

int n;

string *array;

};

//---------------------------------------------------------------------------

signed long int SelectTable::StrCount (string FileName) {

        string str;

        signed long int String_Counter = 0;

        ifstream File(FileName.c_str());

        if( File.fail() ) return -1;

            while( !File.eof() )
            {
             getline(File, str);
             ++String_Counter;
            };

return String_Counter;
};

//---------------------------------------------------------------------------

void SelectTable::ReadFileToArray(string FileName) {

n=StrCount(FileName);

array = new string[StrCount(FileName)];

        if (n > 0) {

                string str;
                // Чтение файла
                ifstream File(FileName.c_str());

                unsigned long int i=0;

                // Пока нет конца файла.
                while( !File.eof() )
                {
                // Считать в str
                getline(File, str);
                array[i++]=str.c_str();
                };
        }

}
//---------------------------------------------------------------------------

SelectTable::~SelectTable(void) {

if(n) delete[] array;

}

//---------------------------------------------------------------------------

// Конструктор класса.
SelectTable::SelectTable(void):n(0),array(NULL)
{
}

//---------------------------------------------------------------------------



Выдает ошибку:

user posted image

В методе

Код

 ReadFileToArray(string FileName);


ругается на

Код

StrCount(FileName);


Что не так ?  smile 

Автор: NewDima 25.1.2010, 17:34
Полагаю в выделенной строке ошибки не должно быть, по F7 зайдите в саму функцию, а вообще передается нормальный FileName?

Автор: Voldemar2004 25.1.2010, 17:42
NewDima, отдельно ф-ция 
Код

StrCount
 нормально работает - а внутри другого метода нет  smile 

Автор: comcon1 25.1.2010, 17:45
Вот скажите, нахрена:

n=StrCount(FileName);
array = new string[StrCount(FileName)];

??

Автор: Voldemar2004 25.1.2010, 17:47
comcon1, 

Код


void SelectTable::ReadFileToArray(string FileName) {

n = StrCount(FileName);

array = new string[n];



так лучше smile но не в этом ошибка smile это я просто для себя делал smile

Автор: comcon1 25.1.2010, 17:49
Сделай пжлст printf(Filename.c_str()); перед вызовом

Автор: Voldemar2004 25.1.2010, 17:52
comcon1, да это все нормально !

Я так делаю:

Код
  OpenDialog1->Title = "";

  if ( OpenDialog1->Execute() ) {
   if ( OpenDialog1->FileName != "" ) {

        ShowMessage( OpenDialog1->FileName.c_str() );

        MyTable->ReadFileToArray( OpenDialog1->FileName.c_str() );
        ShowMessage( MyTable->StrCount( OpenDialog1->FileName.c_str() ) );

    }

   }



Добавлено через 6 минут и 38 секунд
Среда IDE - С++ Builder 6.6

Автор: Voldemar2004 25.1.2010, 18:09
Ошибка только, когда идет присвоение n значения StrCount()  smile 

Код

void SelectTable::ReadFileToArray(string FileName) {

ShowMessage(FileName.c_str());

ShowMessage(StrCount(FileName.c_str()));

n = StrCount(FileName.c_str());

}



Вот где ошибка:

Код

n = StrCount(FileName.c_str());


и дальше не идет smile 

Автор: comcon1 25.1.2010, 18:18
n = StrCount(FileName.c_str()); - убери c_str, у тебя же функция принимает string

Автор: Voldemar2004 25.1.2010, 18:24
Цитата(comcon1 @  25.1.2010,  19:18 Найти цитируемый пост)
убери c_str, у тебя же функция принимает string


comcon1, и так и так пробовал - убирал - та же ошибка smile Ошибка не имени файла smile А что-то с присвоением  smile 

Автор: comcon1 25.1.2010, 18:28
ну а ничо, что ты конвертируешь signed long int в int, но там такой ошибки не должно быть. У тебя дебаггер в функцию входит?

Автор: Voldemar2004 25.1.2010, 18:33
comcon1, да я уже привел все к одному типу int

Добавлено через 10 минут и 1 секунду
Код


void SelectTable::ReadFileToArray(string FileName) {

ShowMessage(FileName.c_str()); // - Нет ошибки

ShowMessage(StrCount(FileName.c_str())); // - Нет ошибки

StrCount(FileName.c_str()); // - Нет ошибки

n = StrCount(FileName.c_str());  // - ЕСТЬ ОШИБКА !

}



 smile 

Автор: casey 25.1.2010, 19:16
поменяй       if( File.fail() ) return -1;
на                 if( File.fail() ) return 0;
потому что   n=StrCount(FileName);                                   (т.е. -1 при невалидном файле)
                     array = new string[StrCount(FileName)];        (array = new string[-1])
и потом в деструкторе if(n) delete[] array;   -  Access Violation

Автор: Voldemar2004 25.1.2010, 19:39
casey, ерунда - я даже удалил

 
Код
if( File.fail() ) return -1;
 

все равно та же ошибка.... может C++ Builder тупит ?

Автор: andrew_121 25.1.2010, 19:58
похоже что где-то происходит порча кучи.

Автор: Voldemar2004 25.1.2010, 20:08
Как бы проверить ? :( Может переустановка Windows и C++ Builder 6.6 поможет ? smile

Автор: andrew_121 25.1.2010, 20:12
Цитата(Voldemar2004 @  25.1.2010,  20:08 Найти цитируемый пост)
Может переустановка Windows и C++ Builder 6.6 поможет ?

нет.  smile 
тот код что ты показал, весь? ничего не скрыл?  smile

Добавлено через 38 секунд
покажи теперешний код, после всех поправок.

Автор: casey 25.1.2010, 20:14
сорри, тупонул

Автор: andrew_121 25.1.2010, 20:19
Цитата(casey @  25.1.2010,  20:14 Найти цитируемый пост)
array[i++]=str.c_str();      // тут поменяй i++  на ++i  - выходишь за границы и c_str не нужно

а почему бы просто не написать по нормальному? ;)

Код

...
std::vector<std::string> array;
...
size_t SelectTable::StrCount (string FileName) {
        string str;
        ifstream File(FileName.c_str());
        if( File.fail() ) return -1;
array.clear();
            while( !File.eof() )
            {
             getline(File, str);
             array.push_back(str);
            };
return array.size();
};

Автор: Voldemar2004 25.1.2010, 20:31
Цитата(andrew_121 @  25.1.2010,  21:12 Найти цитируемый пост)
покажи теперешний код, после всех поправок.


Код


//---------------------------------------------------------------------------

#include <string.h>
#include <fstream.h>
#include <windows.h>

#include <vcl.h>
#pragma hdrstop

#pragma argsused

//---------------------------------------------------------------------------

class SelectTable {

public:

int StrCount (string FileName);

void ReadFileToArray(string FileName);

SelectTable (void);

~SelectTable (void);

private:

int n;

string *array;

};

//---------------------------------------------------------------------------

int SelectTable::StrCount (string FileName) {

        string str;

        int String_Counter = 0;

        ifstream File(FileName.c_str());

            while( !File.eof() )
            {
             getline(File, str);
             ++String_Counter;
            };

return String_Counter;
};

//---------------------------------------------------------------------------

void SelectTable::ReadFileToArray(string FileName) {

n=StrCount(FileName);

array = new string[n];

        if (n > 0) {

                string str;
                ifstream File(FileName.c_str());

                int i=0;

                while( !File.eof() )
                {
                getline(File, str);
                array[++i]=str;
                };
        }

}
//---------------------------------------------------------------------------

SelectTable::~SelectTable(void) {

if(n) delete[] array;

}

//---------------------------------------------------------------------------

SelectTable::SelectTable(void):n(0),array(NULL)
{
}

//---------------------------------------------------------------------------

int main(int argc, char* argv[])
{

SelectTable *My;

My->ReadFileToArray("C:\\file.TXT");

My->~SelectTable();

return 0;
}

//---------------------------------------------------------------------------


Автор: andrew_121 25.1.2010, 20:34
Voldemar2004, это гуд. а теперь сделай как я показал, и избавишься от лишнего метода. и никакой динамической памяти, ошибок, и остальной каки  smile 

Автор: casey 25.1.2010, 20:39
Код


int main(int argc, char* argv[])
{

SelectTable *My= new SelectTable();

My->ReadFileToArray("C:\\file.TXT");

delete My;

return 0;
}

ну ты даешь...

и тут еще поменяй
Код

void SelectTable::ReadFileToArray(string FileName) {

n=StrCount(FileName);

array = new string[n];

        if (n > 0) {

                string str;
                ifstream File(FileName.c_str());

                int i=0;

                while( !File.eof() )
                {
                getline(File, str);
                array[++i]=str;     на i++   -  тоя тупонул :crazy , я ж написал там
                };
        }

}

Автор: andrew_121 25.1.2010, 20:48
Voldemar2004, не понятно какой именно тайный смысл ты преследовал написав такой код, но вот эквивалентный код. помоему проще  smile 
Код

#include <string>
#include <vector>
#include <fstream>
#include <iostream>
#include <iterator>

int main() {
    std::ifstream file("main.cpp");
    std::vector<std::string> array((std::istream_iterator<std::string>(file)), (std::istream_iterator<std::string>()));
    std::cout << array.size();
    return 0;
}

Автор: Voldemar2004 25.1.2010, 20:57
andrew_121, 

Код


//---------------------------------------------------------------------------

#include <string.h>
#include <fstream.h>
#include <windows.h>
#include <vector.h>


#include <vcl.h>
#pragma hdrstop

#pragma argsused

//---------------------------------------------------------------------------

class SelectTable {

public:

size_t StrCount (string FileName);

SelectTable (void);

~SelectTable (void);

private:

std::vector<std::string> array;

int n;

};

//---------------------------------------------------------------------------

size_t SelectTable::StrCount (string FileName) {
        string str;
        ifstream File(FileName.c_str());
        if( File.fail() ) return -1;
array.clear();
            while( !File.eof() )
            {
             getline(File, str);
             array.push_back(str);
            };
return array.size();
};

//---------------------------------------------------------------------------

SelectTable::~SelectTable(void) {

}

//---------------------------------------------------------------------------

SelectTable::SelectTable(void):n(0),array(NULL)
{
}

//---------------------------------------------------------------------------

int main(int argc, char* argv[])
{

SelectTable *My;

My->StrCount("C:\\file.TXT");

return 0;
}

//---------------------------------------------------------------------------



не работает  smile 

Автор: andrew_121 25.1.2010, 20:59
Цитата(Voldemar2004 @  25.1.2010,  20:57 Найти цитируемый пост)
SelectTable *My;

должно быть так:
Код

SelectTable *My = new SelectTable;


Добавлено через 57 секунд
а лучше так:
Код

SelectTable My;


Добавлено через 1 минуту и 25 секунд
и стрелочки не забудь на точки поменять.

Добавлено через 1 минуту и 44 секунды
и на плюсик нажать smile 

Автор: casey 25.1.2010, 21:04
Еще раз, нужно память выделить:

Код

int main(int argc, char* argv[])
{

SelectTable *My= new SelectTable();

My->ReadFileToArray("C:\\file.TXT");

delete My;

return 0;
}



или

Код

int main(int argc, char* argv[])
{

SelectTable My;

My.ReadFileToArray("C:\\file.TXT");

return 0;
}


Автор: Voldemar2004 25.1.2010, 21:14
Цитата(andrew_121 @  25.1.2010,  21:59 Найти цитируемый пост)
должно быть так:

 smile  smile мда....  smile 
andrew_121, 

Код


int main(int argc, char* argv[])
{
SelectTable *My = new SelectTable;

My->StrCount("...");

ShowMessage();

return 0;
}



Так ? smile

Добавлено @ 21:15
Цитата(andrew_121 @  25.1.2010,  21:59 Найти цитируемый пост)
и на плюсик нажать  


это где ?  smile 

Как получить значение по индексу из array ? Какой там метод ?

Автор: andrew_121 25.1.2010, 21:23
Цитата(Voldemar2004 @  25.1.2010,  21:14 Найти цитируемый пост)
это где ?

там где повышение репутации, снизу под аватаром.


Цитата(Voldemar2004 @  25.1.2010,  21:14 Найти цитируемый пост)
Как получить значение по индексу из array ?

Код

std::string str = array[idx];

Автор: Voldemar2004 25.1.2010, 23:21
andrew_121, хотя интересно, в чем же была проблема ?  smile Глючность C++ Builder'a ?

Автор: Леопольд 26.1.2010, 09:16
Цитата(Voldemar2004 @  25.1.2010,  23:21 Найти цитируемый пост)
andrew_121, хотя интересно, в чем же была проблема ?   Глючность C++ Builder'a ? 

Был access violation, т.е. запись за границы приложения по адресу 0. Это, по любому, глючность приложения, а не компилятора... smile

Автор: mes 26.1.2010, 09:47
Цитата(Voldemar2004 @  25.1.2010,  22:21 Найти цитируемый пост)
andrew_121, хотя интересно, в чем же была проблема ?  smile Глючность C++ Builder'a ? 

при вызове метода this был не валидным - так как не создавали объекта.

Добавлено через 1 минуту и 51 секунду
Цитата(Леопольд @  26.1.2010,  08:16 Найти цитируемый пост)
т.е. запись за границы приложения по адресу 0. 

адрес был не 0  smile 

Автор: Леопольд 26.1.2010, 13:30
Цитата(mes @  26.1.2010,  09:47 Найти цитируемый пост)
адрес был не 0    


Цитата(Voldemar2004 @ 25.1.2010,  17:24)
user posted image

Это не я сказал smile "Write of address 00000000"

Автор: mes 26.1.2010, 13:37
 smile 
Цитата(Леопольд @  26.1.2010,  12:30 Найти цитируемый пост)
"Write of address 00000000"

нда.. на картинку то я и не посмотрел.. исходил из опыта о неинициализрованном указателе..
в любом случае,  что  адрес оказался 0 это случайность зависящая в частности от настроения (то бишь настроек)  компилятора.
 smile 


Автор: Voldemar2004 26.1.2010, 20:23
mes,

Код

void SelectTable::ReadFileToArray(string FileName) {

ShowMessage(FileName.c_str()); // - Нет ошибки

ShowMessage(StrCount(FileName.c_str())); // - Нет ошибки

StrCount(FileName.c_str()); // - Нет ошибки

n = StrCount(FileName.c_str());  // - ЕСТЬ ОШИБКА !

}


странно, что ошибку выдает только при присвоении !!

Сегодня еще поковырял свой код:

то, что было:

Код


//---------------------------------------------------------------------------

#include <string.h>
#include <fstream.h>
#include <windows.h>

#include <vcl.h>
#pragma hdrstop

#pragma argsused

//---------------------------------------------------------------------------

class SelectTable {

public:

int StrCount (string FileName);

void ReadFileToArray(string FileName);

SelectTable (void);

~SelectTable (void);

private:

int n;

string *array;

};

//---------------------------------------------------------------------------

int SelectTable::StrCount (string FileName) {

        string str;

        int String_Counter = 0;

        ifstream File(FileName.c_str());

            while( !File.eof() )
            {
             getline(File, str);
             ++String_Counter;
            };

return String_Counter;
};

//---------------------------------------------------------------------------

void SelectTable::ReadFileToArray(string FileName) {

n=StrCount(FileName);

array = new string[n];

        if (n > 0) {

                string str;
                ifstream File(FileName.c_str());

                int i=0;

                while( !File.eof() )
                {
                getline(File, str);
                array[++i]=str;
                };
        }

}
//---------------------------------------------------------------------------

SelectTable::~SelectTable(void) {

if(n) delete[] array;

}

//---------------------------------------------------------------------------

SelectTable::SelectTable(void):n(0),array(NULL)
{
}

//---------------------------------------------------------------------------

int main(int argc, char* argv[])
{

SelectTable *My;

My->ReadFileToArray("C:\\file.TXT");

My->~SelectTable();

return 0;
}

//---------------------------------------------------------------------------



Поменял всего в одной строке:

Код

array[++i]=str;


И заработало !  smile 

Код


//---------------------------------------------------------------------------

#include <string.h>
#include <fstream.h>
#include <windows.h>

#include <vcl.h>
#pragma hdrstop

#pragma argsused

//---------------------------------------------------------------------------

class SelectTable {

public:

int StrCount (string FileName);

void ReadFileToArray(string FileName);

void SaveToFile(string FileName);

SelectTable (void);

~SelectTable (void);

private:

int n;

string *array;

};

//---------------------------------------------------------------------------

int SelectTable::StrCount (string FileName) {

        string str;

        int String_Counter = 0;

        ifstream File(FileName.c_str());

            while( !File.eof() )
            {
             getline(File, str);
             ++String_Counter;
            };

return String_Counter;
};

//---------------------------------------------------------------------------

void SelectTable::ReadFileToArray(string FileName) {

n=StrCount(FileName);

array = new string[n];

        if (n > 0) {

                string str;
                ifstream File(FileName.c_str());

                int i=0;

                while( !File.eof() )
                {
                getline(File, str);
                array[i++]=str;
                };
        }

}
//---------------------------------------------------------------------------

SelectTable::~SelectTable(void) {

if(n) delete[] array;

}

//---------------------------------------------------------------------------

SelectTable::SelectTable(void):n(0),array(NULL)
{
}

//---------------------------------------------------------------------------

void SelectTable::SaveToFile(string FileName)
{
        std::ofstream file(FileName.c_str());
        for(unsigned int i=0; i<n; ++i)
        file << array[i] << '\n';
}

//---------------------------------------------------------------------------

int main(int argc, char* argv[])
{

SelectTable *My = new SelectTable;

My->ReadFileToArray("C:\\file");

My->SaveToFile("C:\\file 2");

return 0;
}

//---------------------------------------------------------------------------



Добавлено через 3 минуты и 6 секунд
Вот где была ошибка:

Код
SelectTable *My= new SelectTable;


надо было дописать:

Код
= new SelectTable;


забыл  smile Теперь и сам доехал  smile Всем спасибо smile

Еще метод добавил SaveToFile - чтобы проверить считывает ли в массив, нормально считал и записал в файл smile

Автор: andrew_121 26.1.2010, 20:40
я бы сделал так:
Код

class SelectTable {
    public:
        int StrCount (const std::string& FileName);
        void ReadFileToArray(const std::string& FileName);
        void SaveToFile(const std::string& FileName);
        SelectTable (void);
        ~SelectTable (void);
    private:
        string array;
};
//---------------------------------------------------------------------------
int SelectTable::StrCount (const std::string& FileName) {
    std::ifstream File(FileName.c_str());
    std::vector<std::string> sarr((std::istream_iterator<std::string>(File)),
                                            (std::istream_iterator<std::string>())
                                            );
    return sarr.size();
};
//---------------------------------------------------------------------------
void SelectTable::ReadFileToArray(const std::string& FileName) {
    array.clear();
    array.assign((std::istream_iterator<std::string>(File)),
                     (std::istream_iterator<std::string>())
                     );
}
//---------------------------------------------------------------------------
SelectTable::~SelectTable(void) {
}
//---------------------------------------------------------------------------
SelectTable::SelectTable(void) {
}
//---------------------------------------------------------------------------
void SelectTable::SaveToFile(const std::string& FileName) {
    std::ofstream File(FileName.c_str());
    if ( !file ) return;
    std::copy(array.begin(), array.end(), std::ostream_iterator<std::string>(File, "\n"));
}
//---------------------------------------------------------------------------
int main(int argc, char* argv[])
{
    SelectTable *My = new SelectTable;
    My->ReadFileToArray("C:\\file");
    My->SaveToFile("C:\\file 2");
    return 0;
}

 smile

Добавлено через 42 секунды
но в таком случае пропадает надобность в методе StrCount()

Автор: Voldemar2004 26.1.2010, 20:48
andrew_121, где почитать, чтобы толково объяснено было про вектора:

  
Код

sarr.size(), array.clear(), array.assign, array.begin


 smile 

Автор: andrew_121 26.1.2010, 20:51
Цитата(Voldemar2004 @  26.1.2010,  20:48 Найти цитируемый пост)
где почитать, чтобы толково объяснено было про вектора:

где и всегда: http://www.cplusplus.com/reference/
весь STL описан. даже примеры почти ко всему.

Автор: mes 26.1.2010, 20:54
Цитата(Voldemar2004 @  26.1.2010,  19:23 Найти цитируемый пост)
Вот где была ошибка:
SelectTable *My= new SelectTable;

Ну так все кроме Вас уже это давно знали..  smile   и Вам еще вначале 2й страницы сообщили:
Цитата(casey @  25.1.2010,  19:39 Найти цитируемый пост)
SelectTable *My= new SelectTable();

 smile

 
Цитата(Voldemar2004 @  26.1.2010,  19:23 Найти цитируемый пост)
странно, что ошибку выдает только при присвоении !!

пока this не используется  чего ошибками то кидать, а как попытались записать (n член класса, а значит доступ через this)
так сразу и обнаружилась невалидность указателя.
 smile 



Автор: andrew_121 26.1.2010, 20:58
по идее кол-во строк можно посчитать так:
Код

std::ifstream file("main.cpp");
int count = std::distance((std::istream_iterator<std::string>(file)),
                                  (std::istream_iterator<std::string>())
                                  );

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

Автор: Voldemar2004 26.1.2010, 22:10
andrew_121, у меня пишет "Ошибка синтаксиса" даже не компилится...

Автор: andrew_121 26.1.2010, 22:54
Voldemar2004, покажи вывод компилятора.

Автор: Voldemar2004 26.1.2010, 23:03
andrew_121, да пишет "Ошибка синтаксиса" и все... ругается на: 

Код
int count = std::distance((std::istream_iterator<std::string>(file)),
                                  (std::istream_iterator<std::string>())
                                  );


у меня C++ Builder, а ты в какой IDE ?


Автор: andrew_121 26.1.2010, 23:12
Цитата(Voldemar2004 @  26.1.2010,  23:03 Найти цитируемый пост)
"Ошибка синтаксиса" и все...

борланд славится своими падениями на шаблонах. может внутренняя ошибка компилятора?


Цитата(Voldemar2004 @  26.1.2010,  23:03 Найти цитируемый пост)
у меня C++ Builder, а ты в какой IDE ?

у меня нет IDE smile текстовый редактор, консоль, компилятор. linux-gcc

Добавлено через 1 минуту и 51 секунду
попробуй убери "std::"
где-то видел что у борланда пространство имен std открыто по умолчанию.

Добавлено через 2 минуты и 50 секунд
кстати, добавь: #include <algorithm>

Автор: Voldemar2004 26.1.2010, 23:20
Цитата(andrew_121 @  27.1.2010,  00:12 Найти цитируемый пост)
где-то видел что у борланда пространство имен std открыто по умолчанию.


завтра попробую smile Да Борланд (теперь уже CodeGear) глючные делает IDE... И цена неадекватная просто... и поддержка. Благо нам, студентам универа бесплатно MicroSoft дает VisualStudio и Windows  smile 

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)