As Ratch mentioned, you have a problem with the outputs -- they don't go anywhere. The outputs of the previous JKs have to go to the JK inputs of the next one in line. It's simple enough. Without that connection, you can't expect to shift the data. What makes things complicated, whether parallel loading, synchronous clearing or especially changing the thing from right shift to left shift, is that fact that you end up using four times more external gates than you do flip flops to get the job done. All that is why you use MSI shift registers. It is a good exercise in design to "roll your own" just to see how it's done and understand the logic, even if it is impractical to use in a circuit. Same goes with a synchrohous counter circuit.