A opravdu čekáš, že budou lidi zkoumat zdroják, ve kterých nepojmenováváš ani tlačítka? Kdo si myslíš, že bude luštit, co je button1, button2?

Proč tam máš milionkrát ten case? Udělej si procedury pro ty asi 3 případy, bude to mnohem přehlednější.
Pro kliknutí můžeš udělat jednu společnou udalost a přiřadit ji všem obrázkům. V property Sender máš obrázek, na který bylo kliknuto. Tento kód tam máš 36x. Proč?
I kdybys napsal jednu proceduru, které předáš hodnotu TAG a nechal si tam svých 36 událostí pro jednotlivé obrázky, tak by to bylo lepší, ze kterých bys ji volal.

Pak si jednoduše ošetříš to, že pokud byudeš mít otočeny 2 obrázky, tak metoda pro klik nebude reagovat, dokud se karty neotočí zpět.

Vytvořil jsi program se 3 000 řádky a přitom by ses vešel do desetinny. Chápu, že začínáš, ale u programování je potřeba přemýšlet, společné části nekopírovat v programu dokola, ale dát je do procedury a volat, kde potřebuješ.