[Jewbacca2010]

Zur Aufgabe:
über taste P5.0 soll modus 1 gewählt werden mit taste P6.0 soll modus 2 gewählt werden.
inhalt von modus 1: über die matrixtastatur soll mit der taste1 die 16 leds (P1 und P4) links rotieren taste 2 nach rechts rotieren taste 3 von außen nach innen der signalwechsel der leds soll im 1/4 sekundentakt sein (timer)
modus 2:
werte zwischen 0 und 50 gradsollen ausgelesen werden und an so nem drehknopf erhöht oder erniedrigt werden wenn die temperatur unter 25 grad hat soll an der 7segmentanzeige tief stehen bei über 25 grad soll hoch angezeigt werden:

Mein Lösungsansatz habe ich grade geschrieben:
Was haltet ihr davon bitte um Verbesserungsvorschläge und Hilfe bei den auskommentierten leeren Feldern danke

#include <reg517a.h>
#include <stdio.h>
unsigned int wert;
sbit p_5=P5^0;


void main(void)
{
if(p_5==0)
{
P6=0xBF;
if(P8==0xFE)
{
//P1 und P4 Links Rotieren lassen

}

}
P6=0x0F;
if(P8==0xFE)
{
//P1 und P4 rechts Rotieren lassen

}
P6=0xEF;
if(P8==0xFE)
{
// P1 und P4 außen nach innen der signalwechsel der leds soll im 1/4 sekundentakt sein (timer)

}
if((P6&0x01)==0)
{
ADCON0=0x01;
BD=1;
TMOD=0x02;
while(1)
{
ADDATL=0;
TR1=1;
wert=ADDATH;
if(wert >=0x00 && wert <=0x19)
{
// TIEF an 7 Segment
}
if(wert >=0x19 && wert <=0x31)
{
//HOCH an 7 Segment

}
}

}

[flotti]

so wie es aussieht willt du nen microcontroller programmieren... welchen? ^^'

verbresserungsvorschläge:

1) benutze "" oder poste deine quelltexte über pastebin.com, dadurch werden sie besser lesbar -

2) schreib kommentare welche variablen/constanten aus dem header kommen und welche von dir sind - und was sie machen

3)
statt if wäre eine switch-case struktur übersichtlicher

funktioniert dein programm soweit eigentlich?