atmega 8, семисегментный индикатор, кнопка.

[глеб материкин]

Здраствуйте читатели форума.  Относительно недавно я начал работать с микроконтроллерами .  Собрав конструкцию из кнопки ,мк и одного индикатора я написал вот такой код : 

 

 

#include <avr/io.h>
#define F_CPU 1000000UL
#include <util/delay.h>

int main(void)
{
     int button = 0;
     int i =0;
    DDRC = 0xFF;
    DDRB = 0xFF;
    DDRD = 0x00;
    PORTD = 0b000001000;
void segchar (unsigned char seg)
    {
     switch(seg){
     case 0:PORTB = 0b0110111; PORTC = 0b0000001;  break;
     case 1:PORTB = 0b0000001; PORTC = 0b0000001;  break;
     case 2:PORTB = 0b1111011; PORTC = 0b0000000;  break;
     case 3:PORTB = 0b0011011; PORTC = 0b0000001;  break;
     case 4:PORTB = 0b0001101; PORTC = 0b0000001;  break;
     case 5:PORTB = 0b0011110; PORTC = 0b0000001;  break;
     case 6:PORTB = 0b0111110; PORTC = 0b0000001;  break;
     case 7:PORTB = 0b0000011; PORTC = 0b0000001;  break;
     case 8:PORTB = 0b0111111; PORTC = 0b0000001;  break;
     case 9:PORTB = 0b0011111; PORTC = 0b0000001;  break; 
     case 10:PORTB = 0b0111110; break;
     }
            
    }    
    while (1) 
    {
        
        if (!PIND&&button==0)
        { _delay_ms(100);
            i++;
            button = 1;
            
        }
        if (PIND&&button==1)
        {
            _delay_ms(100);
            button = 0;
        }

     if (i==0)
     {
         segchar(0);
     }
   if (i==1)
   {
      segchar(1);
   }
     if (i==2)
     {
        segchar(2);
    }
      if (i==3)
      {
          segchar(3);
      }
       if (i==4)
            {
                segchar(4);
            }
     if (i==5)
          {
              segchar(5);
          }
     if (i==6)
          {
              segchar(6);
          }
      if (i==7)
          {
         segchar(7);
       }
      if (i==8)
      {
          segchar(8);
      }
      if (i==9)
      {
          segchar(9);
      }
      if (i==10)
      {
          segchar(10);
      }
       if (i==11)
       {
           i = 0;
       }

 

        

    
    }
}
 

Код отрабатывает только от 0 до 2 , а дальше не работает(

Подскажите пожалуйста что я написал не так . 

Заранее спасибо )

[Реклама]

Реклама: ООО ТД Промэлектроника, ИНН: 6659197470, Тел: 8 (800) 1000-321

[IMXO]

41 минуту назад, глеб материкин сказал:

Подскажите пожалуйста что я написал не так . 

начиная с

 if (i==0)
     {

и заканчивая

  segchar(10);
      }

код бессмыслен по своей сути
все это заменяется на
 

  segchar(i);

Вангую, не считает выше 2х скорее всего из-за неопределенного состояния на PIND. Зачем Вы опрашиваете весь порт сразу не понятно...

Изменено 28 ноября, 2023 пользователем IMXO

[Реклама]

"Летний фестиваль скидок" ТМ Электроникс!

20% скидка на все электронные компоненты + бесплатная доставка и 15% кешбэк!

Перейти на страницу акции

Реклама: ООО ТМ ЭЛЕКТРОНИКС, ИНН: 7806548420, info@tmelectronics.ru, +7(812)4094849

[skripach]

1 час назад, глеб материкин сказал:

 PORTD = 0b000001000;

 

1 час назад, глеб материкин сказал:

case 0:PORTB = 0b0110111; PORTC = 0b0000001;  break;

Сначала разберись сколько бит в байте 7 или 9.

Изменено 28 ноября, 2023 пользователем skripach

[Реклама]

Аккумуляторы INR21700 от EVE Energy со стандартной и увеличенной емкостью

В Компэл представлены аккумуляторы 21700 с емкостями 4000 мА⋅ч и 5000 мА⋅ч (INR21700-40P и INR21700-50E, соответственно). Аккумуляторы INR21700-50E характеризуются повышенной емкостью и предназначены для  устройств с длительным сроком службы. Для приложений, где требуется экстремальный ток разряда до 30 или 50 А, подойдет аккумулятор INR21700-40P. Аккумуляторы INR21700 предназначены для  электротранспорта, а также для промышленных и бытовых приложений. Подробнее>>

 

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[глеб материкин]

13 часов назад, skripach сказал:

Сначала разберись сколько бит в байте 7 или 9.

Это я уже исправил , хотя код и до этого работал.

14 часов назад, IMXO сказал:

Вангую, не считает выше 2х скорее всего из-за неопределенного состояния на PIND. Зачем Вы опрашиваете весь порт сразу не понятно...

Если не трудно не могли бы подсказать как реализовать по другому ?

[Реклама]

Выбираем схему BMS для корректной работы литий-железофосфатных (LiFePO4) аккумуляторов

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

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[skripach]

15 минут назад, глеб материкин сказал:

как реализовать по другому ?

https://diodov.net/pobitovye-operatsii-programmirovanie-mikrokontrollerov-avr-na-c

[Реклама]

Помехоподавляющие пленочные конденсаторы Hongfa для бытовых и промышленных сетей

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

Реклама: АО КОМПЭЛ, ИНН: 7713005406, ОГРН: 1027700032161

[глеб материкин]

Спасибо всем тем кто откликнулся .Я сумел подключить еще и вторую кнопку . Вот обновленный код:

#include <avr/io.h>
#define F_CPU 1000000UL
#include <util/delay.h>

int main(void)
{
    int button = 0;
    int button1 = 0;
    int i = 0;
    DDRC = 0xFF;
    DDRB = 0xFF;
    DDRD = 0b00000000;
 PORTD = 0b11111111; 
void segchar (unsigned char seg)
    {
     switch(seg){
     case 0:PORTB = 0b0110111; PORTC = 0b0000001;  break;
     case 1:PORTB = 0b0000001; PORTC = 0b0000001;  break;
     case 2:PORTB = 0b1111011; PORTC = 0b0000000;  break;
     case 3:PORTB = 0b0011011; PORTC = 0b0000001;  break;
     case 4:PORTB = 0b0001101; PORTC = 0b0000001;  break;
     case 5:PORTB = 0b0011110; PORTC = 0b0000001;  break;
     case 6:PORTB = 0b0111110; PORTC = 0b0000001;  break;
     case 7:PORTB = 0b0000011; PORTC = 0b0000001;  break;
     case 8:PORTB = 0b0111111; PORTC = 0b0000001;  break;
     case 9:PORTB = 0b0011111; PORTC = 0b0000001;  break; 
     case 10:PORTB = 0b0111110; break;
     }
            
    }    
    while (1) 
    {

  if ((PIND&(1 << PD3))==0&&button==0)
  {     
     i++;
     _delay_ms(100);
     button = 1;
  }
 if (!(PIND&(1 << PD3))==0&&button==1)
 {  _delay_ms(100);
     button = 0;
     
 }

 if ((PIND&(1 << PD4))==0&&button1==0)
 {
     i--;
     _delay_ms(100);
     button1 = 1;
 }
 if (!(PIND&(1 << PD4))==0&&button1==1)
 {  _delay_ms(100);
     button1 = 0;
     
 }

segchar(i);
 
if(i>9){
    
    i = 0;}
if(i<0){
     
 i = 9;}

    }
}

[IMXO]

это
 

if(i<0){
      i = 9;} 

очень плохая практика... и работает только по тому что использована знаковая переменная  int i = 0; к тому же в вашем коде абсолютно не нужна.

У вас  i  изменяется от 0 до 9, это беззнаковое целое размерностью байт, те должно быть unsigned char ( точно такое же, как Вы принимаете в функцию segchar (unsigned char seg).
К тому же у Вас нарушена последовательность действий в коде.
вывод значения i происходит раньше, чем проверка на выход из диапазона
попробуйте делать проверку на диапазон сразу при увеличении/уменьшении переменной

// инкремент i++ 
if(i<9) {i++;}
  else {i=0;}
// дикримент i--
if(i>0) {i--;}
  else {i=9;}

 

Изменено 30 ноября, 2023 пользователем IMXO

[Огонёк]

Можно без проверки:

//инкремент
i=(i+1)%10;
//декремент
i=(i+9)%10;

 

[IMXO]

1 минуту назад, Огонёк сказал:

Можно без проверки:

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

[глеб материкин]

19 часов назад, IMXO сказал:

очень плохая практика... и работает только по тому что использована знаковая переменная  int i = 0; к тому же в вашем коде абсолютно не нужна.

У вас  i  изменяется от 0 до 9, это беззнаковое целое размерностью байт, те должно быть unsigned char ( точно такое же, как Вы принимаете в функцию segchar (unsigned char seg).
К тому же у Вас нарушена последовательность действий в коде.
вывод значения i происходит раньше, чем проверка на выход из диапазона
попробуйте делать проверку на диапазон сразу при увеличении/уменьшении переменной

Спасибо большое за ответ , но я только учусь . А так вижу что у вас отличный вариант , возьму на вооружение