Неправильное чтение из файла

@25th_July · Регистрация: 01.03.2017

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

Имеется файл, в нем записано что-то вроде {1 2 3}{2 1 3}{3 1 2}
Моя задача считать все цифры, за исключением первой внутри {} и определенным образом занести в вектор(не суть важна). На выводе у меня все заполнено нулями. Где-то ошибка в реализации чтения, может сможете помочь.

C++

ifstream in("input.txt");
    int i = 0;
    int countSize = 0;
    char simb = 'n';
int flag = 1;
    i = 0;
    while (simb=in.get()!=EOF) {
        //F.getline(simb, 1);
        if (simb != '{' && simb != '}' && simb != ' ' && flag == 0) {
            if (flag == 0) {
                int sim = int(simb) - 48;
                SMatrix[i][sim] = 1;
                SMatrix[sim][i] = 1;
            }           
        }
        else { if (flag == 1) flag = 0; }
        if (simb == '}') {
            flag = 1;
            i++;
        }       
        }
    in.close();

@nmcf · 03.03.2017, 15:26

Для начала в 7-й строке присваивание возьми в скобки.

@DemolitionMan · 03.03.2017, 15:54

Сообщение от 25th_July

i = 0;

- зачем 2 раза i обнулять.

В 6 строке.

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

Сообщение от 25th_July

flag == 0) {

- может из-за этого. Сначала Вы приcвоили flag = 1, а теперь сравниваете и программа туда никогда не заходит.

@25th_July · 03.03.2017, 16:00 **[ТС]**

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

- зачем 2 раза i обнулять.

В 6 строке.

Добавлено через 24 минуты
- может из-за этого. Сначала Вы приcвоили flag = 1, а теперь сравниваете и программа туда никогда не заходит.

можно увидеть else, где флаг изменяется с 1 на 0

@DemolitionMan · 03.03.2017, 16:03

Нет счетчика символов,

Сообщение от 25th_July

C++

1 2	SMatrix[i][sim] = 1; SMatrix[sim][i] = 1;

- вот это бред полнейший, Вы читаете номер числа массива из файла и используете его как номер строки и столбца, тем более, что это 2 раза пишется. Делайте другую программу, с более легкой архитектурой, чтобы более легко читалась.

@25th_July · 03.03.2017, 16:09 **[ТС]**

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

Нет счетчика символов,
- вот это бред полнейший, Вы читаете номер числа массива из файла и используете его как номер строки и столбца, тем более, что это 2 раза пишется. Делайте другую программу, с более легкой архитектурой, чтобы более легко читалась.

Это преобразование списка смежности в матрицу смежности, которое полностью логично. Просто список смежности по заданию имеет убогую структуру в файле.
Такая запись {0 1 2} означает, что вершина 0 смежна с 1 и 2. Соответственно 1 и 2 также связаны с 0. Собственно поэтому и дважды присвоение.

@DemolitionMan · 03.03.2017, 16:15

Я не понял, ее что не нужно читать в матрицу 3x3?

@25th_July · 03.03.2017, 16:19 **[ТС]**

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

Я не понял, ее что не нужно читать в матрицу 3x3?

Матрица смежности n*n элементов, где n - количество вершин в графе. Каждая запись в файле такого рода {* 1 2 3} означает, что вершина с номером * смежна с вершинами с номерами идущими до }. Так что просто прохожусь по файлу и заношу данные в матрицу.

@DemolitionMan · 03.03.2017, 16:27

Что, все работает?

@25th_July · 03.03.2017, 16:34 **[ТС]**

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

Что, все работает?

именно

@25th_July 0 / 0 / 1 Регистрация: 01.03.2017 Сообщений: 32
	03.03.2017, 16:34 [ТС]	10
	Сообщение от DemolitionMan Что, все работает? именно 0

@nmcf 7525 / 6391 / 2913 Регистрация: 14.04.2014 Сообщений: 27,845
	03.03.2017, 15:26	2
	Сообщение было отмечено 25th_July как решение Решение Для начала в 7-й строке присваивание возьми в скобки. 1

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	03.03.2017, 15:54	3
	Сообщение от 25th_July i = 0; - зачем 2 раза i обнулять.В 6 строке. Добавлено через 24 минуты Сообщение от 25th_July flag == 0) { - может из-за этого. Сначала Вы приcвоили flag = 1, а теперь сравниваете и программа туда никогда не заходит. 0

@25th_July 0 / 0 / 1 Регистрация: 01.03.2017 Сообщений: 32
	03.03.2017, 16:00 [ТС]	4
	Сообщение от DemolitionMan - зачем 2 раза i обнулять.В 6 строке. Добавлено через 24 минуты - может из-за этого. Сначала Вы приcвоили flag = 1, а теперь сравниваете и программа туда никогда не заходит. можно увидеть else, где флаг изменяется с 1 на 0 0

@25th_July 0 / 0 / 1 Регистрация: 01.03.2017 Сообщений: 32
	03.03.2017, 16:09 [ТС]	6
	Сообщение от DemolitionMan Нет счетчика символов, - вот это бред полнейший, Вы читаете номер числа массива из файла и используете его как номер строки и столбца, тем более, что это 2 раза пишется. Делайте другую программу, с более легкой архитектурой, чтобы более легко читалась. Это преобразование списка смежности в матрицу смежности, которое полностью логично. Просто список смежности по заданию имеет убогую структуру в файле. Такая запись {0 1 2} означает, что вершина 0 смежна с 1 и 2. Соответственно 1 и 2 также связаны с 0. Собственно поэтому и дважды присвоение. 0

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	03.03.2017, 16:15	7
	Я не понял, ее что не нужно читать в матрицу 3x3? 0

@25th_July 0 / 0 / 1 Регистрация: 01.03.2017 Сообщений: 32
	03.03.2017, 16:19 [ТС]	8
	Сообщение от DemolitionMan Я не понял, ее что не нужно читать в матрицу 3x3? Матрица смежности nn элементов, где n - количество вершин в графе. Каждая запись в файле такого рода { 1 2 3} означает, что вершина с номером * смежна с вершинами с номерами идущими до }. Так что просто прохожусь по файлу и заношу данные в матрицу. 0

@DemolitionMan 132 / 158 / 87 Регистрация: 06.04.2016 Сообщений: 992
	03.03.2017, 16:27	9
	Что, все работает? 0

Опции темы

Неправильное чтение из файла

Решение