C++ Sum Seconds problem

dZf1aeA-NLulchev 5 Точки

C++ Sum Seconds problem

Здравейте! Задачата ми дава желаните в упражнението изходи, но в джъджа имам 90/100, като не ми показва къде е грешката (аутпута е наред). Условието е да въведем три стойности за секунди (до 50) , да ги сумираме и да изкараме всичко в минути и секунди, като пред секундите трябва да има 0 (ако числото за секунда е по-малко от 10) Някой може ли да помогне?

#include <iostream>

using namespace std;


int main()
{
	int a, b, c;
	cin >> a >> b >> c;
	int sum = a + b + c;
	if (sum < 59 && sum < 10) {
		cout << "0" << ":0" << sum << endl;
	}
	else if (sum < 59) {
		cout << "0" << ":" << sum << endl;
	}
	if (sum > 59 && sum < 119 && sum - 60 < 10) {
		cout << "1" << ":0" << sum - 60 << endl;
	}
	else if (sum > 59 && sum < 119) {
		cout << "1" << ":" << sum - 60 << endl;
	}
	if (sum > 119 && sum - 120 < 10) {
		cout << "2" << ":0" << sum - 120 << endl;
	} 
	else if (sum > 119) {
		cout << "2" << ":" << sum - 120 << endl;
	}
    return 0;
}

Тагове:

27/03/2018 18:00:17 Programming Basics

dZf1aeA-NLulchev 5 Точки

Написах решението по друг начин и ми го прие 100/100, но все пак бих се радвал, ако някой ми посочи къде греша :)

27/03/2018 18:11:28

MartinPaunov 77 Точки

Здравей,

Прегледах кода ти - имаш едно изпуснато равно на 20-ти ред (приблизително):

else if (sum > 59 && sum <= 119) {
        cout << "1" << ":" << sum - 60 << endl;
    }

по-малко или равно на 119. Само това промених и кода ти мина за 100/100. Не съм сигурен дали някъде другаде няма граничен случай, за който просто не е написан тест, но с това равно кода работи и си вземаш точките.

Мисля, че и на другите гранични стойности би следвало да има проверка за равно.

Пример, ако времето е по-малко от 59 следва, че максималната стойност може да бъде до 58, за да се изпълни това условие, но по този начин пропускаш стойността 59. Аналогично и за другите гранични стойности.

Поздрави

27/03/2018 19:59:07 27/03/2018 20:05:06

dZf1aeA-NLulchev 5 Точки

Да, пробвах така нещата и вървят, благодаря :) Явно невниманието е най-големият неприятел :D Благодаря! :)

27/03/2018 20:09:51

MartinBG 4803 Точки

Няколко коментра по логиката на решението ти:

if (sum < 59 && sum < 10)

Проверката sum < 59 е излишна, защото горното условие ще е валидно само при sum < 10.

Надолу проверките ти стават прекалено сложни като логика и съответно са предпоствака за допускане на грешки.

По-просто и универсално решение би било да разделиш сумата на 60, като това ще ти даде минутите.

int minutes = sum / 60;

Секундите може да получиш като разделиш сумата по модул на 60:

int seconds = sum % 60;

При принтирането на секундите може да използваш padding:

std::cout << minutes << ":" << std::setw(2) << std::setfill('0') << seconds << std::endl;

#include <iostream>
#include <iomanip>

int main()
{
	int a, b, c;
	
	std::cin >> a >> b >> c;
	
	int sum = a + b + c;
	int minutes = sum / 60;
	int seconds = sum % 60;

	std::cout << minutes << ":" << std::setw(2) << std::setfill('0') << seconds << std::endl;

	return 0;
}

28/03/2018 11:02:01 28/03/2018 11:18:59