Куда теряется 1 в цикле при работе с мк?

Question

[Ingernirated]

8 портов - 8 светодиодов, первое зажигание на i == 0
И на сколько я понимаю, когда i == 7 и выполняется зажигание последнего(8 диода), i становится 8 и в проверке должно поменяться flag на 1 и сделать i == 6, чтобы загорелся предпоследний светодиод. (бесконечная дорожка из светодиодов), но всё почему-то не так.

int main(void)
{
	DDRD = 0xFF;
	PORTD = 0x00;
	
	char i = 0;
	char flag = 0;
    while(1) 
    {
			
		if(flag == 0) {
			PORTD = (1 << i);
i++;
		}
		if(flag == 1) {
			PORTD = (1 << i);
i--;
		}
		
		if(i == 8) {
			flag = 1;
                        i--;
		}
		if(i == 0) {
			flag = 0;
                        i++;
		}
		
	_delay_ms(200);
	}
}

А вот этот код работает, как я и хотел, только я не пойму, почему

int main(void)
{
	DDRD = 0xFF;
	PORTD = 0x00;
	
	char i = 0;
	char flag = 0;
    while(1) 
    {
			
		if(flag == 0) {
			PORTD = (1 << i);
			i++;
		}
		if(flag == 1) {
			PORTD = (1 << i);
			i--;
		}
		
		if(i == 7) {
			flag = 1;
		}
		if(i == 0) {
			flag = 0;
		}
		
	_delay_ms(200);
	}
}

Answer 1

[Сергей Сергей]

В части с переключение флага исчезли изменения счетчика, там терялось значение 0 для него.

Comments

[Ingernirated]

Ещё раз, помогите, пожалуйста логику понять

Установили все значения в 0.

DDRD = 0xFF;
	PORTD = 0x00;
	
	char i = 0;
	char flag = 0;

Начинается бесконечный цикл.

if(flag == 0) {
			PORTD = (1 << i);
			i++;
		}

У нас 8 светодиодов.

1-ый цикл i = 0, горит по условию лампа 0000 0001
i становится 1
2-ый цикл i = 1, горит по условию лампа 0000 0010
i становится 2
3-ый цикл i = 2, горит по условию лампа 0000 0100
i становится 3
4-ый цикл i = 3, горит по условию лампа 0000 1000
i становится 4
5-ый цикл i = 4, горит по условию лампа 0001 0000
i становится 5
6-ый цикл i = 5, горит по условию лампа 0010 0010
i становится 6
7-ый цикл i = 6, горит по условию лампа 0100 0000
i становится 7
8-ый цикл i = 7, горит по условию лампа 1000 0000
i становится 8

Выполняется проверка и на 8-ом цикле, когда i было ровно 7 и зажгло последний диод, увеличила следующей строкой i до 8 и

if(i == 8) {
			flag = 1;
			i - 2;
		}

i становится равным 6 и срабатывает
цикл i = 6, горит по условию лампа 0100 0000
и светодиод идёт в по циклу i-- в обратном направлении.

Что я не так понял?

[Сергей Сергей]

Если проверять на ==8 и отнимать 2, то так и будет.

В другом варианте была проверка на ==7 без изменения счетчика, там в цикле переход на 6 по условию на уменьшение проходит самостоятельно. Запись в порт нормально стоит, перед изменением счетчика.

Answer 2

[Rsa97]

В первом варианте у вас в цикле i получает значение 8, затем в начале следующего цикла сначала идёт попытка зажечь девятый светодиод и только потом i уменьшается до 7. Ну и программа написана неэффективно.

int main(void)
{
  DDRD = 0xFF;
  PORTD = 0x00;
  char position = 0;
  char step = -1;
  while(1) {
    PORTD = (1 << position);
    if (0 == position || 7 == position) {
      step = -step;
    }
    position += step;
    _delay_ms(200);
  } 
}

Comments

[Ingernirated]

Ещё раз, помогите, пожалуйста логику понять

Установили все значения в 0.

DDRD = 0xFF;
	PORTD = 0x00;
	
	char i = 0;
	char flag = 0;

Начинается бесконечный цикл.

if(flag == 0) {
			PORTD = (1 << i);
			i++;
		}

У нас 8 светодиодов.

1-ый цикл i = 0, горит по условию лампа 0000 0001
i становится 1
2-ый цикл i = 1, горит по условию лампа 0000 0010
i становится 2
3-ый цикл i = 2, горит по условию лампа 0000 0100
i становится 3
4-ый цикл i = 3, горит по условию лампа 0000 1000
i становится 4
5-ый цикл i = 4, горит по условию лампа 0001 0000
i становится 5
6-ый цикл i = 5, горит по условию лампа 0010 0010
i становится 6
7-ый цикл i = 6, горит по условию лампа 0100 0000
i становится 7
8-ый цикл i = 7, горит по условию лампа 1000 0000
i становится 8

Выполняется проверка и на 8-ом цикле, когда i было ровно 7 и зажгло последний диод, увеличила следующей строкой i до 8 и

if(i == 8) {
			flag = 1;
			i - 2;
		}

i становится равным 6 и срабатывает
цикл i = 6, горит по условию лампа 0100 0000
и светодиод идёт в по циклу i-- в обратном направлении.

Что я не так понял?

[Rsa97]

Ingernirated: i - 2 только вычисляет значение, но никуда его не записывает. Чтобы изменить переменную, надо написать i = i - 2

[Ingernirated]

Rsa97: спасибо, подскажи, пожалуйста, ты этот код сам придумал или это последствие чтения книги алгоритмов?
просто как-то одну начинал читать, но там как-то сразу жестко непонятно всё началось(может для продвинутых попалась), не подскажешь для начинающих какую-нибудь?

[Rsa97]

Ingernirated: Сам. У меня с алгоритмизацией обычно проблем нету.
Литературу не подскажу, начинал я очень давно, на ассемблере и совсем на других компьютерах. Могу только посоветовать научится пользоваться отладкой, пошагово прослеживать работу алгоритмов. Ну и понять, как программа и данные размещаются в памяти, как работают указатели. Тогда будет гораздо проще в любых языках программирования, поскольку все они по сути своей одинаковы.

[Ingernirated]

Rsa97: т.е. начать с ассемблера и на нём всё это сначала пытаться писать?

[Rsa97]

Ingernirated: Не обязательно писать на ассемблере. Многие компиляторы имеют возможность сгенерировать код на ассемблере из программы на C. Можно генерировать такой код для своих небольших программ и изучать его, видя одновременно строку на языке высокого уровня и её представление в машинных командах.

[Ingernirated]

Rsa97: это возможность есть в любом компиляторе или как это делают?
пытался сейчас найти это в Atmel Studio 7, не получилось.

[Rsa97]

Ingernirated: Это от компилятора зависит. В Atmel Studio вроде ассемблерный код можно увидеть только при дебаге или отдельной программой ReAVR