Problem mit Taster ab frage

[radbruch]

//
//    ATMega644
//    lcd-routines.c, lcd-routines.h
//
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <stdlib.h>
#include "lcd-routines.h"

uint8_t temp;

int main(void)
{
	DDRD = 0x00; // Pins sind Eingang
	PORTD = 0x0f; // mit aktiviertem PullUp, Taster schalten nach GND

	// Initialisierung des LCD
	// Nach der Initialisierung müssen auf dem LCD vorhandene schwarze Balken
	// verschwunden sein
	lcd_init();

	lcd_string("LCD Start !!!");
	_delay_ms(100); //Gibt einem die moeglichckeit zu lesen was auf dem LCD steht

	temp = PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0)) ; // Status der Tasten einlesen

	while(1)
	{
		if((PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0))) != temp) // Hat sich was geändert?
		{
			temp = PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0)); // Ja, deshalb neuen Status merken ...
			lcd_clear(); // ... und ausgeben
			lcd_home();
			switch(~temp & 0x0f) // Achtung! temp wird mit ~ invertiert!
			{
				case (1<<PD0): lcd_string("Das ist Taste 0"); break;
				case (1<<PD1): lcd_string("Das ist Taste 1"); break;
				case (1<<PD2): lcd_string("Das ist Taste 2"); break;
				case (1<<PD3): lcd_string("Das ist Taste 3"); break;
				default:lcd_string("Taste druecken");
			}
		}
	}
	return(0);
}

Ob das besser zu lesen ist? Deine Taster schalten doch nach GND, oder?

[Kampi]

Wenn die Taster nach GND schalten musst du eine Low Abfrage machen.
Die machst du (mein ich) so:

if(!(1<<PD0))
{
}

Das "!" steht dabei für eine Invertierung d.h. wenn PD0 = 1 NICHT erfüllt ist ist die If-Abfrage erfüllt.
Und wenn du deinen Code besser lesen willst versuch die Kommentare alle an die Seite zu schreiben und auf gleicher Höhe

[Ferdinand]

ok danke, es geht jetzt.

//
//    ATMega644
//    lcd-routines.c, lcd-routines.h
//
#include <avr/io.h>
#include <util/delay.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <stdlib.h>
#include "lcd-routines.h"

uint8_t temp;

int main(void)
{
    DDRD = 0x00; // Pins sind Eingang
    PORTD = 0x0f; // mit aktiviertem PullUp, Taster schalten nach GND

    // Initialisierung des LCD
    // Nach der Initialisierung müssen auf dem LCD vorhandene schwarze Balken
    // verschwunden sein
    lcd_init();

    lcd_string("LCD Start !!!");
    _delay_ms(2000); //Gibt einem die moeglichckeit zu lesen was auf dem LCD steht
    
    lcd_clear();
    lcd_home();
    lcd_string("Taste druecken");


    temp = PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0)) ; // Status der Tasten einlesen

    while(1)
    {
        if((PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0))) != temp) // Hat sich was geändert?
        {
            temp = PIND & ((1<<PD3) | (1<<PD2) | (1<<PD1) | (1<<PD0)); // Ja, deshalb neuen Status merken ...
            lcd_clear(); // ... und ausgeben
            lcd_home();
            switch(~temp & 0x0f) // Achtung! temp wird mit ~ invertiert!
            {
                case (1<<PD0): lcd_string("Das ist Taste 0"); break;
                case (1<<PD1): lcd_string("Das ist Taste 1"); break;
                case (1<<PD2): lcd_string("Das ist Taste 2"); break;
                case (1<<PD3): lcd_string("Das ist Taste 3"); break;
                default:lcd_string("Taste druecken");
            }
        }
    }
    return(0);
}

Nun möchte ich aus einem Taster einen Schalter Programmiren, ich habe keine Ahnung, wie ich das anstelle.
Ja meine Taster schalten nach GND, ich benutze das STK500.

[Kampi]

Wenn es ein einfacher Ein/Aus Schalter ist kannst du ihn wie einen Taster im Programm behandeln. Ein Taster schaltet ja auch Ein und wenn du ihn los lässt Aus. Ein Schalter schaltet halt Ein und beim zweiten betätigen Aus. Aber das macht für die Software keinen wesentlichen Unterschied. Nur du darfst den Zustand des Schalters nicht mit "while" Abfragen, weil sich der Controller dann die ganze Zeit in der Schleife aufhält solange du den Schalter nicht nochtmal betätigst.

[Ferdinand]

ok und wie macht Mann das, wen sich der Controller nur in Schleifen bewegt.
Ich müsste den Aktuellen wert zwischen Speichern, da die aktuelle Abfrage ständig überschrieben wird, nur weis ich nicht wie man so was wegen den Ganzen schleifen hin bekommt.

Danke

[Kampi]

Was möchtest du genau machen?
Beschreib mal das was du tun möchtest.

[Ferdinand]

Hi Kampi, ich find es wirklich toll das du mir bei meinem Problem hilfst.
Ich möchte irgendwann mal ein aus Wahl Menü schreiben, wo man Funktionen ein und aus schalten kann.

Das heißt also das ich erlernen möchte wie man sich ein Volum fassendes Menü mit untergruben und Auswahl von Funktionen schreibt, weil man so was gelegentlich brauchen könnte.

Danke