Упростить условия

@FortT · Регистрация: 02.01.2015

Author24 — интернет-сервис помощи студентам

Здравствуйте, подскажите, возможно ли как-то упросить данное условие:

C++

if ((XY[empty].position == XY[Nums[i] - 1].position) || (XY[empty].position == XY[Nums[i] + 4].position) || (XY[empty].position == XY[Nums[i] - 4].position))
                            {
                                isMove = true; t = i; 
                            }

Пишу на SFML игру, и получается большой код ибо много таких условий, хотела сделать через истину и ложь условия чтобы хоть как то уменьшить код:

C++

if ((XY[empty].position == (XY[Nums[i] - 1].position || XY[Nums[i] + 4].position || XY[Nums[i] - 4].position))
                            {
                                isMove = true; t = i; 
                            }

Но, к моему сожалению, этого нельзя сделать с координатами. Надеюсь понятно объяснила, у кого есть хоть какие-то идеи, кидайте.

@nonedark2008 · 07.11.2015, 22:59

Сообщение от FortT

возможно ли как-то упросить данное условие

Тут важно знать что есть что и зачем оно там есть.
Что такое XY, empty, Nums, -1, +4, isMove, t?

@AntonIv · 08.11.2015, 01:38

знать бы что означают эти переменные.
Но для упрощения вида можно записать так.
Но переменным first, second, third нужно дать какие-то более информативние названия

C++

    {
        bool first =    (XY[empty].position == XY[Nums[i] - 1].position);
        bool second =   (XY[empty].position == XY[Nums[i] + 4].position);
        bool third =    (XY[empty].position == XY[Nums[i] - 4].position);
 
        if (first || second || third) {
            isMove = true; t = i; 
        }
    }

@ASCII · 08.11.2015, 02:23

Ну самое банальное, если хотите, чтобы Вам не мазолили глаза длинные условия, но нужны именно такие, то засуньте их в функцию. Ну например.

C++

bool isCondition()
{
if ((XY[empty].position == XY[Nums[i] - 1].position) || (XY[empty].position == XY[Nums[i] + 4].position) || (XY[empty].position == XY[Nums[i] - 4].position))
                            {
                                isMove = true; t = i; 
                            }
}

Где Condition - название условия, например isBang - произошел взрыв и тд.
Только придется позаботиться о том, чтобы функция видела isMove и t, XY, empty, Nums и так далее.
Я думаю у Вас это труда не составит.

@FortT · 08.11.2015, 12:45 **[ТС]**

Достаточно знать что XY - массив типа вектор из позиций спрайтов. AntonIv дал хорошую идею как улучшить условие для читабельности, а меня еще интересует можно ли отсюда:

C++

1	if ((XY[empty].position == XY[Nums[i] - 1].position) \|\| (XY[empty].position == XY[Nums[i] + 4].position) \|\| (XY[empty].position == XY[Nums[i] - 4].position))

как либо вынести XY[empty].position что бы не писать его по 10 раз, а перечислять в условие только не повторяющийся значения, хочу максимальный минимализм... С переменными типа вектор просто ранее не работала

Tulosba · 08.11.2015, 12:54

Сообщение от FortT

как либо вынести XY[empty].position что бы не писать его по 10 раз,

создаешь ссылку например:

C++

1	const auto& epos = XY[empty].position;

И дальше используешь epos.

@DrOffset · 08.11.2015, 13:00

FortT,

C++

<your type> const & pos = XY[empty].position;
 
if (pos == XY[Nums[i] - 1].position 
 || pos == XY[Nums[i] + 4].position 
 || pos == XY[Nums[i] - 4].position)
{
    //....

Совсем не писать не выйдет. Но если эта операция часто нужна (т.е. таких мест много), то лучше всего сделать функцию и вынести многословную проверку туда.

@AntonIv · 10.11.2015, 01:48

подсказал бы как короче сделать с помощью итераторов, но не знаю, что такое Nums [i]
а так можно что-то типа такого:

C++

    {
        vector< тип >::iterator iter = XY.begin() + empty;
 
        bool first =    (iter->position == XY[Nums[i] - 1].position);
        bool second =   (iter->position == XY[Nums[i] + 4].position);
        bool third =    (iter->position == XY[Nums[i] - 4].position);
 
        if (first || second || third) {
            isMove = true; t = i; 
        }
    }

Добавлено через 20 минут
Можно еще так..

C++

    {
        тип e = XY[empty];
        vector< тип >::iterator it = XY.begin() + Nums[i];
 
        bool first =    (e == (it - 1)->position);
        bool second =   (e == (it + 4)->position);
        bool third =    (e == (it - 4)->position);
 
        if (first || second || third) {
            isMove = true; t = i; 
        }
    }

@nonedark2008 1394 / 1023 / 325 Регистрация: 28.07.2012 Сообщений: 2,813
	07.11.2015, 22:59	2
	Сообщение от FortT возможно ли как-то упросить данное условие Тут важно знать что есть что и зачем оно там есть. Что такое XY, empty, Nums, -1, +4, isMove, t? 0

	10.11.2015, 01:48