As a block diagram for the idea in this question, that is a start.
It is, however, only a block diagram to show the concept.
As mentioned in the other question, you will (probably) need more than just a photodiode on your inputs.
I would suggest you setup a single photodiode on a single input and see how it responds before you go connecting a bunch of them.
Your H-Bridge drawing is very strange. There are usually four transistors driven by two control signals and just one motor. Your diagram seems to show four motors, each driven by a single control signal. A single line isn't enough to reverse the motor as you said you wanted to do in the other question.
Also remember that you will need resistors for the LEDs, and that putting LEDs in parallel as you have them is a bad idea.
Also note, you must use the correct Arduino. An Arduino Uno, for example, doesn't have enough IO pins. You should mark on your diagram which Arduino you are going to use.