вывод массива чисел

[texac123]

Привет всем , в программировании новечек, есть задание:
Реализовать МПС на базе МК фирмы Atmel серии AVR, которая принимает с
компьютера по UART 16 случайных чисел (байт). Произвести обработку принятых чисел
на Микро Контроллере согласно заданию. Исходник компилировал в CodevisionAWR. Язык программирования С.

Есть фрагмент исходника , код работает, должен выводить массив 16 чисел который ввожу я, а он выводит совсем не то число..
Я приложил скрины, слева в Virtual terminal массив чисел которые ввожу я, передаю по COM порту, а справа на LCD должно быть максимальное число 208. А появляется число 56... Где я допустил ошибку. Исходник ниже
Hel[P!!! Спасибо !

// инициализация внешних библиотек
#include <mega8535.h>
#include "lcd.h"
#include "delay.h"
#include "stdlib.h"
//Инициализация порта А для передачи данных в ЖКИ.
//Для просмотра номера порта, нужно обратиться к заголовочному //файлу mega8535.h, с
целью определения номера порта.
//PortA = 0x1b;
#asm
.equ __lcd_port=0x1b; //PORTA
#endasm
//объявление служебных переменных
//однобайтный буфер
volatile unsigned char usartRxBuf = 0;
//Определение констант для конфигурации UART
#define RXCIE 7
#define RXEN 4
#define TXEN 3
#define URSEL 7
#define UCSZ1 2
#define UCSZ0 1
unsigned char MAXnum,buff = 0;
unsigned char i,j,n = 0;
unsigned char mas1 [16];
char* str1;
//Процедуры для управления передачей через UART
//инициализация usart`a
void USART_Init(void)
{
UBRRH = 0;
//скорость обмена 9600 бод
UBRRL = 51;
//разр. прерыв при приеме, разр приема, разр передачи.
UCSRB = (1<<RXCIE)|(1<<RXEN)|(1<<TXEN);
//размер слова 8 разрядов
UCSRC = (1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0); }
#define UDRE 5
//отправка символа по usart`у
void USART_SendChar(unsigned char sym)
{
while(!(UCSRA & (1<<UDRE)));
UDR = sym;
}
//чтение буфера
unsigned char USART_GetChar(void)
{
unsigned char tmp;
unsigned char saveState = SREG;
#asm("cli");
tmp = usartRxBuf;
usartRxBuf = 0;
SREG = saveState;
return tmp;
}
//прерывание при приеме символа по usart в буфер
interrupt [USART_RXC] void usart_rxc_my(void)
{
usartRxBuf = UDR;
buff = usartRxBuf;
if(buff)
{
if(i<16)
{
mas1[i] = buff;
i++;
}
}
}
void main( void )
{
lcd_init(20); //инициализация ЖКИ
USART_Init(); //инициализация UART
lcd_clear(); //очистка ЖКИ
lcd_gotoxy(0,0); //переместить курсор на ЖКИ в позицию 0,0
#asm("sei"); //разрешение прерываний
while(1) //бесконечный цикл
{
if(i == 15)
{
//обработка массива
for(i=0;i<16;i++)
{
if (mas1[i]%2 == 0)
{
if (MAXnum<=mas1[i])
{
MAXnum = mas1[i];
}
}
}
//вывод массива на ЖКИ
for(i=0;i<16;i++)
{
itoa(mas1[i],str1);
lcd_puts(str1);
j++;
if(j%4!=0)
{
lcd_gotoxy(j*5,n);
}
else
{
j = 0;
n++;
lcd_gotoxy(j*5,n);
}
}
delay_ms(10000);
//вывод результата на ЖКИ
lcd_clear();
lcd_gotoxy(0,0);
itoa(MAXnum,str1);
lcd_puts(str1);
}
}}

[texac123]

скрины

[digitalis]

Все вооружились лупами и бросились рассматривать, что там на экранчике накорябал Протеус.
Разбей задачу на 2 части: поиск четного максимума и передача заданного числа.
Навскидку: не вижу начального присваивания i = 0 и Maxnum.
Если не заниматься ###ом в Протеусе, с помощью отладчика в Студии ошибка находится минуты за 3.
------------------
И что-то я не въеду: в одном модуле - и передача, и прием по UART? Я так думал, на компе должна крутиться программа генерации рандома и передачи, на МК - приема и отображения. Это 2 разных проги в двух разных средах программирования. Или это какой-то виртуальный терминал = комп?
А судя по тому, что видим на правом экранчике - 56 это и есть нормальный максимум. Только чисел всего 13 - это, похоже, следствие того, что (см.выше) .
Проще всего забить массив 16 известными значениями и прокрутить прогу от //обработка массива. Насчет виртуального терминала ничего сказать не могу, никогда такой хней не пользовался.
------------------------------
В поиске максимума ошибок нет, кроме (см. выше), да тут и ошибиться невозможно. Фигурных скобок {} лишняя куча, но это не ошибка.

for(i=0;i<16;i++)
    if (mas1[i]%2 == 0)
       if (MAXnum<=mas1[i])  MAXnum = mas1[i];

Или это у CV так принято ?

Царица Екатерина в слове из 3 букв делала 4 ошибки: "исчо" Но ты уже близко: новечек. Ну разве что забугорный иностранец? В связи с веяниями на форуме - местоположение участника - top secret

[Desc]

Сообщение от digitalis

Или это какой-то виртуальный терминал = комп?

В "Протеусе" есть виртуальный терминал для тестирования проектов. Удобная фишка.

[texac123]

На компьютере прога для передачи информации по COM порту, в нее записываю 16 чисел , и отправляю их по COM порту на Virtual Terminal, там они и отображаются, дело в том что код готовый был из метод, по сути должен работать...
Если не трудно можешь указать поточнее где именно ошибка у меня? в С новечек...несколько дней уже пытаюсь с этим кодом что то сделать.
Спасибо !

[digitalis]

Насчет виртуального терминала ничего не могу сказать, не в курсе, мне Шпротес был без надобности - все легко отлаживалось в Студии и проверялось на железе.
Поэтому почему записалось 13 чисел, а не 16 - х.з. Само определение максимума правильное - 56 он и есть.
На этом форуме не приветствуются ссылки на другие, даже не конкурирующие, поэтому сделаем так: я знаю место, где мучаются дурью тусуются любители Шпротеуса, напиши на digitun@tut.by. Они смогут объяснить, наверно, каким боком эта хрень слилась с виртуального терминала не по задумке. А в остальном прога сработала правильно.
PS А слово новичок пишется вот так.