pickup items please help

well im not going to do a full tutorial but

1 you tag the item that you can pick up as "pickupableitem"

1. in the player object put two variables for each weapon/item

2 in your script raycast to see if you hit an object with the tag "pickupableitem"

3 in your playerobject out an array with all the weaponprefabs

4 in the pickupableitem you put a script with the index of the array which corresponds with the item

5 then you Instantiate the object from the array destroy the one in the map

6 Instantiate the weapon variable from the array

7 parent it to the player

8 delete whta ever object you were holding and Instantiate it with a rigid body and add force

Hope this helps

If some thing dosnt make sense ill explain it as well as i can

 using UnityEngine;
 using System.Collections;
 
 public class weaponchange : MonoBehaviour {
     public Transform[] weaponlist;
     public Transform spawnpoint;
     private GameObject Pickup
     private Float num; 
 
     void Update(){
     Ray ray = maincamera.camera.ScreenPointToRay(new Vector3(Screen.width/2,Screen.height/2,0));
     
         RaycastHit hit;
     
         if(Physics.Raycast(ray, out hit,pickuprange)) {
            
         if (hit.collider.gameObject.tag == "pickable"){
       Pickup = hit.collider.gameobject
       num = hit.collider.gameObject.getComponent<weaponnum>().num;
     
         }else{ 
 
         Pickup = null;
        }
         if(Pickup != null){
 
       Transform pickupob =  insatainte(weaponnum[num],spawnpoint.position,spawnpoint.rotation) as Transform;
        pickupob.parent = transform;
       Destroy(Pickup):
        }
 
 }
 }
 
 sorry its in c# but idk javascript well

just put a script on the radio with a public variable named num and put the radioprefab into the array and make sure the num variable = the array index

From your 3rd picture, I can see that the radio collider is too small - the character will never touch it! Modify its size (Box Collider/Size) so that the collider becomes a cube about 1x1x1 units (or more), and the character can touch it from any side.
The script attached to the character must be something like this (similar to what @fafase wrote):

var hasRadio = false; // tells whether the player has radio or not

function OnTriggerEnter(other: Collider){ if (other.tag == "pickup"){ hasRadio = true; // now player has got a radio! Destroy(other.gameObject); // destroy the radio object } }

EDITED: Well, since you prefer to use a distance method, I modified your script to only check distances to the pickable objects when you press Q - this will not impact your frame rate, because the distance checking will only be done in the frame where Q was pressed. I also changed ObjectsToPick to a builtin array instead of a List: it's faster, and you will not need a List anyway - removing the picked items inside the for..in loop could screw up the loop control. Instead of removing a picked item, this script just ignores the inactive ones (the items are deactivated when picked up). It also ignores items behind the player, and finishes the loop when an item is picked up to avoid picking other close items, if any:

import System.Collections.Generic;

var Inventory : List. = new List.< GameObject>(); var pickUpDistance : float = 5f; var pickSound : AudioClip; // drag the sound here

private var ObjectsToPick : GameObject[];

function Start () { // get the pickable objects in a builtin array: ObjectsToPick = GameObject.FindGameObjectsWithTag("PickUps"); }

function Update () { if (Input.GetKeyDown("q")){ // only check distances when q pressed var here: Vector3 = transform.position; // get the character position in a variable for (var Item : GameObject in ObjectsToPick) { // for each pickable item... if (Item.active){ // that's still active... // get the vector player->item: var dir: Vector3 = Item.transform.position - here; // if item close enough and not behind the player... if (dir.magnitude < pickUpDistance && Vector3.Dot(dir, transform.forward) > 0) { // play the sound... AudioSource.PlayClipAtPoint(pickSound, Item.transform.position); // deactivate the object... Item.SetActiveRecursively(false); // add it to the inventory... Inventory.Add(Item); // and finish the loop: break; } } } } }

I have a scripting tutorial on this, I wrote the code in C# in the video but you can download the code in either C# or JAVASCRIPT if you want. The video goes over item pick ups like the code the others suggested. Here's the link http://youtu.be/Pc2uIIlzYes