C++ program - N učenika, srednja ocjena

Tek sam poceo upotrebljavati C++ i naisao sam na problem u  zadatku pa bih molio pomoc. 

Svaki prijedlog je dobrodosao I PRIHVATIT ĆU GA SA ZAHVALNOŠĆU.

Unesite broj N (najviše 10 – provjerite i ispišite upozorenje ukoliko je broj prevelik!) i
omogućite unos imena i po 5 ocjena za svakog od N učenika.
a. Izračunajte srednju ocjenu za svakog učenika, zapamtite i ispišite popis koji sadrži
ime učenika i srednju ocjenu.
b. Ispišite popis učenika (imena) sortirana abecedno.
c. Implementirajte funkciju za izračun srednje ocjene (kao parametri ulaze ocjene, a
funkcija vraća srednju ocjenu).
d. Omogućite unos podataka korištenjem funkcije za unos (dodatno uz standardni unos).

Ja sam isto tek poceo koristit C++. Ma ustvari da budem iskren prvi put se sluzim programiranjem. Totalni pocetnik sam.

Uglavnom citao sam nekakav tutorial, pitao nekolicinu ljudi i svima ne radi ovaj source:

#include <iostream>
#include <cstdlib>
using namespace std;
int main ()
{
     string a;
     printf ("Write the password: ");
     scanf ("%s",&a);
if (a=="lozinka")
printf ("Password is correct!");

else
printf ("Password is incorrect");

system ("PAUSE");
     return 0;
}

@Tracer

dobro uradjeno, samo sto nisi u struct SUcenik, umjesto niza uzeo string, za ime..isto tako za ocjene si mogo vektor, mada ovde eto ako se trazi bas 5 onda je brze i lakse niz..za ime si ipak mogo string, il vektor stringove, buduci da moze se i prezime unijet..

neda mi se pisat program, a i predpostavljam da si daleko iskusniji od mene, pa zato i pitam sto ne ovako, jer ja bih koristio string..

i mogao si sortirat pomocu sort() i iteratora..cini mi se da bi to sve bilo malo vise c++ a manje c..a i lakse..

Ono što bih ja rekao da je krivo:

1. nije includan <string>

2. učitavanje stringa ide preko cin

== može kod stringova, to je operator koji postoji u c++-u :D

Ako se želi c-ovski, onda:

1. promjenit početak program, tj includat potrebna zaglavlja

2. kod deklaracije varijable a ide char a[neki_broj] ili char*a pa dinamička alokacija kasnije

3. u c-u nema == za stringove (tj nizove charova), pa ide strcmp

Za sve one koji vole komplicirati, i misle da zapravo time pokazuju nekakvo znanje.. u krivu su. Evo, baš me zanima koliko tih će razumijeti onaj gore isti program napisan na sljedeći način.

#include <iostream>
#define MAX 10
using namespace std;

class CUcenik{
    private:
        char ime[20];
        int ocjena[5];
        double prosjek;
        // tek toliko da budemo cool...
        friend istream& operator >> (istream& izlaz, CUcenik &pom);
    public:
        double GetProsjek();
        char* GetIme() { return ime; }
};
// zašto jednostavno kad možemo zakomplicirati?
double CUcenik::GetProsjek(){
    int sum = 0;
    for(int i = 0; i < 5; i++)
        sum += ocjena[i];
    return (double)sum/5;
}
// e tek smo sada prave face ;)
istream& operator >> (istream& ulaz, CUcenik &pom){
    cout << "Ime ucenika: ";
    ulaz >> pom.ime;
    for(int i = 0; i < 5; i++){
        cout << i+1 << ".Ocjena: ";
        ulaz >> pom.ocjena[i];
    }
    return ulaz;
}
int main()
{
    CUcenik Ucenik[MAX], pom;
    int n;

    do{
        cout << "Unesi n: ";
        cin >> n;
    }while(n > 10 || n < 1);

    for(int i = 0; i < n; i++)
        cin >> Ucenik[i];    // gle ti ove fore!
    for(int i = 0; i < n-1; i++)
        for(int j = 0; j < n-1-i; j++)
            if(strcmp(Ucenik[j+1].GetIme(), Ucenik[j].GetIme()))    {
                pom = Ucenik[j];
                Ucenik[j] = Ucenik[j+1];
                Ucenik[j+1] = pom;
            }
    for(int i = 0; i < n; i++)
        cout << Ucenik[i].GetIme() << ", prosjek: " << Ucenik[i].GetProsjek() << endl;
    return 0;
}

I eto.. mislih dodati još sortiranje preko pokazivača na funkcije (qsort), te eventualno napraviti koji predložak funkcije, ili čak klase pa da i takvi koji kompliciraju budu zadovoljni . No mislim da sam dovoljno izrazio svoju poantu.

I sada.. usporedite prvi program s ovime pa recite koji bi vi odabrali.

EDIT: Ne kanim se živcirati

Nema potrebe kompliciranju. Ako se nešto može napraviti jednostavno - onda to tako i napravi.

jako zakompliciran program, posebno radi friend istream..do toga jos nisam doso sa svojim predavanjem :D

al ocigledno nisi shvatio komentar, objekti, metode i algoritmi ne cine komplikovaniji kod, cak ga cine razumljivijim, laksim za odrzavanje i efikasnijim..i valjda je to, kao vec navedeno, u "duhu c++", a ne nizovi struktura, nizovi karaktera..moze se svaki program rijesit uz petlju, if naredbu i niz, i mozda jos koju funkciju..i ako neko jos uvijek zeli razmisljat tako, i sklapat sebi stvari koje su ljudi vec davno napravili, puno efektivnije, brze i lakse, onda izvoli..moj komentar nije napad na tebe i tvoje sposobnosti, nego samo savjet za onog ko je postavio ovu temu, i savjet koji ja dobijam redovno..zakomplicirao si program tako da ga je teze razumit, al je sposoban ko i onaj prije..a mozes ga napravit prostim za razumit, a duplo efikasnijim, al eto, ocigledno si uvrijedjen, mada ti to stvarno ne treba, jer sam siguran da ce meni trebat puno godina i iskustva ako zelim dostic tvoj nivo

ocigledno je to prosto razlika u stilu..ja sam, za vrijeme osnovne skole i programiranja u QBasic-u, bukvalno "obozavao" naredbu GO TO :D

sad je ne koristim nikako, cak nisam do skoro ni znao da to postoji u c i  c++, ali nam je profesor srdacno preporucio da je nikad ne koristimo :p

a kako se sv moglo rijesit uz if uslov i go to, sve, al zato kasnije nemam ni sam pojma sta pise..upravo radi takvih stvari, i ponekad kompliciranije izvedbe, ja preferiram ono malo "vise" sto c++ nudi, a to su algoritmi i predefinirani objekti, kontejneri itd..prosto mi je program pregledniji, laksi za obradu, promjene..mozda neko ko ne zna za te naredbe preferira klasicni c pristup, ali kad nauci to sve, vjerujem da ce mu bit lakse i "ljepse" radit tako, al to je ipak stil, i otiso sam daleko u offtopic..

izvinjavam se jos jednom ako si imao osjecaj da te napadam, samo sam malo skreno u offtopic i poceo pricat o stvarima koje idu malo dalje od ove zadace, prosto radi diskusije..ali vjerovatno za to ima neka druga tema..