Program nie działa [Instrukcja warunkowa if]

[spreqq]

Co jest źle w tym kodzie?

 

 

#include <iostream>

int main()
{
   int wys, punkty_karne;


 std::cout <<"Na jakiej wysokosci lecimy? [w metrach]: ";
 std::cin >> wys;

if(wys < 500)
{

 std::cout <<"\n" << wys <<"metrow to za niasko!\n"
 punkty_karne = 1;
}
else
{
 std::cout <<"\nNa wysokosci" << wys <<"metrow jestes juz bezpieczny! \n";
 punkty_karne = 0;
}

std::cout <<"Masz" << punkty_karne <<"punktów karnych \n";
id(punkty_karne) std::cout <<"Popraw sie!";
}

[JestemSlepy]

#include <iostream>

int main()
{
int wys, punkty_karne;


std::cout <<"Na jakiej wysokosci lecimy? [w metrach]: ";
std::cin >> wys;

if(wys < 500)
{

std::cout <<"\n" << wys <<"metrow to za niasko!\n"; // tu zapomniałeś zakończyć ";"
punkty_karne = 1;
}
else
{
std::cout <<"\nNa wysokosci" << wys <<"metrow jestes juz bezpieczny! \n";
punkty_karne = 0;
}

std::cout <<"Masz" << punkty_karne <<"punktów karnych \n";
std::cout <<"Popraw sie!";														 // id(punkty_karne) ?
}

 

Nie rozumiem po co tam napisałeś id(punkty_karne).

 

@EDIT

 

Aha, tam miał być if ;p Nie pomyślałem.

[Sopelek997]

brakuje ci srednika i id zamiast if

 

@edit

up szybszy

[spreqq]

Okej - dzięki teraz działa. Jeszcze mam takie pytanie program szybko kończy prace jak go zatrzymać/zwolnić tzn. Wpisuję np. 100 i wyświetla się np. napis "100 metrów to za niasko! (...)" i chodzi o to, że program ma się zatrzymać, a nie zamknąć. Próbowałem system("Pause"), ale coś nie wychodziło.

[Sopelek997]

Bo programy konsole właśnie tak mają działać. Jeśli chcesz, aby się zatrzymał to uruchom go z poziomu cmd, albo, co jest lepszym wyjściem, ogarnij sobie jakieś normalne IDE, np Code::Blocks, czy MVS

[spreqq]

Ja używam Microsoft visual express 2010 to zmieniać, czy pozostać na tym?

[Evergreen]

Pozostań na tym. Używasz dobrego IDE.

[Sopelek997]

IDE jest dobre (spodziewałem się deva, bo najczęściej tam ludzie mają ten problem), możliwe, że masz coś zryte w opcjach, albo coś źle robisz.

Zajrzyj tu http://stackoverflow.com/questions/6137009/microsoft-visual-studio-how-to-keep-the-console-open-without-manually-reading-i

[Symeg]

albo daj na końcu

std::cin.sync();
std::cin.get();

 

lepiej zrobić tak jak napisali poprzednicy, bo program konsolowy ma zrobić to co miał zrobić i się zamknąć.

[AdirasOwns]

Dajesz #include <windows.h> i na końcu Sleep(1200); gdzie zamiast tego 1200 podajesz czas w milisekundach na jaki ma się zatrzymać program.

[krzysiekc++]

widze czytasz symfonie c++ , no ja tez zaczynam ale jestem troche dalej:D

[GGG205]

I radzę Ci dopisać pod bibliotekami using namespace std;

Dzięki temu nie będziesz musiał dopisać std::cout.

[MrHex]

Możesz też dodać include'a:

#include <conio.h>

I gdy chcesz zatrzymać program to piszesz:

getch();

A potem, po wciśnięciu dowolnego klawisza program zacznie się wykonywać dalej.

[rasiel1987]

Dzięki za pomoc.