Need Help With This Random Giver Script!

[DarkEmperor13]

Hey guys I found this script on the wiki:

default
{
    touch_start(integer num_detected)
    {
    //  the key of the avatar touching
        key id = llDetectedKey(0);
 
        integer randomIndex = llGetInventoryNumber(INVENTORY_OBJECT);
    //  random >> [0.0, max)
        randomIndex = (integer)llFrand(randomIndex);
 
        string itemName = llGetInventoryName(INVENTORY_OBJECT, randomIndex);
 
        llGiveInventory(id, itemName);
    }
}

Ok so it gives you a random item from the prim's(that this script is in) inventory. Now my question is how can you make the script remove the item once it is given?

[Chaser Zaks]

llRemoveInventory is what you are looking for:

default
{
    touch_start(integer num_detected)
    {
    //  the key of the avatar touching
        key id = llDetectedKey(0);
 
        integer randomIndex = llGetInventoryNumber(INVENTORY_OBJECT);
    //  random >> [0.0, max)
        randomIndex = (integer)llFrand(randomIndex);
 
        string itemName = llGetInventoryName(INVENTORY_OBJECT, randomIndex);
 
        llGiveInventory(id, itemName);

        llRemoveInventory(itemName); //Remove the item from inventory
    }
}

KEEP IN MIND: This permanently removes the item from the object's inventory! It does not go to your trash, or a trash folder in the object. The item will be gone for good!

[DarkEmperor13]

ok thanks!

 

[DarkEmperor13]

just in case, is there a way to make it go to ur trash?

[Ruthven Ravenhurst]

13 minutes ago, DarkEmperor13 said:

just in case, is there a way to make it go to ur trash?

No, but you could give it to yourself at the same time

default
{
    touch_start(integer num_detected)
    {
    //  the key of the avatar touching
        key id = llDetectedKey(0);
 
        integer randomIndex = llGetInventoryNumber(INVENTORY_OBJECT);
    //  random >> [0.0, max)
        randomIndex = (integer)llFrand(randomIndex);
 
        string itemName = llGetInventoryName(INVENTORY_OBJECT, randomIndex);
 
        llGiveInventory(id, itemName);
	llGiveInventory(llGetOwner(),itemName);

        llRemoveInventory(itemName); //Remove the item from inventory
    }
}

 

[DarkEmperor13]

ok well for some reason it isn't working. the objects inside the prim are still there and I'm not receiving anything

Edited: Nvm I figured out why. I had to use INVENTORY_NOTECARD cause my prim has notecards and not objects

Edited February 6, 2018 by DarkEmperor13

[Rolig Loon]

Scripting is 90% about logic, and sadly, it's difficult to learn by copying down someone else's logic.  The best way to address a problem like this is to stand back and take a deep breath. then:

1. Think the problem through carefully, one step at a time.

2. If it's a complicated problem, solve it in small pieces.

3. Test everything, using plenty of llOwnerSay messages to yourself so that you can see exactly what the script is doing.

And of course, my own favorite rule:

4. Trust only your own work. If you found a free script somewhere, that's fine, but assume that you can do better if you just start fresh. Other people make mistakes, or at least different assumptions, so fall into your own potholes -- not theirs. You can get great ideas from other people's scripts, but the best ideas for solving your problem are the ones you figure out yourself.

[DarkEmperor13]

I know this isn't relevant to this post or whatever u call u, but I need to know how I can get this code to remove object from inventory.

default

    
{
    state_entry() {
        llSetText("",<1,1,1>,1);
    }
    touch_start(integer param)
    {
        llGiveInventory(llDetectedKey(0),"Box");
    }
    
}

this is for giving object with name given in script to anyone who touches the prim

[Rolig Loon]

Use llRemoveInventory, as directed earlier.

[DarkEmperor13]

Are you sure? I tried that and it gave me an error

 

Edited February 6, 2018 by DarkEmperor13

[Love Zhaoying]

Looked like the same basic code to me..just more direct not random: give thing named “this”. Hey, didja know that if you make the item “no copy” (carefully) then you won’t have to remove it?

[DarkEmperor13]

I've tried llremoveinventory(itemName), llremoveinventory(llDetectedKey(0),"Box") and even llremoveinventory("Box")

 

edited in response to Love Zhaoying: oh really? How do u do it?

Edited February 6, 2018 by DarkEmperor13

[DarkEmperor13]

oh wait nvm I forgot, ur talking about objects but what I am using as content are notecards which cant be set to uncopiable. So is there a way I can use the random item giver code and just remove the "random" parts? if so how?

 

[Love Zhaoying]

4 minutes ago, DarkEmperor13 said:

I've tried llremoveinventory(itemName), llremoveinventory(llDetectedKey(0),"Box") and even llremoveinventory("Box")

 

edited in response to Love Zhaoying: oh really? How do u do it?

I best let someone else answer how to make something “no copy”, I want to go get a “good” answer!

Some if your attempts at llRemoveInventory should not have compiled. See http://wiki.secondlife.com/wiki/LlRemoveInventory

[Rolig Loon]

6 minutes ago, Love Zhaoying said:

I best let someone else answer how to make something “no copy”,

Um....   Like this:

UNclick the box marked Next Owner: COPY.  You can do that on any asset that you own (Object, Script, notecard, animation, sound ....).

[Love Zhaoying]

9 minutes ago, Rolig Loon said:

Um....   Like this:

UNclick the box marked Next Owner: COPY.  You can do that on any asset that you own (Object, Script, notecard, animation, sound ....).

Ok, wasn’t sure if rezzed copy or in-inventory.

[Rolig Loon]

3 minutes ago, Love Zhaoying said:

Ok, wasn’t sure if rezzed copy or in-inventory.

Either way. Right click, select Properties, and then set your next owner perms.  You can never make something MORE permissible for the next guy, but you can always make it LESS permissible.  The only catch at all is that Next Owner perms only become permanent on an Object when it is rezzed, you you don't want to set perms on objects while they are in your inventory.  Set them on rezzed copies, which you then take to inventory.

Edited February 6, 2018 by Rolig Loon

[Love Zhaoying]

18 minutes ago, Rolig Loon said:

Either way. Right click, select Properties, and then set your next owner perms.  You can never make something MORE permissible for the next guy, but you can always make it LESS permissible.  The only catch at all is that Next Owner perms only become permanent on an Object when it is rezzed, you you don't want to set perms on objects while they are in your inventory.  Set them on rezzed copies, which you then take to inventory.

And then, the giver script will give the no-copy version, thusly removing from the object’s Inventory?

[Rolig Loon]

That's the way it's supposed to work, yup. 

[Xiija]

mebbe capitalization?

llremoveinventory("Box") to ... llRemoveIinventory("Box");