Zadanie SPOJ: Input and Output

Question

Napisałem za trzecim razem dobrze! Jak Oceniacie mój Kod?

#include <cmath>
#include <cstdio>
#include <vector>
#include <iostream>
#include <algorithm>
using namespace std;

int a, b, c, s;

int main()
{

    cin>>a>>b>>c;

    cout << (s=a+b+c ) << endl;

    return 0;

}

Link do Zadania: https://www.hackerrank.com/challenges/cpp-input-and-output/problem

Comments

Przenieś swoje pytanie do kategorii SPOJ

---

Tak?

Answer 1

Uh... nie wiem po co dołączasz np. bibliotekę vector, stosujesz zmienne globalne, używasz namespaca std, czy po co tworzysz zmienną s.

Comments

s oznacza sum, Sume tych liczb, a czym jest namespace std? A Bibliotheke vector mam sobie zostawic?

---

s nie jest tutaj kompletnie potrzebne (nigdzie tego nie wykorzystujesz, a samą sumę liczysz w coucie)

nie wiem po co dołączasz np. bibliotekę vector

 A Bibliotheke vector mam sobie zostawic?

:) 

---

Skrocony i Dziala!

#include <cmath>
#include <cstdio>
#include <iostream>
#include <algorithm>
using namespace std;

int a, b, c;

int main()
{

    cin>>a>>b>>c;

    cout << (a+b+c ) << endl;

    return 0;
}

 

---

Działać, będzie działał, ale pytałeś o samą jakość kodu, więc mówię: dalej nie jest dobrze.

Po co załączasz cmath czy alghoritm? (nie jestem pewien, ale te biblioteki na 99% nie są Ci potrzebne) Dlaczego deklarujesz zmienne jako globalne? Dlaczego korzystasz z using namespace std;?

---

Jednak Kod bez using namespacea std i deklarowania zmiennych jako globalne nie dziala,

#include <cmath>
#include <cstdio>
#include <iostream>
#include <algorithm>

using namespace std;

int a, b, c;

int main()
{
    cin>>a>>b>>c;

    cout << (a+b+c ) << endl;

    return 0;
}

 

---

Bez using namespace std przed cout czy cin musisz dać std:: - wtedy będzie działać. Podobnie jak zmienne globalne - nie każe Ci ich usuwać tylko przenieść w int main()

Answer 2

Najkrotsza Wersja jaka znalazlem, Opiera sie na std::cin i std:cout;

#include <iostream>

int main() 
{
    int x,y,z;
    
    std::cin>>x>>y>>z;
    
    std::cout<<x+y+z;
    
    return 0;
}