Программа выводит лишнее число

@Гриневич Кирилл · Регистрация: 12.08.2015

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

Текст задачи: [cut]
В двух словах о проблеме: По идее в первую строку выходного файла OUTPUT.TXT нужно вывести числа, которые соответствуют дням месяцев, в которые Вася получил тройки, а во второй строке соответственно расположить числа месяца, в которые Вася получил четверки. В третьей строке нужно вывести «YES», если Вася может рассчитывать на четверку и «NO» в противном случае. В каждой строчке числа следует выводить в том же порядке, в котором они идут во входных данных. При выводе, числа отделяются пробелом.
Проблема: У меня выводится последннее число дважды. Пожалуйста объясните в чем проблема и как ее решить?

Код:

C++

#include <iostream>
#include <cstdio>
#include <vector>
using namespace std;
 
int main()
{
  int n, a, i;
  freopen("input.txt","r",stdin);
  freopen("output.txt","w",stdout);
  cin >> a;
  vector<int> k;
  vector<int> l;
  while (!cin.eof())
  {
    cin >> n;
    n % 2 == 0 ? k.push_back(n) : l.push_back(n); 
    i++;
  }
  for (int x = 0; x < l.size(); x++)
  cout << l[x] << " ";
  cout << endl;
  for (int x = 0; x < k.size(); x++)
  cout << k[x] << " ";
  cout << endl;
  k.size()<l.size() ? cout << "NO" : cout << "YES";
  return 0;
}

@zss · 12.08.2015, 19:47

1. У Вас в векторе k окажутся четные оценки, т.е. 2 и 4, в векторе l - нечетные (1,3,5)
2. При вводе после cin>>n надо проверить на конец файла

C++

while (true)
  {
    cin >> n;
    if(!cin)
       break;

@Гриневич Кирилл · 12.08.2015, 19:59 **[ТС]**

УРА!!! Спасибо!

C++

#include <iostream>
#include <cstdio>
#include <vector>
using namespace std;
 
int main()
{
  int n, a, i;
  freopen("input.txt","r",stdin);
  freopen("output.txt","w",stdout);
  cin >> a;
  vector<int> k;
  vector<int> l;
  while (!cin.eof())
  {
    cin >> n;
    if(!cin)
       break;
    n % 2 == 0 ? k.push_back(n) : l.push_back(n); 
    i++;
  }
  for (int x = 0; x < l.size(); x++)
  cout << l[x] << " ";
  cout << endl;
  for (int x = 0; x < k.size(); x++)
  cout << k[x] << " ";
  cout << endl;
  k.size()<l.size() ? cout << "NO" : cout << "YES";
  return 0;
}

lss · 12.08.2015, 21:14

Сообщение от Гриневич Кирилл

C++

while (!cin.eof())
{
     cin >> n;
     if(!cin)
         break;
     n % 2 == 0 ? k.push_back(n) : l.push_back(n); 
     i++;
}

C++

while (cin >> n)
{
    n % 2 == 0 ? k.push_back(n) : l.push_back(n); 
    i++;
}

@Гриневич Кирилл · 13.08.2015, 08:33 **[ТС]**

C++

while (cin >> n)
{
    n % 2 == 0 ? k.push_back(n) : l.push_back(n); 
    i++;
}

будет дольше выполняться и больше памяти занимать

lss · 13.08.2015, 08:39

Сообщение от Гриневич Кирилл

будет дольше выполняться и больше памяти занимать

С какого испуга? Обоснуй утверждения.

@Гриневич Кирилл · 13.08.2015, 10:00 **[ТС]**

выполнил свой код и модернизироваееый тобой на сайте [cut] (задача 5). Выдает ,что мой выполняется за 0,123 и занимает объем памяти 1248 Кб ,а с твоим исправлением за 0,18 и 1336 Кб Кб

lss · 13.08.2015, 10:08

Сообщение от Гриневич Кирилл

выполнил свой код и модернизироваееый тобой

Не смеши меня.

@Гриневич Кирилл · 13.08.2015, 10:11 **[ТС]**

Это правда хочешь сам глянь.

lss 941 / 869 / 355 Регистрация: 10.10.2012 Сообщений: 2,706
	13.08.2015, 08:39	6
	Сообщение от Гриневич Кирилл будет дольше выполняться и больше памяти занимать С какого испуга? Обоснуй утверждения. 0

@Гриневич Кирилл 5 / 5 / 1 Регистрация: 12.08.2015 Сообщений: 57
	13.08.2015, 10:00 [ТС]	7
	выполнил свой код и модернизироваееый тобой на сайте [cut] (задача 5). Выдает ,что мой выполняется за 0,123 и занимает объем памяти 1248 Кб ,а с твоим исправлением за 0,18 и 1336 Кб Кб 0

lss 941 / 869 / 355 Регистрация: 10.10.2012 Сообщений: 2,706
	13.08.2015, 10:08	8
	Сообщение от Гриневич Кирилл выполнил свой код и модернизироваееый тобой Не смеши меня. 0

@Гриневич Кирилл 5 / 5 / 1 Регистрация: 12.08.2015 Сообщений: 57
	13.08.2015, 10:11 [ТС]	9
	Это правда хочешь сам глянь. 0

Программа выводит лишнее число

Решение