You need to use symbols for components, not outline drawings (a 3 pin package could be anything), and organise the layout sensibly - ground at the bottom, supply rail and the top, and where possible current flow vertically through components. Component types and values should be clear and what is "?" supposed to be?
A circuit diagram or schematic follows these rules in order to be readable - its totally different from a physical layout.