Модераторы: Daevaorn

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Ошибка внутри класса 
V
    Опции темы
Voldemar2004
  Дата 25.1.2010, 17:24 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
Есть класс:

Код

#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 

Это сообщение отредактировал(а) Voldemar2004 - 25.1.2010, 17:34


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
NewDima
Дата 25.1.2010, 17:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 922
Регистрация: 20.2.2006
Где: <?here?>

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


--------------------
Начинаем черкать!
PM ICQ   Вверх
Voldemar2004
  Дата 25.1.2010, 17:42 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
NewDima, отдельно ф-ция 
Код

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

Это сообщение отредактировал(а) Voldemar2004 - 25.1.2010, 17:43


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
comcon1
Дата 25.1.2010, 17:45 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 838
Регистрация: 11.6.2005
Где: Москва ДАС-МГУ

Репутация: 12
Всего: 17
Вот скажите, нахрена:

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

??


--------------------
user posted image
comcon1 on twitter
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 17:47 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
comcon1, 

Код


void SelectTable::ReadFileToArray(string FileName) {

n = StrCount(FileName);

array = new string[n];



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


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
comcon1
Дата 25.1.2010, 17:49 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 838
Регистрация: 11.6.2005
Где: Москва ДАС-МГУ

Репутация: 12
Всего: 17
Сделай пжлст printf(Filename.c_str()); перед вызовом


--------------------
user posted image
comcon1 on twitter
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 17:52 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
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


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 18:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
Ошибка только, когда идет присвоение 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 


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
comcon1
Дата 25.1.2010, 18:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 838
Регистрация: 11.6.2005
Где: Москва ДАС-МГУ

Репутация: 12
Всего: 17
n = StrCount(FileName.c_str()); - убери c_str, у тебя же функция принимает string


--------------------
user posted image
comcon1 on twitter
PM MAIL   Вверх
Voldemar2004
Дата 25.1.2010, 18:24 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

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


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


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
comcon1
Дата 25.1.2010, 18:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 838
Регистрация: 11.6.2005
Где: Москва ДАС-МГУ

Репутация: 12
Всего: 17
ну а ничо, что ты конвертируешь signed long int в int, но там такой ошибки не должно быть. У тебя дебаггер в функцию входит?


--------------------
user posted image
comcon1 on twitter
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 18:33 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
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 


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
casey
Дата 25.1.2010, 19:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 74
Регистрация: 20.12.2006
Где: Украина

Репутация: 1
Всего: 2
поменяй       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
PM MAIL WWW   Вверх
Voldemar2004
Дата 25.1.2010, 19:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
casey, ерунда - я даже удалил

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

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


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 25.1.2010, 19:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
похоже что где-то происходит порча кучи.


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 20:08 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
Как бы проверить ? :( Может переустановка Windows и C++ Builder 6.6 поможет ? smile


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 25.1.2010, 20:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Цитата(Voldemar2004 @  25.1.2010,  20:08 Найти цитируемый пост)
Может переустановка Windows и C++ Builder 6.6 поможет ?

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

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


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
casey
Дата 25.1.2010, 20:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 74
Регистрация: 20.12.2006
Где: Украина

Репутация: 1
Всего: 2
сорри, тупонул

Это сообщение отредактировал(а) casey - 25.1.2010, 20:15
PM MAIL WWW   Вверх
andrew_121
Дата 25.1.2010, 20:19 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Цитата(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();
};



--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
  Дата 25.1.2010, 20:31 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
Цитата(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;
}

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



Это сообщение отредактировал(а) Voldemar2004 - 25.1.2010, 20:31


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 25.1.2010, 20:34 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Voldemar2004, это гуд. а теперь сделай как я показал, и избавишься от лишнего метода. и никакой динамической памяти, ошибок, и остальной каки  smile 


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
casey
Дата 25.1.2010, 20:39 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 74
Регистрация: 20.12.2006
Где: Украина

Репутация: 1
Всего: 2
Код


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 , я ж написал там
                };
        }

}


Это сообщение отредактировал(а) casey - 25.1.2010, 20:50
PM MAIL WWW   Вверх
andrew_121
Дата 25.1.2010, 20:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
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;
}



--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 25.1.2010, 20:57 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
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 


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 25.1.2010, 20:59 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Цитата(Voldemar2004 @  25.1.2010,  20:57 Найти цитируемый пост)
SelectTable *My;

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

SelectTable *My = new SelectTable;


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

SelectTable My;


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

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


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
casey
Дата 25.1.2010, 21:04 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 74
Регистрация: 20.12.2006
Где: Украина

Репутация: 1
Всего: 2
Еще раз, нужно память выделить:

Код

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;
}
PM MAIL WWW   Вверх
Voldemar2004
  Дата 25.1.2010, 21:14 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
Цитата(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 ? Какой там метод ?

Это сообщение отредактировал(а) Voldemar2004 - 25.1.2010, 21:16


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 25.1.2010, 21:23 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Цитата(Voldemar2004 @  25.1.2010,  21:14 Найти цитируемый пост)
это где ?

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


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

Код

std::string str = array[idx];



--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 25.1.2010, 23:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
andrew_121, хотя интересно, в чем же была проблема ?  smile Глючность C++ Builder'a ?


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
Леопольд
Дата 26.1.2010, 09:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 943
Регистрация: 17.6.2009

Репутация: 10
Всего: 13
Цитата(Voldemar2004 @  25.1.2010,  23:21 Найти цитируемый пост)
andrew_121, хотя интересно, в чем же была проблема ?   Глючность C++ Builder'a ? 

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


--------------------
вопросов больше чем ответов
PM MAIL   Вверх
mes
Дата 26.1.2010, 09:47 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

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

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

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

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


--------------------
http://opendots.net
PM MAIL WWW   Вверх
Леопольд
Дата 26.1.2010, 13:30 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 943
Регистрация: 17.6.2009

Репутация: 10
Всего: 13
Цитата(mes @  26.1.2010,  09:47 Найти цитируемый пост)
адрес был не 0    


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

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

Это сообщение отредактировал(а) Леопольд - 26.1.2010, 13:31


--------------------
вопросов больше чем ответов
PM MAIL   Вверх
mes
Дата 26.1.2010, 13:37 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250
 smile 
Цитата(Леопольд @  26.1.2010,  12:30 Найти цитируемый пост)
"Write of address 00000000"

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




--------------------
http://opendots.net
PM MAIL WWW   Вверх
Voldemar2004
  Дата 26.1.2010, 20:23 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 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



--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 26.1.2010, 20:40 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
я бы сделал так:
Код

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()


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 26.1.2010, 20:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
andrew_121, где почитать, чтобы толково объяснено было про вектора:

  
Код

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


 smile 


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 26.1.2010, 20:51 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

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

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

Это сообщение отредактировал(а) andrew_121 - 26.1.2010, 20:51


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
mes
Дата 26.1.2010, 20:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 144
Всего: 250
Цитата(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 





--------------------
http://opendots.net
PM MAIL WWW   Вверх
andrew_121
Дата 26.1.2010, 20:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
по идее кол-во строк можно посчитать так:
Код

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

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

Это сообщение отредактировал(а) andrew_121 - 26.1.2010, 22:01


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 26.1.2010, 22:10 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
andrew_121, у меня пишет "Ошибка синтаксиса" даже не компилится...


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 26.1.2010, 22:54 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Voldemar2004, покажи вывод компилятора.


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 26.1.2010, 23:03 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

Репутация: нет
Всего: 23
andrew_121, да пишет "Ошибка синтаксиса" и все... ругается на: 

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


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




--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
andrew_121
Дата 26.1.2010, 23:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Кодофей
****


Профиль
Группа: Завсегдатай
Сообщений: 3448
Регистрация: 3.1.2008

Репутация: 6
Всего: 33
Цитата(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>


--------------------
Удалил аккаунт. Прощайте!
PM MAIL   Вверх
Voldemar2004
Дата 26.1.2010, 23:20 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Завсегдатай
Сообщений: 1650
Регистрация: 25.12.2004

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


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

Это сообщение отредактировал(а) Voldemar2004 - 26.1.2010, 23:21


--------------------
i_i 
(';') 
(V)

user posted image
PM MAIL   Вверх
Страницы: (3) [Все] 1 2 3 
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "С++:Общие вопросы"
Earnest Daevaorn

Добро пожаловать!

  • Черновик стандарта C++ (за октябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика(4.4мб).
  • Черновик стандарта C (за сентябрь 2005) можно скачать с этого сайта. Прямая ссылка на файл черновика (3.4мб).
  • Прежде чем задать вопрос, прочтите это и/или это!
  • Здесь хранится весь мировой запас ссылок на документы, связанные с C++ :)
  • Не брезгуйте пользоваться тегами [code=cpp][/code].
  • Пожалуйста, не просите написать за вас программы в этом разделе - для этого существует "Центр Помощи".
  • C++ FAQ

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, Earnest Daevaorn
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | C/C++: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0990 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.