Uppgift: Fråga användare efter tal och skriv ut summa och medelvärde

Alltså din kod fungerar ju, med några ändringar som nämnts i tråden, #include <climits> och att du behöver ha en brytpunkt i slutet av koden.

Här har jag tagit din kod i första inlägget och lagt till det som nämnts i tråden. Jag har även flyttat ut "cout << "Skriv in tal" << endl;" ur loopen så att det inte skrivs ut varje gång.

#include <iostream>
#include <climits>

using namespace std;

int main(int argc, char *argv[])
{
    int max = INT_MIN;
    int nMax = INT_MIN;
    int min = INT_MAX;
    int sum = 0;
    int antal = 0;
    int inp = 0;
    
    cout << "Skriv in tal" << endl;
    
    while (true) {
        
        cin >> inp;

        if (inp == 0) {
            break;
        }

        if (inp > max) {
            nMax=max;
            max = inp;
        }
        if (inp > nMax && inp < max) {
            nMax = inp;
        }

        if (inp < min) {
            min = inp;
        }
        sum += inp;
        antal++;
    }

    cout << "Summa: " << sum << endl;
    cout << "Medelvärde: " << (double)sum/(double)antal << endl;
    cout << "Störst tal: " << max << endl;
    cout << "Näst störst tal: " << nMax << endl;
    cout << "Minst tal: " << min << endl;

    system("PAUSE");
    return EXIT_SUCCESS;
}

När jag kör den och skriver in siffrorna (ett tryck på enter efter varje siffra) 1, 2, 3, 4, 5 och avslutar med 0 så blir resultatet detta.

Alltså precis vad som efterfrågades i uppgiften, det är dock inte "uppbyggt av ett antal funktioner." men det fungerar, tar in talen, levererar summa, medelvärde, största, näst största.

Vad hade du tänkt att du skulle lära dig på att någon annan serverar lösningen till dig? Jag hoppas ingen är dum nog att lösa hela uppgiften till dig.

Du har själv löst 90%. Ditt program funkar och allt du behöver göra är att bryta ut några delar till några funktioner så är du klar. Att du sedan stirrar upp dig till astronomiska nivåer fast du är så nära fattar jag inte. Men det skulle kunna bero på att någon annan har skrivit den kod som du postar här och att du inte fattar vad den egentligen gör.

Så tillbaka till ritbordet. Den pseudokod du har skrivit är en bra utgångspunkt för att göra funktioner. Typiskt bra funktioner är saker som utförs flera gånger, alternativt block av en eller flera saker som hör ihop och som kan förenkla läsningen av ditt program. Jag tycker dock att du ska undvika funktioner som behöver tillgång till allt för många variabler, för att du i så fall måste skicka väldigt mycket saker fram och tillbaka för att få det att funka bra.

Du har fått tips på en bra funktion innan. Jag tycker att en annan lämplig kandidat är

int ReadANumber()

Nu kommer den inte att göra så mycket om du bara bryter ut den ur din kod, men detta är ett bra ställe att lägga in lite felhantering. Vad händer om användaren skriver ett A istället för en siffra? Kan du lägga in hantering av det tillsammans med lite kommentarer i koden om varför du gör vad du gör så kan du nog få lite bonuspoäng av din lärare. Felhantering är aldrig fel.