Pick up, throw and drop an interactable object.

Hey guys, out of boredoom I made a simple script which allows an object to be picked up, and then throw it or drop it down on the ground. Here's the link to GitHub.
https://github.com/Qkyyy/PickupablesObjects-UNITY
Feel free to use it accordingly to your need. I encourage you to expand it, because it's only a basis for general good. Let me know what do you think about it, cheers!

 

The seperation of concerned is really strange here, you let the object determine if it should be picked up, I would have put that logic on the player. Also if any object is within 2.2 it will be picked up so if 5 objects are within distance all 5 will be picked up.

Also it will Clip objects, you should use force instead to move it so it can't clip the world.

Just some quick things I saw when I looked at it quickly

 

Thanks for the answer!
Only one object will be picked up, because the code responsible for it is in OnMouseDown()

 

Ah it has some build in raycast logic that tests in center of screen?

 

From what I know it detects if raycast had hit the collider of the object.

Well, I did not want to put that code in PlayerController, not to clutter up the script. If someone will add later interactions with other objects or some enemy damage, I think it would also be in separate scripts.
But that's only my opinion, correct me if im wrong

 

You don't need to put it directly on the player. You can add a component that does it.

 

This.

I would do this because it allows you to have NPCs use the exact same code.

 

That makes sense. Thanks, I'll probably change it

 

That automagic for the pickup wont work though I guess

 

bump, I fixed some things and added amnesia-like objects rotation.
https://github.com/Qkyyy/PickupablesObjects-UNITY