Task 1 Pipes

Filipbg 26 Точки

Task 1 Pipes

#include <iostream>
#include <cmath>
#include <array>
using namespace std;

int main()
{
    int numberOfPipes = 0;
    int lifetimes = 0;
    array<int, 100> checkup = {0};
    array<int, 100> installation = {0};
    cin >> numberOfPipes;
    if(numberOfPipes > 0 && numberOfPipes <= 500)
    {
        for(int i = 0; i < numberOfPipes; ++i)
        {
            cin >> checkup[i];
        }
        for(int n = 0; n < numberOfPipes; ++n)
        {
            cin >> installation[n];
        }
        if(installation.size() > 0 && installation.size() <= 500)
        {
            for(int i = 0; i < numberOfPipes; ++i)
            {
                if(checkup[i] < installation[i])
                {
                    lifetimes = abs(checkup[i] - installation[i]);
                }
                else
                {
                    lifetimes = installation[i];
                }
                cout << lifetimes << " ";
            }
        }
    }
    return 0;
}

От задачата разбирам че годините се намират като се извадят всички checkup от installation. Или бъркам логиката за изчисляването на годините?

Тагове:

27/09/2019 15:55:42 C++ Fundamentals

MartinBG 4803 Точки

Best Answer

По условие се търси след колко години тръбата трябва да бъде заменена (0 - трябва да се смени веднага, 1 - след 1 година и т.н.). Годините се получават като се извади здравината от текущото (new) замерване от предходното (old) замерване - така получаваме стъпката на износване и с нея делим текущата здравина (new):

years = new / (old - new)

В решението ти има много ненужни проверки (проверяваш неща, които по условие са ти гарантирани) и това може би те обърква допълнително.

Следните валидации са излишни:

if(numberOfPipes > 0 && numberOfPipes <= 500)

if(installation.size() > 0 && installation.size() <= 500)

if(checkup[i] < installation[i])

27/09/2019 17:23:50

Filipbg 26 Точки

Благодаря за отговора. Реших я но Judge даде 60/100 и 4 Runtime Errors. Нямам идея къде греша :/
EDIT: Промених масивите с големина 500 и мина. Явно това са имали в предвид в условието.

#include <iostream>
#include <cmath>
#include <array>
using namespace std;

int main()
{
int numberOfPipes = 0;
int lifetimes[] = {0};
array<int, 100> checkup = {0};
array<int, 100> installation = {0};
cin >> numberOfPipes;
for(int i = 0; i < numberOfPipes; ++i)
{
    cin >> checkup[i];
}
for(int n = 0; n < numberOfPipes; ++n)
{
    cin >> installation[n];
}
for(int i = 0; i < numberOfPipes; ++i)
{
lifetimes[i] = 1.0 * installation[i] / (checkup[i] - installation[i]);
cout << abs(lifetimes[i]) << " ";
}
return 0;
}

27/09/2019 18:08:29 27/09/2019 18:17:22

MartinBG 4803 Точки

В случая по-удачно е да използваш vector вместо array, тъй като размерът му може да се променя динамично (т.е. по време на изпълнение на програмата), а не е фиксиран:

#include <iostream>
#include <vector>
using namespace std;

int main()
{
  int numberOfPipes;
  vector<int> checkup;
  vector<int> installation;
  cin >> numberOfPipes;
  int pipeStrength;
  for(int i = 0; i < numberOfPipes; ++i)
  {
    cin >> pipeStrength;
    checkup.push_back(pipeStrength);
  }
  for(int n = 0; n < numberOfPipes; ++n)
  {
    cin >> pipeStrength;
    installation.push_back(pipeStrength);
  }
  for(int i = 0; i < numberOfPipes; ++i)
  {
    int yearsLeft = installation[i] / (checkup[i] - installation[i]);
    cout << yearsLeft << " ";
  }
  return 0;
}

27/09/2019 19:16:27

zzerro 16 Точки

Здравейте!

Аз нещо не разбрах.. Следното работи:

void readArr(vector<int> &inArr)
{
    for(size_t i = 0; i < inArr.size(); ++i)
        cin >> inArr[i];
}

Обаче ето така не става:

void readArr(vector<int> &inArr)
{
    for(int i : inArr)
        cin >> inArr[i];
}

Защо?

30/09/2019 00:50:48

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

Във форума е въведено ограничение, което позволява на потребителите единствено да разглеждат публикуваните въпроси.

Task 1 Pipes