kolego doman89a czy teraz kod sie bardziej podoba czy muszę jeszcze nad nim popracować.W razie jak się nie podoba proszę o przykłady jak już wspominałem jestem początkującym i sczytywanie wciśnięć klawiszy było zaczerpnięte z książki tak jak kod hex czy binarny, wszelkie błędy są niezamierzone i wynikają tylko z mojej niewiedzy.
Kolega proszę może nie z tak dużych liter ok? Ja nie miałem zamiaru się czepiać, tylko w dobrym zamiarze Ci napisałem, bo naprawdę sam zobacz, że już teraz jest o niebo lepiej
Tak by post wisiał nikt by nic nie napisał, daje sobie rękę uciąć (no dobra z tą ręką to przesadziłem 
)Chcesz przykładu, ok tu masz Twój kod:
[syntax=c]/*
* main.c
*
* Created on: 28 Oct 2014
* Author: Marcin Pieta
*/
#include <avr/io.h>
#include <util/delay.h>
#define key_pin PINC
#define key1_pin PC0
#define key2_pin PC1
#define switch_port PORTD
#define switch_pin PIND
#define switchmaster_pin PD6
#define switchslave_pin PD7
uint8_t number_LED[10] = {0xc0,0xf9,0xa4,0xb0,0x99,0x92,0x82,0xf8,0x80,0x90}; //Table for 7 segment display
//USART INITIALIZATION 9600 baud
void USART_Init( unsigned int baud )
{
UCSRC = (1<<URSEL) | (1<<UPM1)|(1<<UCSZ1)|(1<<UCSZ0);
UBRRH = (unsigned char)(baud >> 8);
UBRRL = (unsigned char) baud;
UCSRB = (1<<RXEN)|(1<<TXEN);
}
uint8_t sendchar( char data ) //sends single char over usart
{
while(!(UCSRA&(1<<UDRE)));
UDR = data;
}
uint8_t receivechar( void ) //receive single char over usart
{
while(!(UCSRA&(1<<RXC)));
return UDR;
}
int main( void ) //Start of program
{
uint8_t data = 1;
DDRA = 0xFF; //porta output
DDRC = 0x00; //portd input
DDRD = 0x00; //portd input
PORTA = 0xFF; //all bits high (1)
PORTC = 0xFF; //portc all high (1)
PORTD |= (1<<switchmaster_pin) | (1<<switchslave_pin); //bit7,8 set as high (1)
while(1)
{
if(!(switch_pin & (1<<switchmaster_pin)))
{
PORTA = number_LED[data];
if(!(key_pin & (1<<key1_pin)))
{
_delay_ms(230);
data++;
if(data>5)
data=1;
}
else if(!(key_pin & (1<<key2_pin)))
{
USART_Init(12);
sendchar(data);
}
}
if(!(switch_pin & (1<<switchslave_pin)))
{
USART_Init(12);
PORTA = number_LED[receivechar()];
}
}
}[/syntax]
Porównaj to teraz z Twoim pierwszym który tu był i odpowiedz szczerze który jest przyjemniejszy w analizie?
Co do liczb hexadecymalnych to w małym programie ogarniesz luz, ale wyobraź sobie, że masz program na 1000 linijek kodu, zmieniasz kolejność klawiszy bo tak projekt np. zakłada i teraz sieczka... każdą liczbę hex analizujesz i zmieniasz krok po kroku. Tak więc zobacz na starcie ile chce Ci nerwów zaoszczędzić
Teraz to jak wyżej zmienisz kilka makrodefinicji, a w kodzie automatycznie wszystko masz tak jak chcesz.Pamiętaj to z czym się pomęczysz na początku, to gwarantuję Ci, że zwróci się z naddatkiem.
z Niebieskim pozdRowieniem,
Doman89
------------------------ [ Dodano po: 15 minutach ]
Co do programu nie wiem, czy to jest zamierzony cel, ale w pętli while masz 2 ify.
Strzelam, że masz tam podłączone jakieś przełączniki i na podstawie pozycji chcesz przełączać tryb. Więc ja bym dał if... else if. Inaczej jeżeli będziesz miał 2 włączone to program będzie skakał od jednego do drugiego. No chyba, że tak ma być to ok.
Po drugie nie rozumiem czemu masz inicjalizację USART'u w programie. Przecież chyba wystarczy raz w main? Tym bardziej, że obie się niczym raczej nie różnią. Tak jak koledzy wyżej napisali, przydały by się przerwania. Wtedy ładujesz do bufora dane do wysłania oraz ewentualnie z bufora odczytujesz dane odebrane. Czyli jak wspomnieli full duplex.
Statystyki: Napisane przez doman89 — 29 paź 2014, o 23:31
]]>
