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

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

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

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

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
#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++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
#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 указывает на одну область памяти.

Новые блоги и статьи Все статьи Все блоги /
Программный контроль заполнения реквизита табличной части документа Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: реализовать контроль заполнения реквизита табличной части. . .	wmic не является внутренней или внешней командой Maks 02.04.2026 Решение: DISM / Online / Add-Capability / CapabilityName:WMIC~~~~ Отсюда: https:/ / winitpro. ru/ index. php/ 2025/ 02/ 14/ komanda-wmic-ne-naydena/	Программная установка даты и запрет ее изменения Maks 02.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "СписаниеМатериалов", разработанного в конфигурации КА2. Задача: при создании документов установить период списания автоматически. . .	Вывод данных в справочнике через динамический список Maks 01.04.2026 Реализация из решения ниже выполнена на примере нетипового справочника "Спецтехника" разработанного в конфигурации КА2. Задача: вывести данные из ТЧ нетипового документа. . .
Функция заполнения текстового поля в реквизите формы документа Maks 01.04.2026 Алгоритм из решения ниже реализован на нетиповом документе "ВыдачаОборудованияНаСпецтехнику" разработанного в конфигурации КА2, в дополнении к предыдущему решению. На форме документа создается. . .	К слову об оптимизации kumehtar 01.04.2026 Вспоминаю начало 2000-х, университет, когда я писал на Delphi. Тогда среди программистов на форумах активно обсуждали аккуратную работу с памятью: нужно было следить за переменными, вовремя. . .	Идея фильтра интернета (сервер = слой+фильтр). Hrethgir 31.03.2026 Суть идеи заключается в том, чтобы запустить свой сервер, о чём я если честно мечтал давно и давно приобрёл книгу как это сделать. Но не было причин его запускать. Очумелые учёные напечатали на. . .	Модель здравосоХранения 6. ESG-повестка и устойчивое развитие; углублённый анализ кадрового бренда anaschu 31.03.2026 В прикрепленном документе раздумья о том, как можно поменять модель в будущем

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

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

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

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

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

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