2 Eingänge abfragen

[Achim S. (achims)]

Hallo
seit 2 h Stunden lese ich mein C Buch und versteh es nicht. 
Wahrscheinlich nur ein Denkfehler.
Es sollen 2 Eingänge abgefragt werden. Nur wenn beide eingänge entweder 
H oder L haben soll geschaltet werden. Bisher habe ich dazu:

if (!(PINB & (1<<PINB6) && (1<<PINB7) ))
    return 1;
    else
    return 0;

Kann mir jemand auf die Sprünge helfen?
achim

[Peter X. (peter_x)]

Nicht einzeln, sondern maskiert abfragen?

[MWS (Gast)]

Ein AND & ist kein EXOR ^.

[Achim S. (achims)]

ja

[Cian Xang Re (Gast)]

MWS schrieb:
> Ein AND & ist kein EXOR ^.

Warum nicht?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

PINB & (1<<PINB6) && (1<<PINB7)

Sowas sollte man nur dann machen, wenn man ganz ausgezeichnet die 
Prioritäten der Operatoren & und && weiß, und somit wenigstens grob 
abschätzen kann, was dabei herauskommt...

Achim Seeger schrieb:
> Es sollen 2 Eingänge abgefragt werden. Nur wenn beide eingänge entweder
> H oder L haben soll geschaltet werden.
Dann würde ich dir Vorschlagen, erst einmal jeden Einzelnen der beiden 
Eingänge einzulesen. Und anschließend in einem getrennten 
Rechenschritt auszuwerten. Also etwa so:

 if (PINB & (1<<PINB6)) e1 = 1;
 else                   e1 = 0;

 if (PINB & (1<<PINB7)) e2 = 1;
 else                   e2 = 0;

 if (e1==e2) Schalten();

Das kapiert jeder...

[Achim S. (achims)]

Hallo Lothar
wollte es in einer Zeile hinbekommen. Möchte damit einen Codierschaltzer 
auswerten. Der hat bis zu 4 Ausgänge. 1,2,4,8 zu 10 pol.
Dadurch muss ich bis zu 4 Pins gleichzeitig abfragen.
Dachte dabei an

if ( PINB5 & PINB 6 & PINB7 & PIN ? )(Vereinfacht)
achim

[Achim S. (achims)]

Sorry vergesen bzw nicht richtig geschrieben. Entweder H oder L nicht 
beide

[MWS (Gast)]

Cian Xang Re schrieb:
> MWS schrieb:
>> Ein AND & ist kein EXOR ^.
>
> Warum nicht?

Warum ist Grün nicht Rot?
Und doof nicht intelligent?
Fragen über Fragen...

[MWS (Gast)]

Achim Seeger schrieb:
> Es sollen 2 Eingänge abgefragt werden. Nur wenn beide eingänge entweder
> H oder L haben soll geschaltet werden.

Achim Seeger schrieb:
> Sorry vergesen bzw nicht richtig geschrieben. Entweder H oder L
> nicht beide

Deine Problembeschreibung ist wirr.
Möglicherweise beschreibst Du besser, was es am Ende machen soll.

[Achim S. (achims)]

Ganz einfach
Habe Taster 1 und Taster 2 am PB6 und PB7 angeschlossen. Liegen über 
einen Widerstand nach Vcc. Wenn ich beide taster betätige (nach Masse) 
soll if schalten. Dabei ist es nicht wichtig ob sie entprellt oder 
gleichzeitig gedrückt werden

[Grummel (Gast)]

Achim Seeger schrieb:
> Ganz einfach
> Habe Taster 1 und Taster 2 am PB6 und PB7 angeschlossen. Liegen über
> einen Widerstand nach Vcc. Wenn ich beide taster betätige (nach Masse)
> soll if schalten. Dabei ist es nicht wichtig ob sie entprellt oder
> gleichzeitig gedrückt werden

Eine UND Verknüpfung mit NICHT ODER?

[c.m. (Gast)]

hm…

return (PINB6 ^ PINB7)

richtig?

[MWS (Gast)]

c.m. schrieb:
> hm…
> return (PINB6 ^ PINB7)

Jetzt wird's langsam albern.

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Achim Seeger schrieb:
> wollte es in einer Zeile hinbekommen
Tja, dann wirst du wohl ein wenig mehr Nachdenken müssen. Probier mal 
aus, ob du folgen kannst:
Dein Schalter wird in geschickter so an dem Port angeschlossen, dass die 
Bitwertigkeiten aufsteigend sind. Also z.B. 1-PB4, 2-PB5, 4-PB6 und 
8-PB7. Dann liest du den Port ein, markierst das obere Nibble aus und 
schiebst den Wert um 4 Bits nach rechts. Fertig.

Wenn du beim Schaltplan nicht aufgepasst hast, musst du die Bits eben 
überall her zusammenklauben.

Und vergiss einfach diese "alles-in-einer-Zeile" Programmiererei. Das 
ergibt nur obfuscated Code, den nicht mal du selbst nach 3 Monaten noch 
kapierst.

[achim (Gast)]

Hallo Lohtar
Die Hardware gibt es noch nicht. Dadurch kann ich alles noch machen.
Da bleibt  mir wohl nur die zweite Version. Ein paar gute Sachen stehen 
drin. Werde mal alles testen. Danke für deine Hilfe
achim

[MWS (Gast)]

!((PINB & (1<<PINB6)) || (PINB & (1<<PINB7)))

[Ingo (Gast)]

Ginge nicht auch das hier?

if ( ((PINB & PINB6)>>PINB6) == ((PINB & PINB7)>>PINB7) )
     return 1;
return 0;

Ingo

[asterix (Gast)]

Ingo schrieb:
> Ginge nicht auch das hier?if ( ((PINB & PINB6)>>PINB6) == ((PINB &
> PINB7)>>PINB7) )
>      return 1;
> return 0;
>
> Ingo

das wäre eine elegane Lösung ;-)

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Wenn man jetzt schon mit unnötig kryptischen C-Code um sich schmeißen 
muss, dann schlage ich das vor:

return !!(PINB & PINB6) == !!(PINB & PINB7);

Aber Achtung: bei all diesen "Einzeilern" ist ausser an der 
Übersichtlichtkeit an nichts gespart. Der erzeugte und ausgeführte 
Code ist gleich umständlich oder noch umständlicher...

[Grummel (Gast)]

Steht denn schon eigentlich unzweideutig fest, wann jetzt bei if 
geschaltet werden soll? Eine Wahrheitstabelle wäre doch jetzt hier mal 
angebracht.

[achim (Gast)]

Wenn beide Taster gedrückt werden oder sogar 3 Taster

[Ingo (Gast)]

Grummel schrieb:
> Steht denn schon eigentlich unzweideutig fest, wann jetzt bei if
> geschaltet werden soll?

Achim Seeger schrieb:
> Nur wenn beide Eingänge entweder H oder L haben soll geschaltet werden

[MWS (Gast)]

Lothar Miller schrieb:
> return !!(PINB & PINB6) == !!(PINB & PINB7);

Na, Lothar, so ganz sicher nicht. Fehlt da nicht etwas? ;-)

[Marek L. (mareklew)]

achim schrieb:
> Wenn beide Taster gedrückt werden oder sogar 3 Taster

#define TasterPort PINB

//Taster 1 an PORTB Pin 1, nach Masse schaltend
#define Taster1 1

//Taster 2 an PORTB Pin 2, nach Masse schaltend
#define Taster2 2

//Taster 1 an PORTB Pin 4, nach Masse schaltend
#define Taster3 4


// ist "True" wenn alle Schalter gleichzeitig gedrückt worden sind:
#define SCHALTEN (!(TasterPort & ((1<<Taster1) | (1<<Taster2) | (Taster3))))


...

if SCHALTEN 
 schere_an();

darum gings?

[Karl H. (kbuchegg)]

Ingo schrieb:
> Grummel schrieb:
>> Steht denn schon eigentlich unzweideutig fest, wann jetzt bei if
>> geschaltet werden soll?
>
> Achim Seeger schrieb:
>> Nur wenn beide Eingänge entweder H oder L haben soll geschaltet werden


Was Ingo meint:

Ja, was denn jetzt?
Wenn beide Taster gedrückt wurden, oder wenn beide Eingänge denselben 
Zustand haben.
Das ist nicht dasselbe. Denn die beiden Eingänge haben auch dann 
denselben Zustand, wenn die beiden Taster nicht gedrückt sind.

Ehe man etwas programmiert sollte einem die zu erzielende Logik klar 
sein. Denn sonst kommt man ganz unweigerlich mit den && bzw. || 
durcheinander, mit denen man Teilausdrücke miteinander verknüpft.

Ganz abgesehen davon sollte man sich darüber im klaren sein, dass so 
etwas wie '2 Taster gleichzeitig gedrückt' immer eine etwas knifflige 
Sache ist. Denn für deinen µC sind Zeiten kleiner als Millisekunden 
überhaupt kein Problem. Für dich als Mensch ist es aber praktisch 
unmöglich, 2 Taster in exakt derselben Hunderstel-Sekunde zu drücken. 
D.h. für deinen µC wird es immer so aussehen, dass zuerst die eine Taste 
gedrückt wird und erst dann die andere Taste. Wenn also bereits ein 
einfacher Tastendruck einer der beiden Tasten jeweils eine Aktion 
auslöst, dann wirst du praktisch immer in dieser Einzelaktion landen, 
weil du es nicht schaffst, die beiden Tasten so zu drücken, dass der µC 
sie als tatsächlich gleichzeitig niedergedrückt erkennt.
Ausser natürlich du spickst dein Programm mit diversen Delays. Aber auch 
dann ist es mehr zufällig, ob das klappt oder nicht. Ganz abgesehen 
davon, dass man die Delays nicht haben will.

[Achim S. (achims)]

Habe eine relativ einfache Methode gefunden. Alles in einer Zeile drin.

//if (!(PINB & 0b11000000 ))
    if (!(PINB & 0xc0 ))
    
    return 1;
    else
    return 0;

Es müüsen beide Tasten gedrückt sein damit return 1 ist. Drücke ich nur 
eine Taste, hat es keinen Einfluss.
Die beiden ersten Zeilen sind gleich, nur anders geschrieben. Damit kann 
ich ohne Problem auch andere Eingänge einbinden.
achim

[Marek L. (mareklew)]

Achim Seeger schrieb:
> Habe eine relativ einfache Methode gefunden. Alles in einer Zeile

Nun sollst Du meinen letzten Post genau lesen und über den Unterschied 
zwischen den beiden Schreibweisen gleichen Ausdrucks nachdenken ;-)
Gruß,
Marek

[Michael S. (rbs_phoenix)]

Wenn beide Taster immer gleichzeitig gedrückt werden müssen (z.B. um 
eine Presse runter zu lassen) kann man auch einfach beide Taster in 
Reihe schalten. Quasi eine Hardware-Und-Verschaltung ;) Die Hardware 
steht ja noch nicht.

Geht natürlich nicht, wenn auch beim Drücken einer Taste was passieren 
soll.

[Mike M. (mikeii)]

Achim Seeger schrieb:
> Habe eine relativ einfache Methode gefunden. Alles in einer Zeile drin.
>

> //if (!(PINB & 0b11000000 ))
>     if (!(PINB & 0xc0 ))
> 
>     return 1;
>     else
>     return 0;
>

> Es müüsen beide Tasten gedrückt sein damit return 1 ist. Drücke ich nur
> eine Taste, hat es keinen Einfluss.
> Die beiden ersten Zeilen sind gleich, nur anders geschrieben. Damit kann
> ich ohne Problem auch andere Eingänge einbinden.
> achim

Damit man aber noch lesen kann, welcher Pin gemeint ist:

if (  !(PINB &   ((1<<PINB6)|(1<<PINB7))   )  )
    return 1;
    else
    return 0;

((1<<PINB6)|(1<<PINB7)) ergibt dein Bitmaske 0b11000000

if(<Aussage>)
  return 1;
else
  return 0;

kannst du immer zu return <Aussage> vereinfachen.

Also hier:

return !(PINB & ((1<<PINB6)|(1<<PINB7)) )

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Achim Seeger schrieb:
> Es müüsen beide Tasten gedrückt sein damit return 1 ist.
Das ist aber ganz was Anderes als das, was dort gefordert war:
Achim Seeger schrieb:
> Nur wenn beide eingänge entweder H oder L haben soll geschaltet werden.
Also muss geschaltet werden, wenn beide H sind und wenn beide L 
sind...

Du solltest bei der Problembeschreibung ein wenig weg kommen vom 
Umgangssprachlichen und definierte Angaben machen.

MWS schrieb:
> Lothar Miller schrieb:
>> return !!(PINB & PINB6) == !!(PINB & PINB7);
> Na, Lothar, so ganz sicher nicht. Fehlt da nicht etwas? ;-)
Uuups, ich korrigiere und schreibe dazu noch alles ohne Leerzeichen:

  // Wenn beide gleich: 1 zurückgeben. Wenn ungleich: 0
  return !!(PINB&(1<<PINB6))==!!(PINB&(1<<PINB7));

[Paul Baumann (Gast)]

Lothar Miller schrieb:
> Also muss geschaltet werden, wenn beide H sind und wenn beide L
> sind...

Das wird schwer, denn entweder hat eine Eingang H oder L -aber beides
zur gleichen Zeit?

Achim schrieb:
> Nur wenn beide eingänge entweder H oder L haben soll geschaltet werden.


Das ist ein XNOR.

MfG Paul

[Achim S. (achims)]

Danke für eure Hilfe. Hatte es schon gesagt, mache es nur wenn beide H 
haben nicht L, ist leichter und Eindeutig
achim