Jak zrobić, żeby program wyświetlał tylko spełniony warunek?

Mam zrobić program obliczający wartość premii do sprzedanych aut, niby program napisałem ale nadal jest coś nie tak bo ma zwracać tylko 1 wartość a mi wyświetla wszystko. Pomoże ktoś?

#include <iostream>
using namespace std;
int main () {

unsigned int k;
int p;
cin>>k;
if(k>0 || k<10)
{
	p=0;
	cout<<"Wysokosc premii: "<<p<<endl;
}
else (k>10||k<20);{

p=500;
cout<<"Wysokosc premii: "<<p<<endl;
}
if (k>20||k<=30)
{
	p=750;
	cout<<"Wysokosc premii: "<<p<<endl;
}
else (k>30);
{
	p=1000;
	cout<<"Wysokosc premii: "<<p<<endl;
}
system("pause");
return 0;
}

else (k>10||k<20);{

Coś tutaj (i paru innych miejscach) jest zbędne.

Chodzi Ci o || czy zły zapis warunku jest?

Nie, warunek jest poprawny.

No nie wiem, przecież jak nie dam średników to program nie działa, jeśli nie dam {} to działa tak samo jak wcześniej. Jeśli daje && zamiast || to jest tak samo :D

if(k>0 || k<10)

1. To wykona Ci się zawsze gdy k > 0 LUB k < 10 czyli dla każdej wartości k.

else (k>10||k<20);{

2. Tego średnika być tu nie powinno i patrz 1)

if (k>20||k<=30)

Zobacz 1)

else (k>30);

Kolejny średnik.

No tak ale jak tylko te średniki usunę to program mi nie działa bo nie ma średników

Bo w else nie ma czegoś takiego jak warunek. Do tego używa się else if.
else ma być już ostatnią opcją, kiedy wszystkie if i else if nie wykonały się.

Przykład:

int number = 5;

if (number == 0)
{
	cout << "Number cannot be 0";
}
else if (number < 0)
{
	cout << "Number cannot be negative";
}
else if (number > 6)
{
	cout << "Number cannot be greater than 6";
}
else
{
	cout << "Number is valid";
}

Wynikiem tego programu będzie

Number is valid

Dzięki wielkie. Już działa wszystko. Czyli taki zapis jest poprawny?

#include <iostream>
using namespace std;
int main () {

unsigned int k;
int p;
cin>>k;
if(k>=0 && k<=10)
{
	p=0;
	cout<<"Wysokosc premii: "<<p<<endl;
}
else if (k>10&&k<=20){

p=500;
cout<<"Wysokosc premii: "<<p<<endl;
}
else if (k>20&&k<=30)
{
	p=750;
	cout<<"Wysokosc premii: "<<p<<endl;
}
else 
{
(k>30);

	
	p=1000;
	cout<<"Wysokosc premii: "<<p<<endl;
}
system("pause");
return 0;
}

Ostatni else if jest zły:

else 
{
(k>30);

Powinno być:

else if (k > 30)
{
	...
}

Naucz się formatowania kodu, zobacz jaki skrót klawiszowy odpowiada za automatyczne formatowanie kodu i trzymaj się tego stylu.

A nie łatwiej tak?

int p = 0;
if ( k > 30 ) { p = 1000; }
	else if ( k > 20 ) { p = 750; }
	else if ( k > 10 ) { p = 500; }
	else { p = 0; }

cout << "Wysokosc premii: " << p << "\n";

Wychodzi na to, że łatwiej :D. Dzięki.