Форум программистов, компьютерный форум CyberForum.ru

Работа с файлами - C++

Восстановить пароль Регистрация
 
Dobbos
1 / 1 / 0
Регистрация: 01.03.2012
Сообщений: 16
03.07.2012, 21:20     Работа с файлами #1
Программа полностью рабочая. Кроме одного момента.

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
#include <conio.h>
#include <stdio.h>
#include <iostream>
#include <clocale>
 
 
using namespace std;
 
void main()
{
setlocale (LC_CTYPE, "rus");
 
 char** disc; 
 
 int n = 8; // 1 кластер - 8 байт
 int m = 255; // число строк
 
 disc = new char*[m];
 
 for(int i=0; i < m; i++) 
 {
  disc[i] = new char[i];
 }
 
for(int i=0; i < m; i++) 
    {
        disc[i]=NULL;
    }
 
 
 
char str[9];
 
FILE* indisc;
indisc = fopen("disc.txt", "r");
 
int i=0;
 
do
{   
  fscanf(indisc, "%s", &str);
  printf("%s\n", str);
  disc[i]=str;
  i=i+1;
}
while (!feof(indisc));
 
for(int i=0; i < m; i++) 
    {
        printf("A[%d] = %s\n", i, disc[i]);
    }
 
 _getch();
}
После запуска становится видна проблема: все элементы массива disc становятся равны последнему извлеченному из файла значению (kolovot).

Название файла: disc.txt
Содержимое файла: ades lolo kopilka suss kipish kolovot
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
03.07.2012, 21:20     Работа с файлами
Посмотрите здесь:

Работа с файлами C++
C++ Работа с файлами
C++ РАБОТА С ФАЙЛАМИ
C++ Работа с файлами
C++ Работа с файлами
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
grizlik78
Эксперт C++
 Аватар для grizlik78
1882 / 1414 / 101
Регистрация: 29.05.2011
Сообщений: 2,958
03.07.2012, 21:35     Работа с файлами #2
Вместо
C++
1
disc[i]=str;
надо использовать
C++
1
strcpy(disc[i], str);
Иначе копируется не значение, а лишь адрес str, который всегда один и тот же.
Но в начале память для disk[i] выделяется тоже неправильно. Почему new char[i]?
Лучше память выделять прямо перед считыванием.
C++
1
2
disc[i] = new char[strlen(str)+1];
strcpy(disc[i], str);
Добавлено через 3 минуты
Этот фрагмент умиляет:
Цитата Сообщение от Dobbos Посмотреть сообщение
C++
1
2
3
4
5
6
7
8
9
 for(int i=0; i < m; i++) 
 {
  disc[i] = new char[i];
 }
 
for(int i=0; i < m; i++) 
    {
        disc[i]=NULL;
    }
Цитата Сообщение от Dobbos Посмотреть сообщение
Программа полностью рабочая.
Хе-хе. Выделили память и тут же про это забыли.
Dobbos
1 / 1 / 0
Регистрация: 01.03.2012
Сообщений: 16
03.07.2012, 21:53  [ТС]     Работа с файлами #3
Цитата Сообщение от grizlik78 Посмотреть сообщение
Вместо
C++
1
disc[i]=str;
надо использовать
C++
1
strcpy(disc[i], str);
При использовании данного фрагмента выходит ошибка - нарушение прав доступа при записи.

По поводу освобождения памяти - другая история)

Насчет обнуления, весь массив скорее всего не запонится и для наглядности, и учитывая то, как я планирую его использовать, мне это пригодится.
grizlik78
Эксперт C++
 Аватар для grizlik78
1882 / 1414 / 101
Регистрация: 29.05.2011
Сообщений: 2,958
03.07.2012, 21:55     Работа с файлами #4
Цитата Сообщение от Dobbos Посмотреть сообщение
При использовании данного фрагмента выходит ошибка - нарушение прав доступа при записи.
Ну, если выделение памяти осталось прежним, то неудивительно.
Dobbos
1 / 1 / 0
Регистрация: 01.03.2012
Сообщений: 16
03.07.2012, 22:01  [ТС]     Работа с файлами #5
Цитата Сообщение от grizlik78 Посмотреть сообщение
Ну, если выделение памяти осталось прежним, то неудивительно.
Спасибо большое, я кажется понял. Проблема близка к решению.
Yandex
Объявления
03.07.2012, 22:01     Работа с файлами
Ответ Создать тему
Опции темы

Текущее время: 21:06. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru