C++ no match for "operator>>"

enum konstantama se ne može re-deklarirati ili nanovo unijeti druga vrijednost pa je stoga ta cijela linija besmislena.

Imam jos jedan zadatak.

/*
Requires:
variables, data types, and numerical operators
basic input/output
logic (if statements, switch statements)

Write a program that allows the user to enter the grade scored in a programming
class (0-100). If the user scored a 100 then notify the user that they got a
perfect score.

Modify the program so that if the user scored a 90-100 it informs the user
that they scored an A

Modify the program so that it will notify the user of their letter grade
0-59 F 60-69 D 70-79 C 80-89 B 90-100 A
*/

#include <iostream>

void YourGrade(int nTestScore)
{
    using namespace std;
   
    if(nTestScore == 100)
        cout << "Congratulations! You got a perfect score!" << endl;  
    else if(nTestScore < 100 && nTestScore> 89)
        cout << "You got an A." << endl;
    else if(nTestScore < 90 && nTestScore> 79)
        cout << "You got a B." << endl;
    else if(nTestScore < 80 && nTestScore> 69)
        cout << "You got a C." << endl;
    else if(nTestScore < 70 && nTestScore> 59)
        cout << "You got a D." << endl;
    else if(nTestScore < 60)
        cout << "I am sorry. You got an F." << endl;
       
}

int main()
{
    using namespace std;
   
    tryAgain:
    cout << "Enter your programming test score (0-100)" << endl;
    int nTestScore;
    cin >> nTestScore;
   
    while (nTestScore > 100)
    {
        cout << "Invalid input. Please re-check your score and try again." << endl;
        goto tryAgain;
    }
    YourGrade(nTestScore);
   
    system("pause");
    return 0;      
}

Da ne bi bilo zabune, ovaj kod gore radi. Problem je u tome sto sam procitao da bi se goto trebao koristi sto je manje moguce.

Umjesto toga pokusao sam koristiti do while. Evo i ta verzija.

int main()
{
    using namespace std;
    int nTestScore;
   
    do
    {
    cout << "Enter your programming test score (0-100)" << endl;
    cin >> nTestScore;
    } while (nTestScore > 100); 
   
    YourGrade(nTestScore);
   
    system("pause");
    return 0;      
}

Ne znam gdje, a ni kako da stavim cout << "Invalid input. Please re-check your score and try again." << endl; a da se ispisuje samo kada vrijedi nTestScore > 100.

Ako ga stavim pod do problem je sto ce se to cout ispisati i prvi put, na pocetku programa prije nego sto je uopce bilo ikakvog inputa i tada to ne bi imalo smisla.

Ako ga stavim poslije do ne postoji mogucnost da taj uvjet vrijedi jer program ne dopusta da se dode do tuda dok taj uvjet vrijedi.

Ovako bi bilo logičnije, u while petlju se ulazi kad se mora, a ona ima jedinstvenu poruku, nema ponavljanja koda osim nužnog ponavljanja unosa.

(obrati pažnju i da sam modificirao konzolni ispis i mjesto unosa, tako da se unos vrši u istom redu gdje je obavijest o unosu, a operator cin se pobrine za prelazak u novi red)

A inače, napiši ovu funkciju YourGrade sa switchom, to ti u opisu zadatka i traži, jer je ovdje sa switchom moguće napisati kod jednostavnije, tako da ti je to prikriveni zahtjev.

#include<iostream>
using namespace std;


int main()
{
  
    cout << "Enter your programming test score (0-100): ";
    int nTestScore;
    cin >> nTestScore;
  
    while (nTestScore < 0 || nTestScore > 100 )
    {
       cout << "Invalid input! Enter your programming test score again (0-100): ";
       cin>> nTestScore;
       
    }
    YourGrade(nTestScore);
  
    system("pause");
    return 0;     
}

Programeru šteti i više nego programiranju, bit je da se čovjek nauči logičkim uvjetima, a ne preskocima, pošto je bit programiranja logika, a ne preskok.

Zato je dobro da je pitao, i da hoće pravo krenuti.

Kao i u svem drugom u programirnju i ovdje postoji više izbora za izvršiti provjeru unosa i poslati odgovarajuće poruke: while, do-while,  goto

Bira se način koji daje manje koda, veću efikasnost i robustnost, a bolju preglednost.

Kompromisi uvijek moraju biti u svemu, jer idealnog nema. Tako bi ovdje do-while tražila manje koda, ali na ispis greške bi mogli zaboraviti ili dodati još jednu provjeru u petlju, stavio sam petlju while jer ne želim dodatnu provjeru da bi ispisao grešku, ali sam tim dobio prvi upis izvan petlje.

Zar uopće moram linkat xkcd kad se priča o goto naredbi? :D

http://xkcd.com/292/

U tom slučaju moraš dobiti konstantne vrijednosti za slučajeve u switchu.

Potrebno je samo na varijablu score primjeniti cjelobrojno djeljenje sa 10, a onda nam je npr. vrijednost i za 85 i 80 bodova ista, odnosno iznosi 8.

Da se malo našalim, ovaj switch simulira realnost, defaultna je čovjeku samo jedinica, ostalo postiže radom.

void YourGrade( int score)
{
    score /= 10;
    switch(score)
    {
    case 10:
       {
          cout<< "Perfect score" << endl;
          break;
       }
    case 9:
       {
          cout<< "A" << endl;
          break;
       }
    case 8:
       {
          cout<< "B" << endl;
          break;
       }
    case 7:
       {
          cout<< "C" << endl;
          break;
       }
    case 6:
       {
          cout<< "D" << endl;
          break;
       }
    default:
       {
          cout<< "F" << endl;
       }
    }
}

E, onda to ne bi mogao riješiti switchom, sportskim riječnikom, ovdje si dobio priliku da "zakucaš", odnosno da zahvaljujući distribuciji ocjena iskoristiš cjelobrojno djeljenje za dobiti konstante u switchu.

Onda bi išao na if - else if - else

Ali ne na tvoj način, dovoljna je jednostruka provjera od manjih ka većem:

if(score < 60)

   cout<< "F";

else if(score < 70)

   cout<< "D";

Ja sam za unos koristio getline umjesto cin zato da se moze unijeti i ime i prezime. Znam da mogu to odvojiti na (npr.) cin >> strFirstName i cin >> strSurname ali kako bih to napisao a da zadrzim getline tj. da ne pita posebno ime i nakon toga posebno prezime. Postoji li neka alternativa za cout kao sto je getline za cin?

Zanime me kako bih to napravio ovako, a da i dalje sve funkcionira normalno?

cout << "Enter your full name." << endl;

getline (cin, strFullName);

   Kyle Broflovski

Umjesto ovoga:

cout << "Enter your first name." << endl;

cin >> strFirstName;   Kyle

cout << "Enter your surname." << endl;

cin >> strSurname;   Broflovski

I da, ovo 

    for(int i= 0; i<nSize; i++)
    {
       cout<< astrNames[i] << " : " << anPancakes[i] << endl;
    }

...ispisuje vrijednosti redom kojim su unesene a u zadatku se trazi da ih rasvrsta i ispisuje po velicini, pocevsi od najvece.

Zato sam to napisao ovako

    //Prolazi kroz svaki element mreze
    for (int nCurrentIndex=0; nCurrentIndex < nSize; nCurrentIndex++)
    {
        // prvi element u mrezi je za sada ujedno i najveci
        int nBiggestIndex = nCurrentIndex;
       
        // pocinjemo traziti dalje, od drugog elementa
        for(int nIndex = nCurrentIndex + 1; nIndex < nSize; nIndex++)
        {
            // ako nademo element koji je veci od do tada najveceg
            if(anPancakes[nCurrentIndex] > anPancakes[nBiggestIndex])
                // tada on postaje najveci
                nBiggestIndex = nCurrentIndex;       
        }  
        // ...i zamjenjujemo prvi element s najvecim, tj. redamo ih po velicini
        swap(anPancakes[nCurrentIndex], anPancakes[nBiggestIndex]);

        for (int nIndex2=0; nIndex2 > nSize; nIndex2++)
        {

        cout << astrNames[nIndex2] << " ate " << anPancakes[nIndex2] << " pancakes.";
        cout << endl;

        }

    }

    delete [] anPancakes;

    delete [] astrNames;
    system("pause");
    return 0;
}

Ali kada je to tako napisano ovaj zadnji for se ne izvodi, ne ispisuje nista. Ocito sam negdje pogrijesio ali ne znam gdje.

I sta je s ovim crtama izmedu redova? Nekada ih napravi a nekada ne

Pa kad si u tom algoritmu pomješao indekse niza sa vrijednostima, zato ne radi.

Riješi to na najjednostavniji način:

- pošto se sortiraju dva niza na osnovu kriterija jednog, umjesto dva niza napravi jedan niz pair<string, int>

- za napraviti algoritam za sortitiranje ne treba otkrivati toplu vodu, uzmi bubble sort algoritam

#include <iostream>
#include <string>
using namespace std;


int main()
{
    cout << "How many contestors in pancake eating are there?" << endl;
    int nSize;
    cin >> nSize;
    pair<string, int>* anPancakes = new pair<string, int>[nSize];
  
    for(int i=0; i < nSize; i++)
    {  
       cout << "Enter the full name of the " << i+1 <<
            ". contestor in pancake eating." << endl;
       cin>> anPancakes[i].first;
       cout << "Enter the number of pancakes that " << anPancakes[i].first <<
            " has eaten." << endl;
       cin >> anPancakes[i].second;
    }

    for(int i=0; i < nSize-1; i++)
    {
        for(int j = 0; j<nSize-1-i; j++)
       {
          if(anPancakes[j+1].second > anPancakes[j].second)
          {
             swap(anPancakes[j], anPancakes[j+1]);
          }
       }
    }

    for(int i= 0; i<nSize; i++)
    {
       cout<< anPancakes[i].first << " : " << anPancakes[i].second << endl;
    }
  
    delete [] anPancakes;
    return 0;
}

btw. šteta što se traži strogo korištenje arrraya, inače bi ja ovdje deklarirao pair<string, int> vector, a onda preko iteratora upotrijebio sort metodu iz STL-a, uz jednu bool funkciiju da odredim način sortiranja.

 A što bi bilo da moraš učiti HeapSort ili Merge Sort?

 Zašto onda ih nisi objasnio u knjizi?

Zato što imaju jako puno coda, a taj code zahtjeva malo više znanja i razumijevanja nego što se došlo s gradivom u knjizi (rekurzije, binarna stabla itd). To je za malo napredije, ali kada se dođe do te razine nije ih problem shvatiti tj. logiku kojom oni rade.