Проблем при проверка на решението на задача N.20Тръби в басейн( Simple.Conditions) в judge system.

vladovip 6 Точки

Проблем при проверка на решението на задача N.20Тръби в басейн( Simple.Conditions) в judge system.

Пробвах кода на задача - 20 "Прости проверки" в visual studio 2015 и там си работи, но когато го пробвам в judge system, появява ми се грешка и кода не се изпълнява.

задача 20 " Тръби в басейн. Басейн с обем V има две тръби от които се пълни. Всяка тръба има определен дебит (литрите вода минаващи през една тръба за един час). Работникът пуска тръбите едновременно и излиза за N часа. Напишете програма, която изкарва състоянието на басейна, в момента, когато работникът се върне.

Вход

От конзолата се четат четири реда:

Първият ред съдържа числото V – Обем на басейна в литри – цяло число в интервала [1…10000].
Вторият ред съдържа числото P1 – дебит на първата тръба за час – цяло число в интервала [1…5000].
Третият ред съдържа числото P2 – дебит на втората тръба за час– цяло число в интервала [1…5000].
Четвъртият ред съдържа числото H – часовете които работникът отсъства – число с плаваща запетая в интервала [1.0…24.00]

Изход

Да се отпечата на конзолата едно от двете възможни състояния:

До колко се е запълнил басейна и коя тръба с колко процента е допринесла. Всички проценти се свеждат до цяло число (без закръгляне).
- "The pool is [x]% full. Pipe 1: [y]%. Pipe 2: [z]%."

·Aко басейнът се е препълнил – с колко литра е прелял за даденото време, число с плаваща запетая

"For [x] hours the pool overflows with [y] liters."

* Имайте предвид, че поради свеждането до цяло число се губят данни и нормално сборът на процентите да е 99%, а не 100%. "

Ето го кода, поне това което опитах да напиша :

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace Pipesinpool
{
class Pipesinpool
{
static void Main(string[] args)
{
var V = int.Parse(Console.ReadLine());
var P1 = int.Parse(Console.ReadLine());
var P2 = int.Parse(Console.ReadLine());
var H = double.Parse(Console.ReadLine());
var resultV = (P1+P2)*H;

if (1 < V && V < 10000 && P1 > 1 && P1 < 5000 && P2 > 1 && P2 < 5000 && H > 1.0 && H < 24.00 && V > resultV)
{
Console.WriteLine("The pool is "+((resultV/V)*100)+ "% full. Pipe 1:"+((P1*H/resultV)*100)+"%.Pipe 2:" + ((P2*H/resultV)*100)+"%");

}
else if (1 < V && V < 10000 && P1 > 1 && P1 < 5000 && P2 > 1 && P2 < 5000 && H > 1.00 && H < 24.00 && V < resultV)
{
Console.WriteLine("For " + (resultV-V)/(P1 + P2) + "hours the pool overflows with" +(resultV-V)+ " liters.");
}

}
}
}

Тагове:

20/06/2016 23:27:40 Programming Basics

Filkolev 4482 Точки

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

1 < V && V < 10000 - това може да е отделна променлива, която значи нещо,

P1 > 1 && P1 < 5000 - друга

P2 > 1 && P2 < 5000 - още една

H > 1.0 && H < 24.00 && - и още една

Второ, възниква въпросът - нищо ли не правим ако не влезеш в if или else if условията? Доколкото виждам те са еднакви, с изключение на най-последното условие, т.е. какво се случва ако V == resultV, или ако някое от другите условия не е изпълнено?

Тук по-скоро ти трябва if - else конструкция и трябва да се помисли равенството къде попада.

Също така нужно ли е да се правят другите проверки? По принцип входните данни не се валидират, те са такива, каквито са казани; освен принципът в тези домашни да е друг? Пак няма логика, защото на изхода трябва да има един ред принтиран, а ако се подаде грешен вход няма да има нищо. Това би било малко глупаво, защото с празна програма може да се вземат всички точки от тестовете, в които има невалиден вход.

20/06/2016 23:38:08

Tanina80 11 Точки

И аз имам проблем с джъдж и тази задача... според мен всичко си е ок, само че ми дава 46/100. Това е кода:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace ConsoleApplication5
{
class Program
{
static void Main(string[] args)
{
var v = int.Parse(Console.ReadLine());//1000
var p1 = int.Parse(Console.ReadLine());//100
var p2 = int.Parse(Console.ReadLine());//120
var h = double.Parse(Console.ReadLine());//3
var littresOfp1 = p1 * h;//300
var littresOfp2 = p2 * h;//360

var allTheLittres = littresOfp1 + littresOfp2;//660
var percentOffullnes = (allTheLittres / v) * 100;//66%

var percentOfP1 = Math.Round((littresOfp1 / allTheLittres * 100),0);//45%
var percentOfP2 = Math.Round((littresOfp2 / allTheLittres * 100),0);//54%

if (v > allTheLittres)
{
Console.WriteLine("The pool is {0}% full. Pipe 1: {1}%. Pipe 2: {2}%.", percentOffullnes, percentOfP1, percentOfP2);
}
else
{
Console.WriteLine("For {0} hours the pool overflows with {1} liters.", h, allTheLittres - v);
}
}
}
}

25/06/2016 00:53:16

borislav9212 745 Точки

Здравей, Трябва да използваш Math.Floor(той закръгля към по малкото винаги), а Math.Round - до най-близкото. В иф проверката, трябва да >=. И малко грешки в ичисленията имаш. Ето ти поправен кода -> http://pastebin.com/8fqQhd3D

25/06/2016 01:26:58

rado84 1 Точки

Когато решавах тази задача, първоначално използвах Math.Floor, но гърмеше още на домашния тест (тоест, преди да я пусна в Джъдж) и затова изполвах Math.Truncate, което директно "изрязва" всичко след десетичната точка и така си реших проблема.

Console.WriteLine("The pool is {0}% full. Pipe 1: {1}%. Pipe 2: {2}%.",
Math.Truncate(water / volume * 100),
Math.Truncate(pipe1 * hours / water * 100),
Math.Truncate(pipe2 * hours / water * 100));

Така и не разбрах защо Math.Floor не искаше да си свърши работата, но след като Трънкейт ми реши проблема, не съм се занимавал да разнищвам случая.

25/06/2016 03:43:05

Tanina80 11 Точки

Едва когато дописах Math.Floor и за резултата на - var percentOffullnes = Math.Floor((allTheLittres / v) * 100);//66% , най-накрая ми даде 100/100.

26/06/2016 15:48:03 26/06/2016 20:37:34

Покажи всички коментари