Szia!

Tényleg jó feladat. A mellékelt kód viszont nagyon nem egészen ezt csinálja.

Sorban a gondok:
1. LED nincs kimenetként deklarálva.
2. A nyomógombot érdemes aktív LOW-két használni: pinMode(kapcsolo, INPUT); aztán digitalWrite(kapcsolo, HIGH); Ez a belső pull-up ellenállást bekapcsolja, így a bemenet folyamatosan (kis áramon) magas jelszinten lesz. Ha lemegy 0-ra (ilyenkor a kapcsoló másik felét a földre kell kötni), akkor van gombnyomás. Üzembiztosabb és kevésbé zajérzékeny.
3. Változók. Ez így nem jó. Forrás olvasásra. Itt: data types. A lényeg, hogy bár most még annyira nagy galibát nem okoz, komplexebb programok simán elhasalhatnak rajta, hogy nem következetesen vannak használva a különböző változótípusok. Az int numerikus változó, ennek nincs TRUE és FALSE vagy HIGH és LOW értéke, bár az igaz, hogy az IDE vizsgálatoknál értékelheti úgy, mintha lenne, de a hozzárendelés ebben a formában gondokat okozhat.
4. 20 mp-es delay nagyon nem szerencsés. Mi van, ha ez idő alatt le akarod kapcsolni a lámpát? (Megoldás a blink without delay példában az Arduino tudástárban [lásd fent].)
5. Az első vizsgálatban látható 1 és 4 mp-es kivárások közül az 1 mp-esnek van értelme (bár te 2 mp-et írtál), a 4 mp-nek viszont semmi.
6. Semmi nem vizsgálja a második (bekapcsolt állapotban lévő lámpa kikapcsolására adott utasítást).

Kb. ennyi így első blikkre.