Определить количество троек, которые надо исправить на четвёрки

@tolyanoffna · Регистрация: 13.11.2016

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

Почему-то переменная n не увеличивается, хотя теоретически должно быть верно. Спасите-помогите

Матвей, посмотрев на текущие оценки по информатике обнаружил, что из десяти его оценок, троек больше чем четвёрок. Матвею нравится, когда четвёрок у него ровно столько же, сколько троек, или четвёрок на одну больше. Он хочет написать программу, которая по списку его оценок, выведет количество троек, которые надо исправить на четвёрки, чтобы было так, как ему нравится.

Например, оценки Матвея: 4 2 3 5 3 4 3 3 4 3. Мы видим, что троек у него 5, а четвёрок 3. Значит надо исправить всего одну тройку.

C++

#include <bits/stdc++.h>
using namespace std;
int main(){
  int i,k3,k4,a,m,j,n=0,l;
  m=(k3-k4)/2;
  for (i=0;i<10;i++)
  {cin>>a; 
  if (a==3) k3++; 
  if (a==4) k4++;}
  if ((k3-k4)%2==0)
  {for (j=0;j<10;j++)
  {k3--; k4++; n++;
  if (k3==k4) l=n;}}
  if ((k3-k4)%2!=0)
  {for (j=0;j<10;j++)
  {k3--; k4++; n++;
  if (k4==k3+1) l=n;}}
  cout<<l;
  return 0;
 }

@dreamer_108 · 27.03.2019, 16:35

Форматируйте, пожалуйста, код, прежде чем его выкладывать, а то читать невозможно:

C++

#include <iostream>
 
int main() {
    int i, k3, k4, a, m, j, n = 0, l;
    m = (k3 - k4) / 2;
    for (i = 0; i < 10; i++)
    {
        std::cin >> a;
        if (a == 3) k3++;
        if (a == 4) k4++;
    }
    if ((k3 - k4) % 2 == 0)
    {
        for (j = 0; j < 10; j++)
        {
            k3--; k4++; n++;
            if (k3 == k4) l = n;
        }
    }
    if ((k3 - k4) % 2 != 0)
    {
        for (j = 0; j < 10; j++)
        {
            k3--; k4++; n++;
            if (k4 == k3 + 1) l = n;
        }
    }
    std::cout << l;
    return 0;
}

Добавлено через 21 секунду
Правда ведь, намного легче читается?

Добавлено через 2 минуты
Для начала, вы не инициализируете переменные k3 и k4 перед использовением в строке 5 (в моем варианте), так что работать это при всем желании не будет.
Во вторых, с такими названиями переменных очень сложно проследить ход мысли

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

C++

#include <iostream>
 
int main() {
 
    int marks[10] = { 4, 3, 2, 5, 3, 4, 3, 3, 4, 3 };
 
    int count3 = 0;
    int count4 = 0;
    for (int i = 0; i < 10; i++)
    {
        if (marks[i] == 3) count3++;
        if (marks[i] == 4) count4++;
    }
 
    int needToFix = 0;
    if (count4 < count3)
    {
        needToFix = (count3 - count4) / 2;
        if ((count3 - count4) % 2 == 1)
            needToFix += 1;
    }
 
    std::cout << needToFix << std::endl;
    system("pause");
 
    return 0;
}

Добавлено через 3 минуты
А для вот этого места в Вашем коде:

C++

if ((k3 - k4) % 2 == 0)
    {
        for (j = 0; j < 10; j++)
        {
            k3--; k4++; n++;
            if (k3 == k4) l = n;
        }
    }
    if ((k3 - k4) % 2 != 0)
    {
        for (j = 0; j < 10; j++)
        {
            k3--; k4++; n++;
            if (k4 == k3 + 1) l = n;
        }
    }

специально существует слово else. Кроме того, если уж проходиться по массиву циклом, можно проводить проверку внутри него, ибо эти два цикла почти не отличаются

@tolyanoffna · 27.03.2019, 17:54 **[ТС]**

Как только я инициализировала k3, k4 всё заработало. Спасибо

@tolyanoffna 0 / 0 / 0 Регистрация: 13.11.2016 Сообщений: 9
	27.03.2019, 17:54 [ТС]	3
	Как только я инициализировала k3, k4 всё заработало. Спасибо 0