Question

Does this look like a good circuit diagram?

Answer 1

It's hard to tell what's going on down there in the corner with the resistor and the ammeter. There seems to be as many as 3 or 4 wires in and out of the ammeter, which would be wrong. A real ammeter only has two ... one in and one out. (Same for a resistor.)

It's hard to say whether this circuit works, until we can clearly understand how everything is hooked up in that corner of the drawing.