No. Unless your circuit is fully isolated from users, this is dangerous.
Judging on the number of severe mains-safety errors in this circuit, I would advise you not to build mains-interfacing circuits until you are more experienced in electronics. Mains electricity is dangerous and carries the risk of electrocution and causing a fire.
The TRIAC, on the high-voltage side of your circuit, is connected to the ground of the low-voltage side of your circuit. It's also connected through U7 and R3 to what I assume is the Vcc of the LV-side of the circuit. This means the low-voltage side of the circuit may be at mains potential. This is dangerous if it is connected to other systems or accessible by users.
You need to drive the TRIAC using just the high-side part of the circuit. Consider using a dedicated isolated TRIAC driver instead of a normal optocoupler - which won't work properly on AC anyway. (The MOC302XM series comes to mind, but there are many other options of course)
- The PC817 is only rated for 80V collector-emitter voltage. It is completely unsuited for mains use as U7.
- R2 will pass 23mA RMS, and dissipate 5.3W RMS. It will burn quite impressively.
- Also, R2 and BR1 need to be rated for mains voltages. R2 in particular is at risk of arc-over if using a cheap, common carbon-film resistor, resulting a nice explosion of U5.
- You need a fuse on the mains side of the circuit.
- You might also want a snubber on the TRIAC. (Although you can probably get away without for low-medium power incandescents)