Lauflicht mit LPC1769

[Dennis H. (denn1s)]

Hallo miteinander,

ich muss mit dem LPC1769 ein Lauflicht programmieren und wollte eure 
Meinung zum Code gerne wissen, ob es so Funktionieren könnte. Vielleicht 
sind einige Fehler vorhanden und über Verbesserungsvorschläge würde ich 
mich freuen.

Das Lauflicht soll folgendermaßen funktionieren mit Taster 3 soll Start 
bzw. Stopp des lautliches hervorgerufen werden, zu beginn sollen die 
LEDs nach links laufen. Mit Taster 2 soll sich das Lauflicht nach rechts 
bewegen und mit Taster 1 dementsprechend nach Links.

Datei befindet sich im Anhang, wenn die Header gewünscht ist kann ich 
sie auch hinzufügen.

Mit freundlichen Grüßen

/*
===============================================================================
 Name        : Lauflicht.c
 Author      : $(author)
 Version     :
 Copyright   : $(copyright)
 Description : main definition
===============================================================================
*/

#ifdef __USE_CMSIS
#include "LPC17xx.h"
#endif

#include <cr_section_macros.h>

#include <stdio.h>

#include "gpio.h"

void initial(void);
void LEDReset(void);
uint32_t taste1 (void);
uint32_t taste2 (void);
uint32_t taste3 (void);
void Linkslauf(void);
void Rechtslauf(void);


void initial(void)
{

    GPIOSetDir(1,18, OUTPUT);
    GPIOSetDir(1,19, OUTPUT);
    GPIOSetDir(1,20, OUTPUT);
    GPIOSetDir(1,21, OUTPUT);
    GPIOSetDir(1,22, OUTPUT);
    GPIOSetDir(1,23, OUTPUT);
    GPIOSetDir(1,24, OUTPUT);
    GPIOSetDir(1,25, OUTPUT);

    GPIOSetDir(2,3, INPUT);
    GPIOSetDir(2,4, INPUT);
    GPIOSetDir(2,5, INPUT);
    GPIOSetPull(2, 3, PULLUP);
    GPIOSetPull(2, 4, PULLUP);
    GPIOSetPull(2, 5, PULLUP);

}


void LEDReset(void)
{
  int i;

  for(i=18; i <= 25; i++)
  {
  GPIOSetValue(1, i, LOW);
  }
}

uint32_t taste1(void)
{
  return (GPIOGetValue(2, 3));  //Lesen Taster 3 Start/Stop
}

uint32_t taste2(void)
{
  return (GPIOGetValue(2, 4));  //Lesen Taster 3 Start/Stop
}

uint32_t taste3(void)
{
  return (GPIOGetValue(2, 5));  //Lesen Taster 3 Start/Stop
}

void Linkslauf(void)
{
  int i;

  while(1)
  {

    for(i = 18; i <= 25; i++)
    {
      GPIOSetValue(1, i, HIGH);
      _delay_ms(1000);
      GPIOSetValue(1, i, LOW);

      if(taste2() == 0)
      {
        for(i = 0; i<= 70000;i++){}    // Entprellen
        do{
        }while(taste2() == 0);
        for(i = 0; i<= 70000;i++){}

        Rechtslauf();
      }

      if(taste3() == 0)
      {
        for(i = 0; i<= 70000;i++){}    // Entprellen
        do{
        }while(taste3() == 0);
        for(i = 0; i<= 70000;i++){}

        break;
      }
    }
  }

}


void Rechtslauf(void)
{
  int i;

  while(1)
  {
    for(i= 25; i >= 18; i--)
    {
      GPIOSetValue(1, i, HIGH);
      _delay_ms(1000);
      GPIOSetValue(1, i, LOW);

      if(taste1() == 0)
      {
        for(i = 0; i<= 70000;i++){}    // Entprellen
        do{
        }while(taste1() == 0);
        for(i = 0; i<= 70000;i++){}

        Linkslauf();
      }

      if(taste3() == 0)
      {
        for(i = 0; i<= 70000;i++){}    // Entprellen
        do{
        }while(taste3() == 0);
        for(i = 0; i<= 70000;i++){}
        break;
      }
    }
  }
}

int main(void)
{

  initial();
  LEDReset();

  while(1)
  {

    GPIOSetValue(1, 18, HIGH);

    if(taste3() == 0)
    {
      Linkslauf();
    }
  }
}

[MaWin (Gast)]

Findest du den code nicht ein wenig aufwändig ?

Dann scheinst du es extern so verdrahtet zu haben, daß ein Taster einen 
Eingangspin nach Masse zieht (und ein Pull up ihn sonst nach plus). Das 
muss man im Programm nicht widerspiegeln, man kann dort durchaus taste() 
eine 1 als gedrückt zurückliefern lassen, das macht die Logik 
übersichtlicher.

Zum Entprellen gibt es fertige Methoden, da muss man das Rad nicht neu 
erfinden, und auch noch falsch.

int linksstoprechts=0,t1=1,t2=1,t3=1,last_t1=1,last_t2=1,last_t3=1,led=18;
int main(void)
{
    // initialisierung wie gehabt
    while(1)
    {
      for(i=0;i<100;i++)
      {
        t1=taste1();
        t2=taste2();
        t3=taste3();
        if(t1!=last_t1&&t1==0) linksstoprechts=1;
        if(t2!=last_t2&&t2==0) linksstoprechts=-1;
        if(t3!=kast_t3&&t3==0) linksstoprechts=(linksstoprechts==0);
        last_t1=t1;
        last_t2=t2;
        last_t3=t3;
        _delay_ms(10);
        if(linksstoprechts==0) break;
      }
      GPIOSetValue(1, led, HIGH);
      led+=linksstoprechts;
      if(led<18) led=25; else if(led>25) led=18;
      if(linksstoprechts) GPIOSetValue(1, i, LOW);
    }    
}