Jeg fandt et par småting
#include <iostream>
#include <stdlib.h>
using namespace std;
int main() // dette program skal finde gennemsnittet af 3 tal. der vil dog komme mulighed for at vælge sit eget nummer af tal fra 2-10
{
int number1,number2,number3;
char cChar;
double gennemsnit;
do
{
system("CLS");
cout << "Vælg venligst hvor mange tal du vil finde gennemsnittet af" << " (3)" << endl;
cin >> cChar;
switch (cChar)
{
case '3':
cout << "indtast venligst det foerste nummer, og saa enter" << endl;
cin >> number1;
cout << "indtast venligst det andet nummer, og saa enter" << endl;
cin >> number2;
cout << "indtast venligst det tredje nummer, og saa enter" << endl;
cin >> number3;
cout << "gennemsnittet er: " << number1 << " + " << number2 << " + " << number3 << " = " << (number1 + number2 + number3 / 3.0) << endl;
}
}
while(0);
system("PAUSE");
return 0;
}Ændringer:
1.
Tilføjede
using namespace std;
på linien under
#include <iostream>
for at kunne bruge cout, cin og endl uden at skulle skrive dem som std::cout etc.
2.
Fjernede ; efter switch(cChar)
Kontrolstrukturer som switch(), if(), etc. skal ikke have ; efter sig - det ville skille dem fra det de skulle styre.
3.
cout << "gennemsnittet er: " << number1 << " + " << number2 << " + " << number3 << " = " << (number1 + number2 + number3 / 3.0) endl;
Manglede et << før endl;
Og lidt mindre iøjnefaldende: (number1 + number2 + number3 / 3.0) svarer til (number3 / 3.0) + (number1 + number2) fordi divisionsoperatoren har højere præcedens.
4.
Der mangler en while() med en eller anden stopbetingelse til at afslutte do{...}. Jeg satte while(0); ind lige for at få den til at compile.
5.
Includede <stdlib.h> for at kunne bruge system().
Indlæg senest redigeret d. 23.04.2009 16:10 af Bruger #14919