Schalter und LEDs an Mikrokontroller

Schönen Abend allerseits!

Als absoluter Elektronik-Laie muss ich heute eine, für viele banale Frage stellen:

An einen Port eines AVR-Mikrocontrollers sollen vier Neigungsschalter angeschlossen werden, gleichzeitig soll mit zugehörigen LEDs der Zustand der Schalter angezeigt werden. Habe ich gemäß diesem Schaltplan richtig kombiniert, oder ist mit Problemen zu rechnen?

Vielen Dank im Voraus für eure Kommtare

Die Ports sind ausreichchend hochohmig, sodass bei geöffnetem Schalter auch über die LED ein sicherer Highpegel abgegeben wird.
Grundsätzlich wird die Schaltung funktionern.
(Ich habe noch nicht bei Conrad nachgesehen, um was für einen Schalter es sich handelt.)
Manfred