Подключение нескольких кнопок к ATmga8A

mnirti

Доброго всем дня.

Подскажите, пожалуйста, как объединить код, с помощью какой функции? Заранее большое спасибо.
На основе ATmga8A собрана схема: 4 кнопки через резисторы подсоединены к 4 портам В на вход, на выходе D есть семисегментный индикатор. При не нажатой кнопке горят все сегменты, при нажатии первой кнопки выводится число 0, второй - 1, третьей - 2, четвертой - 3. По отдельности код работает исправно, но как объединить, не знаю.

Код для вывода числа 2 с помощью третьей кнопки:
#include <avr/io.h>

#define F_CPU 8000000L
#include <avr/io.h>
//#include <util/delay.h>

int main(void)
{
unsigned char butcount=0;

DDRD = 0xFF;
DDRB = 0x00;
PORTD = 0b00000000; // 1 2 3 4 5 6 7 8
//PORTB = 0b00000001;// 0b|dp|g|f|e|d|c|b|a
//PORTB = 0b00000010;
PORTB = 0b00000100;
//PORTB = 0b00001111;
//PORTB = 0b00001111; Запитаны 4 ноги порта В, чтобы 4 кнопки реагировали.

PORTD = 0b0111111; //0
PORTD = 0b0000110; //1
PORTD = 0b1011011; //2
PORTD = 0b1001111; //3
PORTD = 0b1111111; //8

while (1)
{
if (!(PINB&0b0111111))
{
PORTD = 0b10100100;
}
else
{
PORTD = 0b00000000;
}

}
}

Реклама

Не понятно, что Вы и с чем хотите объединить. Поподробнее объясните.

PS: На форуме уже есть тема по кнопкам, посмотрите, может чего подчерпнёте оттуда - http://radiokot.ru/forum/viewtopic.php?f=61&t=79801

Реклама

mnirti

Нужно объединить коды, с помощью которых кнопки выводят определенные заданные цифры, в один код. То есть на схеме 4 порта (начиная с верхнего №14) должны быть запитаны одновременно. При нажатии на верхнюю кнопку индикатор показывает 0, на следующую - 1 и т. д. Если объединяю условия if по отображению чисел в цикл while, то программа Atmel рассматривает код как неправильный. И как сделать так, чтобы 4 порта
PORTB = 0b00000001;
PORTB = 0b00000010;
PORTB = 0b00000100;
PORTB = 0b00001000;
в коде были подключены, и не ругалась на это программа?

В Вашей ссылке, к сожалению, не совсем та информация, которая мне нужна на этот случай. Мне бы понять, как собрать код из четырех для вывода разных цифр, и в Atmel получился бы корректный hex.

Ivanoff-iv

вложи каждую следующую проверку в елзе предыдущей, а все сегменты зажигай только внутри елзе самой вложенной проверки.
или на свитч — кейс перепиши, там всего 1 раз условие вводить надо будет.

Реклама

mnirti

То есть если на свитч, то:

void segchar (unsigned char seg)
{
switch (seg)
{
case 1: PORTD = 0b11000000; break;
case 2: PORTD = 0b11111001; break;
case 3: PORTD = 0b10100100; break;
case 4: PORTD = 0b10110000; break;
}
}

А если else:

else
{
{
else
}
PORTD = 0b00000000;
}

Или опять совсем не так?

Реклама

ARV · Сообщение **ARV** » Чт май 25, 2017 10:56:18

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

Ваша задача крайне просто решается, просто у вас нет системного подхода к решению.

Реклама

mnirti

Спасибо за статью. Буду делать дальше, обязательно напишу, что получилось или не получилось. Думаю, будут еще вопросы по этой задаче, так как только начинаю учить программирование, пока что слабо в нем ориентируюсь.

Ivanoff-iv

почти:
switch (PINB|0x0F)
{
case 0: PORTD = 0b00000000; break;
case 1: PORTD = 0b11000000; break;
case 2: PORTD = 0b11111001; break;
case 4: PORTD = 0b10100100; break;
case 8: PORTD = 0b10110000; break;
}

mnirti

Спасибо большое. Получается в свитч операторе пишется условие для 5 вариантов, включая случай, когда индикатор показывает все сегменты:

switch (PINB|0x0F)
{
case 0: PORTD = 0b00000000; break;
case 1: PORTD = 0b11000000; break;
case 2: PORTD = 0b11111001; break;
case 3: PORTD = 0b10100100; break;
case 8: PORTD = 0b10110000; break;
}

И половину битов выставляем со значением 1. А что тогда написать в while? Если остается только одно условие, цифры для индикатора помещены в свитч, то там должен быть код на нажатие кнопки? Свитч лучше помещать до или после main?

ARV · Сообщение **ARV** » Чт май 25, 2017 14:15:27

mnirti писал(а):switch (PINB|0x0F)
{
case 0: PORTD = 0b00000000; break;
case 1: PORTD = 0b11000000; break;
case 2: PORTD = 0b11111001; break;
case 3: PORTD = 0b10100100; break;
case 8: PORTD = 0b10110000; break;
}

mnirti

Подскажите, пожалуйста, почему нельзя выводить цифру 3?

ARV · Сообщение **ARV** » Чт май 25, 2017 14:28:26

mnirti писал(а):почему нельзя выводить цифру 3

потому что

mnirti

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

ARV · Сообщение **ARV** » Чт май 25, 2017 14:43:16

[uquote="mnirti",url="/forum/viewtopic.php?p=3113395#p3113395"]Не надо смеяться, подскажите, в чем ошибка.[/uquote]
вы прочитали про антипаттерны? про магические числа?
никакого смеха - всё, что написано по ссылке - это плохо. не делайте плохо сразу, чтобы потом не переучиваться.

Ivanoff-iv

да, это, так,

хотел просто пример привести использования свитч - кейс (но, чтобы у ТС возникли вопросы, а времени, чтобы по уму пример сделать не хватило)
что я накосячил 4 = 0b00000100, а можно написать (1<<2) это значит 1 сдвинутая на 2 бита - так удобней искать нужную ногу (такая запись какраз и обозначит, в контексте примера, PINB.2).
в этом примере также не учтено, что при нажатии кнопки она делается 0, а не 1 (т.е. "условие" надо сообразно твоей задаче написать).

Добавлено after 3 minutes 59 seconds:
в свитч - кейс всё просто - какой кейс по значению равен условию, тот и выполняется, если ни один не равен - выполняется default:, если он есть.

Добавлено after 10 minutes 45 seconds:
[uquote="mnirti",url="/forum/viewtopic.php?p=3113371#p3113371"]Свитч лучше помещать до или после main?[/uquote] Конечно в майне (или в прерывании, или в подпрограмме), так просто "валяются" (вне функций) всякие #define, #include, описания глобальных переменных... но не сам код.
А если хочешь чтобы он не один раз выполнился (так обычно производятся первоначальные настройки железа), а работал постоянно - найди в майне цикл while (1) {}; (он внизу) и впиши в него.

mnirti

ARV, Ваша статья хорошая, но достаточно сложная для новичка. К тому же я знаю еще очень мало функций и их сочетаний.

Ivanoff-iv, спасибо за помощь. Свитч теперь в while, получается так:

while (1)
{
{
if (!(PINB&(1<<PINB0)))

butcount=0;

else
{
if (!(PINB&(1<<PINB1)))

butcount=1;

else

{
if (!(PINB&(1<<PINB2)))

butcount=2;

else

butcount=4;
}

}

}

switch (butcount)

{
case 0: PORTD = 0b00000000; break;
case 1: PORTD = 0b11000000; break;
case 2: PORTD = 0b11111001; break;
default: PORTD = 0b11111001; break;
}

}
}

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

Ivanoff-iv

можно без переменной буткоунт и без ИФов обойтись: пинБ прямо в свитч пиши (там же и требуемые ему операции (инверсия, маскирование), а в кейс - требуемые результаты.
В общем — пиши без ИФов.

ARV · Сообщение **ARV** » Пт май 26, 2017 13:15:44

mnirti писал(а):К тому же я знаю еще очень мало функций и их сочетаний.

не откладывая срочно изучайте! Изучайте не столько готовые (из стандартной библиотеки) функции, сколько сам принцип их создания: как описать, где разместить, что такое параметр, какие они бывают и как возвращается результат...

Все это есть в книгах.

Добавлено after 1 minute 2 seconds:

Ivanoff-iv писал(а):пинБ прямо в свитч пиши (там же и требуемые ему операции (инверсия, маскирование)

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

Ivanoff-iv

тогда уж из кейса выносить знакогенератор - собрать для него массив констант со значениями PORTD для каждой цифры и присваивать нужный элемент, например: PORTD=SEG[2]; для вывода "2" на индикатор или даже сюда переменную задействовать. т.е. разделение блоков кода должно быть логически обоснованным, и если пинБ (и номер нажатой кнопки) больше нигде не используется - я считаю такой блок допустимым.

Добавлено after 3 minutes 13 seconds:
а в начале кода const unsigned char SEG[]={0b00010101,0b10001110,0b00000000}//0,1,2 (цифры в массиве от фонаря)

mnirti

Код стал рабочим в таком случае:

{
if (!(PINB&(1<<PINB0)))

butcount=0;

else
{
if (!(PINB&(1<<PINB1)))

butcount=1;

else

{
if (!(PINB&(1<<PINB2)))

butcount=2;

else

if (!(PINB&(1<<PINB3)))

butcount=3;

else
butcount=4;
}

}

}

switch (butcount)

{

case 0: PORTD = 0b11000000; break;
case 1: PORTD = 0b11111001; break;
case 2: PORTD = 0b10100100; break;
case 3: PORTD = 0b10110000; break;
default: PORTD = 0b00000000; break;
}

}
}

Сначала горят все сегменты индикатора, при нажатии на кнопку от нулевой ножки появляется 0, отпускаю кнопку - опять все сегменты. Вторая кнопка - 1, третья - 2, четвертая - 3.

Но никак не получается сделать так, чтобы при нажатии двух кнопок одновременно появлялась цифра. Например, зажаты первая и третья кнопка, на индикатор подается число 5, вторая и четвертая - 6. Подскажите, пожалуйста, что добавить в свитч.