Помогите упростить код?

07.02.2009, 13:21. **Показов** 1130. **Ответов** 7

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

Люди пытаюсь писать красиво!
Посоветуйте как можно упростить такой страшный код

C++

        if(!RA3 && !tim[0] && !RC2) tim[0]=0xff;
        if(RA3 && !tim[0] && RC2) tim[0]=0xff; 
            if(RA3 && tim[0] && !RC2) tim[0]=0;
            if(!RA3 && tim[0] && RC2) tim[0]=0; 
        if(tim[0])
            {
            tim[0]--;
            if(!tim[0]) RC2=RA3^1;
            }

@Somebody · 07.02.2009, 15:19

Здесь не все комбинации RA3, tim[0], RC2. Что в остальных случаях?

accept · 08.02.2009, 14:50

C++

    if (!tim[0]) {
        if ((!RA3 && !RC2)
         || (RA3 && RC2))
            tim[0] = 0xff;
    } else {
        if ((RA3 && !RC2)
         || (!RA3 && RC2))
            tim[0] = 0;
    }
    if (tim[0] && --tim[0] == 0)
        RC2 = RA3^1;

@~~rrrFer~~ · 08.02.2009, 22:35

C++

if(!(RA3||RC2)||(RA3&&RC2))
    tim[0]=!tim[0]&&?0xff:0;
if(tim[0]&&!(--tim[0]))
    RC2=RA3^1;

не проверял

Добавлено через 6 часов 49 минут 33 секунды

C++

if(!(RA3||RC2)||(RA3&&RC2))   
    tim[0]=!tim[0]?0xff:0;   
if(tim[0]&&!(--tim[0]))   
    RC2=RA3^1;

лишнее && стояло, исправить сообщение почему-то было нельзя ^(

09.02.2009, 12:14

ОК! Спасибо люди!

Добавлено через 27 минут 37 секунд

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

C++

if(!(RA3||RC2)||(RA3&&RC2))   
    tim[0]=!tim[0]?0xff:0;   
if(tim[0]&&!(--tim[0]))   
    RC2=RA3^1;

Ваше сокрощение не работает.

Добавлено через 30 минут 48 секунд
Быстре всего это будет самый оптимальный вариант.

C++

1
2
3

    if(!tim[0])tim[0]=RA3==RC2?0xff:tim[0];    
else  tim[0]=RA3!=RC2?0:tim[0];    
if(tim[0]&&!(--tim[0]))RC2=RA3^1;

Как Вам?

@Somebody · 09.02.2009, 15:34

Здесь не все комбинации RA3, tim[0], RC2. Что в остальных случаях? Если остальных случаев не может быть, то этот код, по-моему, просто ничего не делает.

@~~rrrFer~~ · 09.02.2009, 16:43

вот у вас
if(!tim[0])
tim[0]=RA3==RC2?0xff:tim[0];
т.е. если RA3!=RC2 то tim[0]=tim[0], не лучше tim[0]=0 ?
может и нет, просто интересно

и вобще зачем тут присваинвание, если значение tim[0] Не изменяется

Добавлено через 42 минуты 18 секунд

C++

1 2	tim[0]=(RA3==RC2&&!tim[0])?0xff:0; if(tim[0]&&!(--tim[0]))RC2=RA3^1;

возможно так

accept · 10.02.2009, 02:53

C++

    if (!tim[0] && (!RA3 == !RC2)){
        tim[0] = 0xff;
        RC2 = RA3^1;    
    } else if (tim[0])
        tim[0] = 0;

RA3 == RC2 нельзя (они могут иметь значения истинные, но не равные)

@Somebody 2835 / 1644 / 254 Регистрация: 03.12.2007 Сообщений: 4,222
	07.02.2009, 15:19	2
	Здесь не все комбинации RA3, tim[0], RC2. Что в остальных случаях? 0

@Somebody 2835 / 1644 / 254 Регистрация: 03.12.2007 Сообщений: 4,222
	09.02.2009, 15:34	6
	Здесь не все комбинации RA3, tim[0], RC2. Что в остальных случаях? Если остальных случаев не может быть, то этот код, по-моему, просто ничего не делает. 0

	10.02.2009, 02:53