Нажатие кнопки не отрабатывает...

[rattlesnake]

Помогите начинающему..
И так имеем: ATMEGA8 к PB0 и PB1 подключены лампочки которые поочередно моргают.
Вешаю на PB2 кнопку.
Хочу: после нажатия кнопки на секунду лампочки потухли.
Явно ответ где-то на поверхности - плз, гляньте..

#include ‹avr/io.h›
#include ‹util/delay.h›
int main (void){
	while (1) {
// 2 бит в ноль те вход; 0 и 1 - выход
		DDRB=0b00000011;		
		PORTB=0x01;
		_delay_ms(500);
// ни один из трех вариантов не работает..
//		if ( bit_is_set(PINB,0x04) ) {
//		if ( bit_is_set(PORTB,0x04)  ) {
		if ( PINB2 == 0 ) {
		PORTB=0x00;
		_delay_ms(1000);
		}
		PORTB=0x02;
		_delay_ms(500);
	}
return 0;
}

ЗЫ. Пробывал разные значения вместо 0x04 - безрезультатно...

[AndreiVV]

И так имеем: ATMEGA8 к PB0 и PB1 подключены лампочки которые поочередно моргают.

чего-то не видно чтобы они моргали и если это действительно лапочки то можно и мк спалить

int main (void){ 
  
      DDRB=0b00000001;// 0 бит   - выход              
 while (1) { 
       
 PORTB^=0x01; //изменяем значение на противоположное
  _delay_ms(500); 

 if ( PINB2 == 0 ) {//если нажата кнопка ждем 1с
    PORTB=0x00; 
    _delay_ms(1000); 
   } 
  } 
return 0; 
}

это только для одного светодиода второй сими добавьте

[tempora]

Код:

int main (void){ ... PORTB^=0x01; //изменяем значение на противоположное ... return 0; }

Менять значение на противоположное в атмелах экономичнее (во всех отношениях - регистров, времени) таким способом:

int main (void){ 
 ...  
 PINB.‹номер пина›=1; //изменяем значение на противоположное
 ...
return 0; 
}

[tempora]

Помогите начинающему..

Код:

int main (void){ while (1) { ... // ни один из трех вариантов не работает.. // if ( bit_is_set(PINB,0x04) ) { // if ( bit_is_set(PORTB,0x04) ) {

Когда кнопка нажата, соответствующий ей бит в прочитанном байте, ну, никак не is_set... Действительно, на поверхности...

[rattlesnake]

Оно, конечно, спасибо, но но у меня аналогичный код, и аналогичная проверка, но на кнопку не реагирует..
Может просто я схемку не правильно сделал (ну не бейте сильно - начинаю)

-- Прилагается рисунок: --

[AndreiVV]

PINB.‹номер пина›=1; //изменяем значение на противоположное

это не изменяем значение на противоположное а просто установка 1 то есть значение не зависит от предыдущего значения
в моем варианте: было 0 -› станет 1, было 1 -› станет 0
в вашем варианте: было 0 -› станет 1,было 1 -› станет 1

[AndreiVV]

Может просто я схемку не правильно сделал

1. светодиод обыно к + подключают(но не принципиально)
2.

if ( PINB2 == 0 )

=если ноль то, у вас никогда не будет нуля тк при нажатии на кнопку у вас будет + пит. тоесть 1. Надо подключить к GND а не к VCC и поставить pullup с вывода мк на VCC или вкл внутренний pullup

[tempora]

Код:

PINB.‹номер пина›=1; //изменяем значение на противоположное

это не изменяем значение на противоположное а просто установка 1 то есть значение не зависит от предыдущего значения:
было 0 -› станет 1,было 1 -› станет 1

Очень советую попробовать, прежде, чем спорить.

... у вас никогда не будет нуля тк при нажатии на кнопку у вас будет + пит.

Да, я тоже был уверен, что кнопка давит на массу.

[AndreiVV]

Очень советую попробовать, прежде, чем спорить

я не спорю просто поспешил с ответом, перепутал PINB с PORTB. Вот проверил, и значение не меняится

Прикрепленный файл: 9493012.rar

[rattlesnake]

Всем спасибо - все заработало:
1. повесил кнопку на GRD и через 10кОМ на МК.
2.

int main (void){
		DDRB=0b00000011;
		PORTB|=_BV(2);
	while (1) {
		PORTB=0x01;
		_delay_ms(250);
		PORTB=0x02;
		_delay_ms(250);
		if ( bit_is_clear(PINB, 2) ) {
		PORTB=0x00;
// что бы видно было..
		_delay_ms(250);
		_delay_ms(250);
		_delay_ms(250);
		_delay_ms(250);
		}
	}
return 0;
}