Разница между двумя идентичными программами - C++

@Parilo · Регистрация: 18.11.2011

Объясните разницу между двумя абсолютно идентичными программами.
Программа 1, в текстовом файле где записаны 10 слов, нужно поменять 1 , 6, K-тый и последнее слово.
Программа 2, в текстовом файле где записаны 10 чисел, нужно поменять 1, 5, K-тый и последнее число.

Обе программа работают, но в 21 строке 1 программы я пишу while (j<i), а в 19 строке 2 программы while (j<i-1). Почему???

C++

 #include <iostream>
 #include <fstream>
 #include <string>
 using namespace std;
 int main ()
 { ifstream f("hw6file.txt");
 string a[100];
 string s;
 getline (cin,s);
 int i=0, k;
 cin >>k;
 
 while (f>> a[i++]);
 
 a[0]=s;
 a[5]=s;
 a[k]=s;
 a[i-2]=s;
 ofstream g("hw6file.txt");
 int j=0;
 while (j<i)
 {g<<a[j]<< endl;j++;}
 
 return 0;}
 
//////////////////////////////////////////////////////////////////////////////////////////////////////////////
 
 #include <iostream>
 #include <fstream>
 using namespace std;
 int main ()
 { ifstream f("hw5file.txt");
 int a[100];
 int k,x;
 cin >> k >>x;
 int i=0;
 
 while (f >> a[i++]);
 a[0]=x;
 a[4]=x;
 a[k]=x;
 a[i-2]=x;
 
 ofstream g("hw5file.txt");
 int j=0;
 while (j<i-1)
 { g<<a[j]<< endl; j++;}
 return 0;}

@Хулиган · 08.11.2012, 21:56

то что здесь написано разве работает?

@Parilo · 09.11.2012, 08:19 **[ТС]**

Сообщение от Хулиган

то что здесь написано разве работает?

Да работает. Скопируйте и проверьте, если не верите. Причём совершенно одинаково работают.

Добавлено через 9 часов 6 минут
Так кто-нибудь сможет дать ответ или нет?
Я написал идентичные программы методом тыка, но не понимаю почему j<i в первом случае, а во втором случае j<i-1.
Если есть возможность узнать у профессиональных программистов, спросите пожалуйста. Мне не у кого спросить, кроме как на этом форуме.
Спасибо заранее.

alsav22 · 09.11.2012, 09:53

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

Почему???

Вы у себя спросите: почему вы в первом примере написали while (j<i)? i, в двух примерах (при 10 считываниях), равен 11. Последнее знчение (десятое) вы правильно записываете по индексу i - 2 = 9, т.е. в a[9]. Потом, в цикле, запись в файл. Выход из цикла будет при j = 11, значит последнее, что запишется в файл: a[10]. А что там, если десятое значение находится в a[9]? Сделайте в цикле:

C++

1	cout << j << ' ' << a[j] << endl;

и всё увидите.
Во втором примере всё правильно сделано.

@SatanaXIII · 09.11.2012, 10:10

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

не понимаю почему j<i в первом случае, а во втором случае j<i-1.

В первом случае последняя итерация избыточна. То есть в первом случае правильнее писать, как и во-втором

C++

1	while (j<i-1)

Допустим у нас в файле 10 строк.
Тогда в массив a записывается только 10 строк. Начиная с a[0] по a[9]. Но так как массив был объявлен на 100 элементов, то все оставшиеся элементы у нас уже заранее объявлены пустыми. И нам не заметно, что в выходной файл на самом деле в конец дописывается пустая строка. Вот и все различие.

А вообще, по хорошему, надо переделать момент

C++

1 2	while (j<i-1) { g<<a[j]<< endl; j++;}

Так как

C++

<< endl;

всегда дописывается. А зачем нам еще одна дополнительная пустая строка в конце файла? Не нужна.
Что-то типа

C++

while (j<i-1)
{
  g << a[j];
  j++;
  if(j!=i-1) g << endl;
}

И просто так, как мысль: можно перед while (j<i-1) сделать i--. Это позволит избавиться от всех i-1, и заменить это выражение на просто i, чтобы каждый раз, на каждой итерации не надо было компилятору вычитать и сравнивать, а надо было лишь только сравнивать.

Не по теме:

Совершенно к делу не относится, но Ближнее небо - Золотой лес

alsav22 · 09.11.2012, 10:28

И считывание с файла лучше так не делать. Последнее считывание - это чтение конца файла, при этом i увеличится. Лучше, например, так делать:

C++

while (f.peek() != EOF)
  {
    f >> a[i];
    i++;
  }

Тогда, при выходе из цикла, i будет равна количеству слов в файле.

@SatanaXIII · 09.11.2012, 10:49

alsav22, логично.

@Parilo · 09.11.2012, 12:47 **[ТС]**

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

Вы у себя спросите: почему вы в первом примере написали while (j<i)? i, в двух примерах (при 10 считываниях), равен 11. Последнее знчение (десятое) вы правильно записываете по индексу i - 2 = 9, т.е. в a[9]. Потом, в цикле, запись в файл. Выход из цикла будет при j = 11, значит последнее, что запишется в файл: a[10]. А что там, если десятое значение находится в a[9]? Сделайте в цикле:

C++

1	cout << j << ' ' << a[j] << endl;

и всё увидите.
Во втором примере всё правильно сделано.

Спасибо большое за разъяснения.
Но меня мучает единственный вопрос.
Почему в первом примере работа со строками хотя я пишу "неправильно" while (j<i), программа работает правильно. Если я даже её исправлю while (j<i-1) он тоже работает правильно.

А во втором примере работа с числами он работает правильно только в одном случае while (j<i-1).

Почем в 1 варианте при неправильном коде, программа работает правильно? Вот последний вопрос.
Спасибо заранее.

@Croessmah · 09.11.2012, 12:55

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

Почем в 1 варианте при неправильном коде, программа работает правильно? Вот последний вопрос.

Ради эксперимента, попробуйте так:
while (j<i)
{a[j]="abcd";j++;}

@Parilo · 09.11.2012, 12:59 **[ТС]**

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

Ради эксперимента, попробуйте так:
while (j<i)
{a[j]="abcd";j++;}

Я Вас не понял.

alsav22 · 09.11.2012, 13:00

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

А во втором примере работа с числами он работает правильно только в одном случае while (j<i-1).

А если while (j<i), то какую ошибку выдаёт? И среда у вас какая?

@Croessmah · 09.11.2012, 13:08

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

Я Вас не понял.

Вам уже объясняли, что у Вас выход за границы происходит, при считывании все нормально, но при записи за границы массива произойдет ошибка.

попробуйте поменять строчку

C++

a[i-2]=s;

на

C++

a[i-1]=s;

Тоже произойдет ошибка, если такого элемента не окажется в массиве

alsav22 · 09.11.2012, 13:15

У меня и в первом случае ошибки при работе нет и во втором. Просто, во втором случае, в файл мусор из массива в конец файла запишется, и это будет видно. А в первом случае, в конец файла запишется пустая строка, и это незаметно. Но если открыть файл после записи и переместить курсор в конец до упора, то будет видно, что между курсором и последним словом две пустые строки. Одну endl сделал, а одну - та самая лишняя пустая строка из массива строк.

Добавлено через 3 минуты
Croessmah, у ТС выхода за границы массива нет. У него массивы на 100 элементов, а пишет 10. У него, при записи в файл, выход за границу записанных в массив значений.

@Croessmah · 09.11.2012, 13:18

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

Croessmah, у ТС выхода за границы массива нет. У него массивы на 100 элементов, а пишет 10. У него выход за границу записанных значений.

Одно от другого не отделимо - будет больше записей, выйдет и за пределы.

alsav22 · 09.11.2012, 13:24

Согласен, но у него вопрос по конкретному коду.

@Хулиган 85 / 80 / 12 Регистрация: 08.08.2012 Сообщений: 737
	08.11.2012, 21:56	#2
	то что здесь написано разве работает? 0

@Parilo 14 / 2 / 0 Регистрация: 18.11.2011 Сообщений: 90
	09.11.2012, 08:19 [ТС]	#3
	Сообщение от Хулиган то что здесь написано разве работает? Да работает. Скопируйте и проверьте, если не верите. Причём совершенно одинаково работают. Добавлено через 9 часов 6 минут Так кто-нибудь сможет дать ответ или нет? Я написал идентичные программы методом тыка, но не понимаю почему j<i в первом случае, а во втором случае j<i-1. Если есть возможность узнать у профессиональных программистов, спросите пожалуйста. Мне не у кого спросить, кроме как на этом форуме. Спасибо заранее. 0

@SatanaXIII Супер-модератор 5616 / 2651 / 246 Регистрация: 01.11.2011 Сообщений: 6,530 Завершенные тесты: 1
	09.11.2012, 10:49	#7
	alsav22, логично. 0

@Croessmah 13416 / 7570 / 855 Регистрация: 27.09.2012 Сообщений: 18,639 Записей в блоге: 3 Завершенные тесты: 1
	09.11.2012, 12:55	#9
	Сообщение от Parilo Почем в 1 варианте при неправильном коде, программа работает правильно? Вот последний вопрос. Ради эксперимента, попробуйте так: while (j<i) {a[j]="abcd";j++;} 0

@Parilo 14 / 2 / 0 Регистрация: 18.11.2011 Сообщений: 90
	09.11.2012, 12:59 [ТС]	#10
	Сообщение от Croessmah Ради эксперимента, попробуйте так: while (j<i) {a[j]="abcd";j++;} Я Вас не понял. 0

alsav22 5421 / 4816 / 442 Регистрация: 04.06.2011 Сообщений: 13,587
	09.11.2012, 13:00	#11
	Сообщение от Parilo А во втором примере работа с числами он работает правильно только в одном случае while (j<i-1). А если while (j<i), то какую ошибку выдаёт? И среда у вас какая? 0

alsav22 5421 / 4816 / 442 Регистрация: 04.06.2011 Сообщений: 13,587
	09.11.2012, 13:15	#13
	У меня и в первом случае ошибки при работе нет и во втором. Просто, во втором случае, в файл мусор из массива в конец файла запишется, и это будет видно. А в первом случае, в конец файла запишется пустая строка, и это незаметно. Но если открыть файл после записи и переместить курсор в конец до упора, то будет видно, что между курсором и последним словом две пустые строки. Одну endl сделал, а одну - та самая лишняя пустая строка из массива строк. Добавлено через 3 минуты Croessmah, у ТС выхода за границы массива нет. У него массивы на 100 элементов, а пишет 10. У него, при записи в файл, выход за границу записанных в массив значений. 0

@Croessmah 13416 / 7570 / 855 Регистрация: 27.09.2012 Сообщений: 18,639 Записей в блоге: 3 Завершенные тесты: 1
	09.11.2012, 13:18	#14
	Сообщение от alsav22 Croessmah, у ТС выхода за границы массива нет. У него массивы на 100 элементов, а пишет 10. У него выход за границу записанных значений. Одно от другого не отделимо - будет больше записей, выйдет и за пределы. 0

alsav22 5421 / 4816 / 442 Регистрация: 04.06.2011 Сообщений: 13,587
	09.11.2012, 13:24	#15
	Согласен, но у него вопрос по конкретному коду. 0

С++ для начинающих

Разница между двумя идентичными программами - C++