Arduino RFID RC522 Meldung senden wenn KEINE Karte vorhanden ist

[Alain Z. (alzen)]

Hallo Leute,

Ich habe aktuell ein Projekt mit dem RC522 Rfid Reader und suche dafür 
noch eine Lösung, wie ich den Arduino programmieren, dass er eine 
Meldung sendet wenn KEINE Chipkarte auf dem Lesegerät liegt.
Mein aktueller Code basiert ursprünglich auf den Programmieranleitungen, 
allerdings habe ich ihn um eine Speicher- und Vergleichsfunktion 
erweitert. Mittels Taster kann ich eine Karte einspeichern und je 
nachdem welche Karte danach gelesen wird leuchtet entweder eine grüne 
oder rote LED. Bei Interesse kann ich den Code auch posten.
Das Problem ist, dass ich einen Code brauch der eine Meldung sendet oder 
eine LED anschaltet so bald KEINE Karte auf dem RC522 liegt. Ich habe 
schon viel rumprobiert und stundenlang gegoogelt aber noch keine Lösung 
gefunden.
Es scheint, dass der Code ohne Karte in folgendem Bereich einfach hängen 
bleibt und ewig wartet bis eine Karte gelesen wird und es im Loop weiter 
geht.

...
if ( ! mfrc522.PICC_IsNewCardPresent())
{
  return;
}
if ( ! mfrc522.PICC_ReadCardSerial())
{
  return;
}
...

Gibt es eine Möglichkeit den Code so zu schreiben, dass beispielsweise 
nach 5 Sekunden erfolglosem Suchen der Suchbefehl abgebrochen wird und 
das Programm weiterläuft bis zum nächsten Loop?

Vielen Dank für eure Hilfe.

Alain

[Helmut H. (helmuth)]

Das mit dem Return haut nicht hin. Gelesen werden darf nur wenn eine 
neue Karte verfügbar ist.


void newcard(){
    if ( ! mfrc522.PICC_ReadCardSerial())
        Serial.print("ReadCard returned 0");
    }
    ...
}
void loop() {
    if (mfrc522.PICC_IsNewCardPresent()) {
        newcard();
    }
    if (Serial.available() > 0) {
    ...
    }
    if 5 Sekunden vorbei   {
    }
}

[Alain Z. (alzen)]

Hallo Helmut,

vielen Dank für deine Antwort. Leider verstehe ich noch nicht so richtig 
wo genau ich deinen Code in meinen einfügen muss.
Ich habe versucht die void newcard(){...} zwischen mein void setup() 
{...} und mein void loop() {...} einzusetzen und den void loop 
entsprechend umgeändert. Ergebnis war dass im serieller Monitor stets 
"ReadCard returned 0" geschrieben wurde, egal ob Karte vorhanden oder 
nicht. Wahrscheinlich habe ich als Anfänger etwas durcheinander 
gebracht.
Hier ist mein orginaler lauffähiger Code, vielleicht kanns du ja mal 
einen kurzen Blick draufwerfen wo ich deine Erweiterungen einbauen muss. 
Vielen Dank.

#include <SPI.h>
#include <MFRC522.h>
#define SS_PIN 10
#define RST_PIN 9
MFRC522 mfrc522(SS_PIN, RST_PIN);

int taster = 0;         // Set Tastervariable
long saveCode=0;        // Gespeicherter Code, wenn diese Karte gelesen wird, geht OK LED an
long aktCode = 0;       // Aktueller Code
long altCode = 0;       // vorheriger Code

void setup()
{
Serial.begin(9600);
SPI.begin();
mfrc522.PCD_Init();
pinMode (2, OUTPUT);       // grüne OK LED
pinMode (3, OUTPUT);       // rote NOK LED
pinMode (4, OUTPUT);       // blaue Set LED
pinMode (5, INPUT_PULLUP); // SetTaster
}
void loop()
{
  if (saveCode == 0)      // Wenn kein Code gespeichert ist, soll das angezeigt werden
 {
  digitalWrite(4,HIGH);   // Set LED an
  taster = 1;             // Tastervariable setzen, erster gelesener Code wird damit automatisch gespeichert 
 }
 if (digitalRead(5)==LOW) // SetTaster betätigt um neuen Code zu speichern, alter wird überschrieben
 {
  taster = 1;             // Tastervariable setzen
  digitalWrite(4,HIGH);   // SetLED anschalten
 } 
if ( ! mfrc522.PICC_IsNewCardPresent())
{
  return;
}
if ( ! mfrc522.PICC_ReadCardSerial())
{
  return;
}
  long code = 0;        // Variable für Code
  for (byte i = 0; i < mfrc522.uid.size; i++)
{
  code=((code+mfrc522.uid.uidByte[i])*10);
  aktCode = code;       
}
if (taster == 1)           // Wenn Tastervariable gesetzt wurde, soll der gelesene Code gespeichert werden
{
 saveCode = aktCode;       // gelesenen Code speichern
 digitalWrite(4,LOW);      // SetLED ausschalten
 taster = 0;               // Tastervariable zurücksetzen  
}
if (altCode == aktCode)   // Statusupdate Unterdrückung wenn kein Kartenwechsel 
{
}
else
{
  Serial.print("Der alte Code lautete:");

  Serial.println(altCode);
  
  altCode = aktCode;    // alten Code durch neuen ersetzen
  
  Serial.print("Die neue Kartennummer lautet:");

  Serial.println(aktCode);

  Serial.print("Der gespeicherte Code lautet:");

  Serial.println(saveCode);
}
if (aktCode==saveCode)     // Wenn der Code dem gespeicherten Code entspricht
{ 
digitalWrite (2, HIGH); // OK LED an
delay (1000);           // für 1 Sekunden
digitalWrite (2, LOW);  // OK LED aus
} 
else 
{ 
digitalWrite (3, HIGH); // NOK LED an
delay(1000);
digitalWrite(3, LOW);
}
}

[Helmut H. (helmuth)]

Hi

schön kommentiert, aber tu dir und dritten einen Gefallen und verwende 
sprechende Namen statt der Portnummern

digitalWrite (2, HIGH); // OK LED an
digitalWrite (LedOK, HIGH);

dann könnte man auf den Kommentar verzichten, das hilft auch ungemein, 
wenn man die LED plötzlich nicht an Port 2 sondern an 13 haben will.
NB: ist konstruiert, aber wem sollte man hier glauben, dem Kommentar 
oder dem Code?

digitalWrite (3, HIGH); // OK LED an

Teile das Programm auf, es gibt Sachen die man nur einmal macht wenn 
eine neue Karte erkannt wird, das wollte ich mit dem newcard andeuten.

Also: wenn eine Karte erkannt wird (beachte, dass hier kein 
Ausrufezeichen steht) dann ruft es die Routine auf.

  if (mfrc522.PICC_IsNewCardPresent()) {
        newcard();
  }

In der newcard() steht alles ab

  
 if ( ! mfrc522.PICC_ReadCardSerial()) {
    Serial.println ("Card lesen hat nicht funktioniert")
    return;      // <----hier ist das return richtig
 }
 usw ...

Dadurch wird das Programm übersichtlicher weil der Ablauf besser 
erkennbar ist.

[Helmut H. (helmuth)]

Hi Alain

habe mal meinen RFID Lese/Schreib Sketch rausgekramt und dabei Dein 
Problem erst jetzt verstanden.
"eine Meldung sendet oder eine LED anschaltet so bald KEINE Karte auf 
dem RC522 liegt"
Mein aktueller Stand: Keine Ahnung wie das gehen soll, weil eben newcard 
nur einmal getriggert wird.
Mein Ansatz wäre: Jede Sekunde lesen, wenn das fehlschlägt ist keine 
Karte mehr da.
Ich versuchs mal heute abend.

PS: der Sketch verwendet LCD über I2C und einen Beeper an SOUND_PIN, 
müsste aber auch ohne gehen.

[Alain Z. (alzen)]

Hallo Helmut,

vielen Dank für deine ausführliche Informationen.

Das Ziel von dem Projekt ist gewissermaßen eine Positionsüberwachung von 
einem Behälter und so lange sich der richtige Chip vor dem Lesegerät 
befindet, bedeutet das, dass sich der richtige Behälter an der richtigen 
Stelle befindet. So bald der Behälter entfernt wird verliert das 
Lesegerät den Kontakt und das soll dann angezeigt werden. Mittels 
zusätzlichem Taster oder Reedkontakt könnte man das wahrscheinlich auch 
lösen, nur sehe ich nicht wirklich ein weshalb ein zusätzliches Bauteil 
unbedingt notwendig ist.

Der Code müsste dafür so sein, dass der Kartensuchbefehl zeitlich 
limiert ist. Findet der RC522 etwas in der Zeit läuft es weiter wie 
bisher, wenn nicht gibt er die Meldung weiter und der Loop startet 
wieder von vorne. Alternativ, gibt es eine Möglichkeit im Code die 
Durchlaufzeit von einem Loop zu messen und bei überschreiten von einem 
Wert diesen Loop abzubrechen und wieder neu zustarten?

[Helmut H. (helmuth)]

Laut MFRC522.cpp:

PICC_IsNewCardPresent()  calls PICC_RequestA. Only "new" cards in state 
IDLE are invited. Sleeping cards in state HALT are ignored.

PICC_RequestA()          Invites PICCs in state IDLE to go to READY

PICC_WakeupA()           Invites PICCs in state IDLE and HALT to go to 
READY

PICC_HaltA()             Instructs a PICC in state ACTIVE(*) to go to 
state HALT.

PCD_StopCrypto1()        Remember to call this function after 
communicating with an authenticated PICC - otherwise no new 
communications can start.


* Man kann eine neue PICC nur lesen wenn man das StopCrypto aufgerufen 
hat.
* wenn man StopCrypto aufruft wird sofort von IsNewCardPresent eine 
eventuell noch vorhandene PICC erkannt

Ist also ganz einfach, jede Sekunde (oder so, siehe nexttime+1000) wird 
ein disconnect(PICC_HaltA und PCD_StopCrypto1) ausgeführt und vermutet, 
dass kein PICC mehr da ist.
Wenn nach zwei Sekunden nicht das newcard getriggert wurde, dann wird 
die Vermutung zur Gewissheit.
Damit sich das nur bei Bedarf so verhält kann man es auch abschalten:
int state=2;
/*                     to
 0  Disabled           ->n by serial '0' to '3'
 1  PICC present       ->2 by disconnect
 2  unknown            ->1 by newcardPresent ->3 by 2nd read
 3  no PICC present    ->1 by newcardPresent
 */

Jetzt muss nur noch der State abgefragt werden, beim Testprogramm gibt 
es das auf LCD aus.

Nachtrag:
Müsste also für dich passen, würde mal mit dem nexttime experimentieren 
wie weit man damit runtergehen kann. Bei mir hat es mit 100 ms auch 
funktioniert, dabei einen bug entdeckt: state case 2 sollte kein 
disconn() machen.
Bei >2cm Abstand wird sicher ein No card erkannt.

[Alain Z. (alzen)]

Hallo,

vielen Dank für die Hilfe.
Inzwischen ist mir noch eine Idee gekommen wie das ganze noch viel 
einfacher gelöst werden kann, ohne dass der RFID Leser zurück gesetzt 
werden muss.
Ich nutze nur die millis() Funktion und mache mir den Umstand zu nutze, 
dass der Chipsuchprozess erst beendet wird wenn eine Karte gelesen 
wurde.
Findet er eine Karte, d.h. code>0 wird dieser Zeitpunkt mit 
letzteZeitEinChip = millis() festgehalten.
Am Loop Anfang vergleiche ich einfach die aktuelle Zeit mit diesem 
gespeicherten Zeitpunkt, wird ein bestimmtes Intervall überschritten 
bedeutet das, dass sich kein Chip vor dem Lesegerät befindet.

MfG