Mijn Pir schema

Beste,

Ik heb geen ervaring in elektronica (enkel de basis van het middelbaar).
Ik wil hierdoor jullie mening vragen van wat ik gebouwd heb.
Ik heb een bewegingsmelder gemaakt zie tekening (schets).

Waarom heb ik deze zo gemaakt:
• Moet klein zijn, moet achter een drukknop kunnen komen
• Het moest laagspanning zijn voor aan een SVV-kabel aan te sluiten
• De uitgang moet +/- 24V zijn voor mijn domotica aan te sturen
• De uitgang moest gescheiden zijn van de bewegingsmelder (daarom een optocoupler Ic)
• Het moest een pir zijn dat in één Bticino module gaat van 18mm past (deze van mij zijn 11mm)

Mijn onderdelen zijn:
• Spanningsregelaar (L7812) van 24Vcd naar 12Vcd te gaan
• Pir (12Vcd in en 3,3Vcd uitgang)
• Transistor (Bc547)
• Optocoupler (4N25)
• Weerstand 1MOhm
• Blauwe Led (3,5V / 20mA)

(nb: de weerstand en led zijn maar voor te testen worden achter af weggelaten)

Nu ik heb de bovenste schakeling gemaakt op een oefen bord resultaat het werkt, maar volgens een vriend die mij vertelde dat ik dat zo niet mag maken, er moeten een paar weerstanden tussen komen?
Wie kan me helpen?
Nb: Na 6 dagen werkt het wel nog steeds.

Op de onderste schakeling heb ik wat metingen gedaan en als lijk volgens mij is dit dik in orde, of ben ik mis?
Spanning =
1 = 23,89V (ingang spanning is een geschakelde voeding van Siemens 24VCD)
2 = 12,09V
3 = 1,866V
4 = 1,197V
5 = 22,34V
6 = 2,954V

Stromen =

7 = Uitgang laag = 4,83mA
Uitgang hoog = 39,70mA

8 = Uitgang laag = 0,01mA
Uitgang hoog = 28,00mA

9 = Uitgang laag = 0,01mA
Uitgang hoog = 0,06mA

10 = Uitgang laag = 0,01mA
Uitgang hoog = 15,28mA

11 = Uitgang laag = 0,01mA
Uitgang hoog = 15,36mA

12 = Uitgang laag = 0,01mA
Uitgang hoog = 0,06mA

13 = Uitgang laag = 0,01mA
Uitgang hoog = 14,86mA

14 = Uitgang laag = 0,01mA
Uitgang hoog = 20,83mA

Nb: Sorry voor de primitieve tekening

Met de vriendelijke groeten
Billy31