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

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Ошибка внутри класса 
V
    Опции темы
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.1817 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


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

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