Чтение из бинарного файла - неведомое исключение

@guest658655 · Регистрация: 01.11.2012

Author24 — интернет-сервис помощи студентам

Добрый день.
Помоги понять почему вылетает с ошибкой:
Необработанное исключение в "0x5a93ad54 (msvcp100d.dll)" в "test_conlose.exe": 0xC0000005: Нарушение прав доступа при записи "0xfeeefeee".

C++

#include <iostream>
#include <string>
#include <fstream>
 
using namespace std;
 
int main()
{
    string s1("111111"), s2("2222");
 
    ofstream save( "data.bin", ios::binary | ios::out );
 
    if( save ) 
    {
        save.write( reinterpret_cast <char *> ( &s1 ), sizeof( string ) ); 
        save.close();
    }
    else system("pause");
 
    ifstream load( "data.bin", ios::binary );
 
    if( load )
    {
        load.read( reinterpret_cast <char *> ( &s2 ), sizeof( string ) );
        load.close();
    }
 
    cout << s2 << "\n";
 
    system("pause");
    return 0;
}

Если стринг заменить на инт, то не вылетает. Объясните пожалуйста почему.
Просьба предлагать только решения на c++, через потоки.

@Tiva · 01.11.2012, 17:27

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

@guest658655 · 01.11.2012, 17:34 **[ТС]**

в visual studio 2010 вылетает с исключением :/

@Afflicted · 01.11.2012, 18:06

http://www.microsoft.com/en-us... px?id=5555

Необходимо для запуска программ, скомпилированных в MVS_2010

@guest658655 · 01.11.2012, 21:43 **[ТС]**

Сомневаюсь, что проблема в этом, так как всё остальное работает прекрасно.

Добавлено через 1 час 42 минуты
Избавился от проблемы так:

C++

int main()
{
    string s1("111111"), s2("2222");
 
    ofstream save( "data.bin", ios::out | ios::binary );
 
    if( save ) 
    {
        save.write( reinterpret_cast <char *> ( &s1 ), sizeof( string ) );
        save.close();
    }
 
    ifstream load( "data.bin", ios::binary );
 
    string *s3 = new string;
 
    if( load )
    {
        load.read( reinterpret_cast <char *> ( s3 ), sizeof( string ) );
        load.close();
    }
 
    cout << *s3 << "\n";
 
//  delete s3;
 
    system("pause");
    return 0;
}

но, теперь есть ещё 1 интересный момент, если открыть комент, опять лезет та ошибка

объясните, если не сложно, почему так.

@soon · 01.11.2012, 22:20

Сообщение от guest658655

Избавился от проблемы так:

Ни черта вы от нее не избавились.

C++

#include <iostream>
#include <string>
#include <cstring>
 
int main()
{
    std::string str1("11111"), str2("222");
 
    std::cout << (std::memcmp(reinterpret_cast<char*>(&str2), reinterpret_cast<char*>(&str1), sizeof(std::string)) == 0) << std::endl;
 
    std::memcpy(reinterpret_cast<char*>(&str2), reinterpret_cast<char*>(&str1), sizeof(std::string));
 
    std::cout << (std::memcmp(reinterpret_cast<char*>(&str2), reinterpret_cast<char*>(&str1), sizeof(std::string)) == 0) << std::endl;
 
    return 0;
}

Суть ясна?

alsav22 · 01.11.2012, 22:28

C++

#include <iostream>
#include <string>
#include <fstream>
using namespace std;
 
int main()
{
    string s1("111111"), s2("2222");
    
    ofstream save("data.bin", ios::binary | ios::out);
 
    if(save) 
    {
        save.write(s1.c_str(), s1.size()); 
        save.close();
    }
    else system("pause");
 
    ifstream load( "data.bin", ios::binary );
    if (load)
    {
        load.read((char*)s2.c_str(), s2.size() );
        load.close();
    }
 
    cout << s2 << "\n";
 
    system("pause");
    return 0;
}

@soon · 01.11.2012, 22:37

Сообщение от alsav22

load.read((char*)s2.c_str(), s2.size() );

За это, кстати, тоже руки отрывать не мешало бы.

alsav22 · 01.11.2012, 23:05

За это, за что?

@soon · 01.11.2012, 23:15

alsav22, как минимум за снятие константности.

alsav22 · 01.11.2012, 23:18

Сообщение от soon

alsav22, как минимум за снятие константности.

А как максимум? И чем, в данном случае, грозит снятие константности?

@soon · 01.11.2012, 23:24

Сообщение от alsav22

И чем, в данном случае, грозит снятие константности?

Запись в область памяти, расположенной по указателю c_str - UB.

alsav22 · 01.11.2012, 23:28

Если так:

C++

#include <iostream>
#include <string>
#include <fstream>
using namespace std;
 
int main()
{
    string s1("111111"), s2("2222");
    
    ofstream save("data.bin", ios::binary | ios::out);
 
    if(save) 
    {
        
        for (int i = 0; i < s1.size(); i++)
            save.write(&s1[i], 1); 
        save.close();
    }
    else system("pause");
 
    ifstream load( "data.bin", ios::binary );
    if (load)
    {
        for (int i = 0; i < s2.size(); i++)
             load.read(&s2[i], s2.size() );
        load.close();
    }
 
    cout << s2 << "\n";
 
    system("pause");
    return 0;
}

@soon · 01.11.2012, 23:36

Поторопились. Считывать надо 1 байт, а не s2.size(). Запись в файл можно было не менять, write принимает const char*.

alsav22 · 01.11.2012, 23:42

Исправляю:

C++

#include <iostream>
#include <string>
#include <fstream>
using namespace std;
 
int main()
{
    string s1("111111"), s2("2222");
    
    ofstream save("data.bin", ios::binary | ios::out);
 
    
    if(save) 
    {
        save.write(s1.c_str(), s1.size()); 
        save.close();
    }
    else system("pause");
 
    ifstream load( "data.bin", ios::binary );
    if (load)
    {
        for (int i = 0; i < s2.size(); i++)
             load.read(&s2[i], 1);
        load.close();
    }
 
    cout << s2 << "\n";
 
    system("pause");
    return 0;
}

@guest658655 · 02.11.2012, 16:52 **[ТС]**

Сообщение от soon

Суть ясна?

Меняются местами блоки памяти, это вы имели ввиду?

Этот код выдаёт ту же ошибку.

alsav22, дело в том, что я взял стринг, как пример, мне требуется считать свою структуру. Но всёравно спасибо, это хотябы решает проблему со стрингом.

@soon · 02.11.2012, 17:04

guest658655, не меняются, а становятся одинаковыми. Как следствие, идет повторное освобождение памяти.

Добавлено через 2 минуты

Этот код выдаёт ту же ошибку.

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

@guest658655 · 02.11.2012, 18:30 **[ТС]**

C++

1	std::memcpy(reinterpret_cast<char>(&str2), reinterpret_cast<char>(&str1), sizeof(std::string));

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

Добавлено через 5 минут
Тоесть когда у меня в s3 копируются байты из файла, память по адресу s3 как то удаляется, и поэтому команда delete s3 выдаёт ошибку?

alsav22 · 02.11.2012, 19:54

Сообщение от guest658655

я взял стринг, как пример, мне требуется считать свою структуру

Советую, в этом случае, пробовать код со своей структурой. Пример вы очень неудачно выбрали: строки, инициализированные литералами, и бинарное чтение/запись, используя адреса этих строк. Тут ошибки по разным причинам могут происходить.

@soon · 02.11.2012, 20:09

Сообщение от guest658655

если не сложно поясните подробней, где тут повторное освобождение памяти?

Разберитесь с классами и динамическим выделением памяти.

Сообщение от alsav22

Тут ошибки по разным причинам могут происходить.

Тут ошибка происходит по одной причине - после считывания/копирования строки, charT* в std::string указывает на одну область памяти.

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	01.11.2012, 23:36	14
	Поторопились. Считывать надо 1 байт, а не s2.size(). Запись в файл можно было не менять, write принимает const char*. 0

@Tiva 96 / 96 / 21 Регистрация: 25.04.2012 Сообщений: 449
	01.11.2012, 17:27	2
	у меня отлично работает этот код, компилил в билдере 0

@guest658655 0 / 0 / 0 Регистрация: 01.11.2012 Сообщений: 8
	01.11.2012, 17:34 [ТС]	3
	в visual studio 2010 вылетает с исключением :/ 0

@Afflicted Обитатель форума 201 / 184 / 54 Регистрация: 28.10.2012 Сообщений: 543
	01.11.2012, 18:06	4
	http://www.microsoft.com/en-us... px?id=5555 Необходимо для запуска программ, скомпилированных в MVS_2010 0

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	01.11.2012, 22:37	8
	Сообщение от alsav22 load.read((char*)s2.c_str(), s2.size() ); За это, кстати, тоже руки отрывать не мешало бы. 1

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	01.11.2012, 23:05	9
	За это, за что? 1

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	01.11.2012, 23:15	10
	alsav22, как минимум за снятие константности. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	01.11.2012, 23:18	11
	Сообщение от soon alsav22, как минимум за снятие константности. А как максимум? И чем, в данном случае, грозит снятие константности? 0

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	01.11.2012, 23:24	12
	Сообщение от alsav22 И чем, в данном случае, грозит снятие константности? Запись в область памяти, расположенной по указателю c_str - UB. 1

@guest658655 0 / 0 / 0 Регистрация: 01.11.2012 Сообщений: 8
	02.11.2012, 16:52 [ТС]	16
	Сообщение от soon Суть ясна? Меняются местами блоки памяти, это вы имели ввиду? Этот код выдаёт ту же ошибку. alsav22, дело в том, что я взял стринг, как пример, мне требуется считать свою структуру. Но всёравно спасибо, это хотябы решает проблему со стрингом. 0

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	02.11.2012, 17:04	17
	guest658655, не меняются, а становятся одинаковыми. Как следствие, идет повторное освобождение памяти. Добавлено через 2 минуты Этот код выдаёт ту же ошибку. Разумеется, он будет выдавать ошибку, я просто сократил ваш код, убрав оттуда использование файлов. 0

alsav22 5498 / 4893 / 831 Регистрация: 04.06.2011 Сообщений: 13,587
	02.11.2012, 19:54	19
	Сообщение от guest658655 я взял стринг, как пример, мне требуется считать свою структуру Советую, в этом случае, пробовать код со своей структурой. Пример вы очень неудачно выбрали: строки, инициализированные литералами, и бинарное чтение/запись, используя адреса этих строк. Тут ошибки по разным причинам могут происходить. 0

@soon 2554 / 1319 / 178 Регистрация: 09.05.2011 Сообщений: 3,086 Записей в блоге: 1
	02.11.2012, 20:09	20
	Сообщение от guest658655 если не сложно поясните подробней, где тут повторное освобождение памяти? Разберитесь с классами и динамическим выделением памяти. Сообщение от alsav22 Тут ошибки по разным причинам могут происходить. Тут ошибка происходит по одной причине - после считывания/копирования строки, charT* в std::string указывает на одну область памяти. 1