Delay in if else funktion

[Sebastian B. (opfer1983)]

Hallo ich möchte eine Schaltung bauen die jeden Tag bei nacht für 3min 
eine L-Diode einschaltet.Ich mache das mit einer Photodiode.Es klappt 
auch schon aber ich kriege die delay funktion nicht eingefügt sodas die 
leuchtdiode 3min anbleibt.Ich bin totaler Anfänger und wäre für hilfe 
sehr dankbar!

Hier ist mein code mit der delay funktion an der falschen stelle im Code

#include <avr/io.h>
#ifndef F_CPU
#define F_CPU 3686400UL
#endif
#include <util/delay.h>



void long_delay(uint16_t ms) {
    for(; ms>0; ms--) _delay_ms(1);
}

int main( void )
{
    DDRB = ( 1 << PB0 );        // Ausgang
   DDRD = ( 0 <<  PD2 );        //Eingang

    while( 1 ) {                // Endlosschleife
        if (!(PIND&0x04))

      PORTB = 0xFF;

    else
      PORTB = 0x00;
        long_delay(1000);       // Eine Sekunde warten...
    }

    return 0;
}

[Karl H. (kbuchegg)]

Du brauchst ein C Buch.

WEnn von einem if oder einem else mehrere Anweisungen abhängen, dann 
muss man die Anweisungen in { } einschliessen und so einen Block bauen.

Grundlagen C - zweite Unterrichtseinheit.

[Sebastian B. (opfer1983)]

Kannst Du mal an einen beispiel mir das zeigen?

[Karsten (Gast)]

Außerdem wird die LED die ganze Nacht leuchten...

[Sebastian B. (opfer1983)]

Stimmt habe ich vertauscht gerade gemerkt.

[Karl H. (kbuchegg)]

#ifndef F_CPU
#define F_CPU 3686400UL
#endif

#include <avr/io.h>
#include <util/delay.h>

void long_delay(uint16_t ms)
{
  for(; ms>0; ms--)
    _delay_ms(1);
}

int main( void )
{
  DDRB  =  ( 1 << PB0 );        // Ausgang
  DDRD &= ~( 1 << PD2 );        //Eingang

  while( 1 )
  {
    if( !( PIND & ( 1 << PD2 ) )
      PORTB |= ( 1 << PB0 );

    else
    {
      PORTB &= ~( 1 << PB0 );
      long_delay(1000);       // Eine Sekunde warten...
    }
  }

  return 0;
}

und rücke deinen Code korrekt ein! Falsch eingerückter Code verursacht 
Augenkrebs :-)

[Sebastian B. (opfer1983)]

Ja danke aber wenn ich das in der if funktion mache statt in der Else 
kommt die fehlermeldung : error: 'else' without a previous 'if'

[Karl H. (kbuchegg)]

Kauf dir ein C-Buch

Oder denk alternativ ein wenig nach. (*)

Wenn du das delay vom else Zweig in den if Zweig holst, dann hast du 
jetzt im if Zweig 2 Anweisungen, die vom if abhängen. Also wirst du dort 
eine { } Klammerung brauchen.


(*) Edit: Das nehme ich zurück. Du sollst nicht alternativ nachdenken. 
Du sollst auf jeden Fall nachdenken UND dir ein C-Buch kaufen.

  while( 1 )
  {
    if( !( PIND & ( 1 << PD2 ) )
    {
      PORTB |= ( 1 << PB0 );
      long_delay(1000);       // Eine Sekunde warten...
    }
    else
    {
      PORTB &= ~( 1 << PB0 );
    }
  }

[Fabi (Gast)]

if( !( PIND & ( 1 << PD2 ) )


DA GEHEN 3 KLAMMERN AUF, ABER NUR 2 ZU!!!

[Karl H. (kbuchegg)]

Danke.
Die Klammer ist beim editieren hier im Forum verschwunden.

Sollte aber klar sein, wo sie sein muss.

[Sebastian B. (opfer1983)]

Danke für die Hilfe ein bißchen nachdenken hat wirklich geholfen!!!


Hier meine Lösung

if (!(PIND&0x04))
{PORTB = 0xFF;
long_delay(1000);
}

[Karl H. (kbuchegg)]

Einrückungen!
eine konsistente und einigermassen schöne Formatierung verhindert schon 
mal 20% der gängigsten Fehler.

[Timmo H. (masterfx)]

Und nutze die Makros für die Ports:

if ( !(PIND & (1<<PD4)) ){
  PORTB = 0xFF;
  long_delay(1000);
}

[Sebastian B. (opfer1983)]

Ja ich werde es in Zukunft versuchen.

[Sebastian B. (opfer1983)]

Also so weit so gut.Aber wie funktioniert das jetzt das ich die Leuchte 
nur einmal anschalte.Weil jetzt bleibt sie ja noch zur zeit an?