Asuroprojekt mit Odometrie und LCD-Display

[Hermann Wessel]

hm ich glaube wir reden an einander vorbei
in dem programm gebe ich ja die variable speed zur funktion start und dort kann ich das ja mithilfe der taster rauf oder runter setzen
das problem ist lediglich das er den wert speed nicht wieder zur main gibt, sondern nur mit dem definierten wert 100 nimmt.
im internet habe ich gelesen das (int &speed) helfen sollte funktioniert aber nicht.
das ist garnicht das fertige programm doch das problem ist mir beim schreiben des programmes aufgefallen das das nich funktioniert.
bei bedarf kann ich hier das fertige programm poste

mfg
Hermann

[radbruch]

Upps, du hast recht, wir reden aneinander vorbei. Versuche es mal so:

void start()
{
....

start() sollte dann die Variable speed von main() übenehmen. Wenn's dann immer noch nicht funzt:

//Hauptprogramm

int speed = 100;

int main(void)
{
Init();
...

So ist speed auf jeden Fall global

Oder start() als Funktion mit Rückgabewert:

int start(int speed)
{
...
}
o--;
return(speed);
}

und Aufruf mit Zuweisung:

...
PrintIntLCD(speed);
speed = start(speed);
}

[Hermann Wessel]

hmm hab alles ausprobiert.. der arbeitet aber trotzdem mit 100

[Valen]

Sind deine änderungen in das .c datei gespeichert und dannach neu ubersetzt? Das speichern wird oft vergessen.

[radbruch]

Oje, langsam wird's zäh. Versuche mal das:

#include "stdlib.h"
#include "asuro.h"
#include "lcd.h"
#include "i2c.h"

int o = 1;

//Geschwindigkeit
int speed = 100;

void start(void)
{
    int keys;
    int i = 1;

    while(i)
    {
        keys = PollSwitchLCD();
        if(!keys)
        {
            Msleep(100);
        }
        else
        {
            switch(keys)
            {
               case LCD_KEY_YELLOW:
                  if(speed < 246) speed = speed +10;
                  break;
               case LCD_KEY_BLUE:
                   if(speed > 9) speed = speed -10;
                   break;
               case LCD_KEY_RED:
                  i = 0;
                  break;
            }
            ClearLCD();
            PrintSetLCD(0,0,"Geschwindigkeit");
            SetCursorLCD(0, 1);
            PrintIntLCD(speed);
        }
    }
    o--;
}

//Hauptprogramm

int main(void)
{

    Init();
    InitI2C();
    InitLCD();
    GREEN_LED_ON;

    while(o > 0)
    {
        PrintSetLCD(0,0,"Geschwindigkeit");
        SetCursorLCD(0, 1);
        PrintIntLCD(speed);
        start();
    }
    MotorDir(FWD,FWD);
    MotorSpeed(speed,speed);

    while(1); // Das Programm darf nie beendet werden!
    return 0;
} 

Nicht getestet, aber mit diesen Dummy-Files fehlerfrei übersetzt:

lcd.h:

#define LCD_KEY_YELLOW 1
#define LCD_KEY_BLUE 2
#define LCD_KEY_RED 4

void InitLCD(void);
int PollSwitchLCD(void);
void ClearLCD(void);
void SetCursorLCD(unsigned char colum, unsigned char line);
void PrintIntLCD(int i);
void PrintSetLCD(unsigned char colum, unsigned char line, char *data);

void InitI2C(void); 

lcd.c

#define LCD_KEY_YELLOW 1
#define LCD_KEY_BLUE 2
#define LCD_KEY_RED 4

void InitLCD(void){}
int PollSwitchLCD(void){return(0);}
void ClearLCD(void){}
void SetCursorLCD(unsigned char colum, unsigned char line){}
void PrintIntLCD(int i){}
void PrintSetLCD(unsigned char colum, unsigned char line, char *data){}

void InitI2C(void){} 

[Valen]

Aber dan wird es immer noch das selbe Zeichen auf dem LCD. (Geschwindigkeit 100) Weil am ende der erste Durchlauf von start wird die globale Variable o auf 0 gesetzt. (ist am Anfang 1) Das verhindert den zweite Durchlauf von den while-schleife in das main Programm. Und geht dann (nach der Motor Kommandos) in dem Endlos--schleife von Radbruch. Was genau bedeutet diese Variable "o". Was ist ihre Funktion in dein (Hermann Wessel) Logik? Wieso wird sie am ende von den start Funktion decrementiert?

[radbruch]

Das Display wird auch in start() nach jedem Tastendruck aktualisiert. Sein Programm ist wohl ein Ausschnitt eines größeren Programms, vielleicht dient o dort zur Steuerung des Ablaufs? In meiner Version wird speed einmalig editiert und dann schwirrt der asuro davon ;)

[Valen]

Radbruch stimmt. Ohne Roten Taster wird es in dem Funktion start stecken bleiben und dem LCD aktualisieren.

(Notiz an selb: NICHT 2 Quellcodes durcheinander liesen!!!!!!)