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

Подскажите, пожалуйста, как объединить код, с помощью какой функции? Заранее большое спасибо.
На основе 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: На форуме уже есть тема по кнопкам, посмотрите, может чего подчерпнёте оттуда - viewtopic.php?f=61&t=79801

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

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

PCBWay - всего $5 за 10 печатных плат, первый заказ для новых клиентов БЕСПЛАТЕН

Сборка печатных плат от $30 + БЕСПЛАТНАЯ доставка по всему миру + трафарет

Онлайн просмотровщик Gerber-файлов от PCBWay + Услуги 3D печати

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

Организация питания на основе надежных литиевых аккумуляторов EVE и микросхем азиатского производства

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

Подробнее>>

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

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;
}

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

Новый аккумулятор EVE серии PLM для GSM-трекеров, работающих в жёстких условиях (до -40°С)

Компания EVE выпустила новый аккумулятор серии PLM, сочетающий в себе высокую безопасность, длительный срок службы, широкий температурный диапазон и высокую токоотдачу даже при отрицательной температуре. Эти аккумуляторы поддерживают заряд при температуре от -40/-20°С (сниженным значением тока), безопасны (не воспламеняются и не взрываются) при механическом повреждении (протыкание и сдавливание), устойчивы к вибрации. Они могут применяться как для автотранспорта (трекеры, маячки, сигнализация), так и для промышленных устройств мониторинга, IoT-устройств.

Подробнее>>

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

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

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

почти:
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;
}

Спасибо большое. Получается в свитч операторе пишется условие для 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?

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

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

вы прочитали про антипаттерны? про магические числа?
никакого смеха - всё, что написано по ссылке - это плохо. не делайте плохо сразу, чтобы потом не переучиваться.

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

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

Добавлено after 10 minutes 45 seconds:
Конечно в майне (или в прерывании, или в подпрограмме), так просто "валяются" (вне функций) всякие #define, #include, описания глобальных переменных... но не сам код.
А если хочешь чтобы он не один раз выполнился (так обычно производятся первоначальные настройки железа), а работал постоянно - найди в майне цикл while (1) {}; (он внизу) и впиши в него.

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;
}

}
}


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

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

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

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

Добавлено after 1 minute 2 seconds:
зачем вы новичка учите плохому стилю программирования? научите хорошему, плохо он и без вас сделает.

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

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

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

{
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. Подскажите, пожалуйста, что добавить в свитч.