Ошибка оператора условия if в c++?

Question

[avion]

Здравствуйте, почему при использование диапазона значений для переменой, оператор условия впускает любые значения переменой?

#include <iostream>
#include <cstdlib>
#include <cmath>

using namespace std;

int main() {
	int hours, min;

	cout << "Input hours: ";
	cin >> hours;

	cout << "Input min: ";
	cin >> min;

	if (9 <= hours <= 17 & 0 <=  min <= 59) {  //Проблема в этой строке
		if (hours == 13 & min >= 30) {
			cout << "Close";
		}
		else if (hours == 14 & min <= 30) {
			cout << "Close";
		}
		else {
			cout << "Open";
		}
	}
	else {
		cout << "Close";
	}

	system("pause>nul");

	return 0;
}

А вот в таком случаи всё нормально?

#include <iostream>
#include <cstdlib>
#include <cmath>

using namespace std;

int main() {
	int hours, min;

	cout << "Input hours: ";
	cin >> hours;

	cout << "Input min: ";
	cin >> min;

	if (hours >= 9 & hours <= 17 & min >= 0 & min <= 59) {
		if (hours == 13 & min >= 30) {
			cout << "Close";
		}
		else if (hours == 14 & min <= 30) {
			cout << "Close";
		}
		else {
			cout << "Open";
		}
	}
	else {
		cout << "Close";
	}

	system("pause>nul");

	return 0;
}

Comments

[Roman]

глаза ломаются)

хоть так

#include <iostream>

struct time
{
  int hours;
  int mins;
  time(int h = 0, int m = 0) : hours{h}, mins{m}{}
};

bool operator==(time& a, time& b)
{
  return a.hours == b.hours && a.mins == b.mins;
}

bool operator<=(time& a, time& b)
{
  return a.hours <= b.hours && a.mins <= b.mins;
}

bool operator>=(time& a, time& b)
{
  return a.hours >= b.hours && a.mins >= b.mins;
}

bool in_range(time& x, time& a, time& b)
{
  return x >= a && x <= b;
}

int main()
{
    time open{9, 0};
    time close{17, 59};
   
    time lunch_start{13, 30};
    time lunch_end{14, 30};
   
    time input_time{};

    std::cout << "Input hours: ";
    std::cin >> input_time.hours;
    std::cout << "Input mins: ";
    std::cin >> input_time.mins;

    if(in_range(input_time, open, close) && !in_range(input_time, lunch_start, lunch_end))
    {
      std::cout << "Open" << std::endl;
    }
    else
    {
      std::cout << "Close" << std::endl;
    }
    std::cin.get();
}

Answer 1

[JaxxDexx]

if (9 <= hours <= 17 & 0 <= min <= 59)

Объясню подробно почему это не работает.
Допустим hours = 27, а min = 90, тогда

if (9 <= 27 <= 17 & 0 <=  90 <= 59) тогда
if (true <= 17 & true <= 59), тогда т.к. true = 1  
if (1 <= 17 & 1 <= 59) тогда 
if (true & true) => if (1 & 1)
в результате побитового сложения получаем if (1) => true => тело блока выполнится

Второй пример
Допустим hours = 1, а min = -1, тогда

if (9 <= 1 <= 17 & 0 <=  -1 <= 59) тогда 
if (false <= 17 & false <= 59) тогда 
if (0 <= 17 & 0 <= 59) тогда 
if (true & true) тогда 
if (1 & 1)
if (1)
true

Т.е. какие бы значения вы не подставляли, блок выполнится всегда

Comments

[avion]

Получается, что в c++ не возможно использовать данный способ?

[JaxxDexx]

В С++ накостылять можно все что угодно! Но лучше придерживаться стандартных вещей!

Answer 2

[laphroaig]

Я предполагаю нужно так:

if (  (hours>=9 && hours <= 17) && (min >=0 && min <= 59) )

Comments

[JaxxDexx]

laphroaig, скобки внутренние не нужны ;)

[laphroaig]

JaxxDexx, Это для большей читабельности

[JaxxDexx]

laphroaig, как по мне, так только ухудшает читаемость, потому что видя открывающуюся скобку, глаза пытаются найти закрывающуюся, на что тратится время

Answer 3

[Алексей Уколов]

Потому что должно быть &&.
& - это битовое И, а не логическое.