object rezzer with next and back

Make it "if( x == 0 ) x = obj - 1;". If there are 5 items on the list, the index of the last one is 4, not 5.

//Object Display Table with next and back buttons.
//Written by Ruthven Willenov.
//May be used freely however you want
//Drop this Script into the root prim, along with the objects you would like to display.

integer x = 0;//tells the script to start at the first item in the list, "0" is the first item
default
{
link_message( integer sender, integer num, string message, key id )//listen for message from other links
{
if( num == 0 )//what do to if the "next" button is pressed
{
llWhisper(5,"die"; //tells the currently displayed object to go away, change the channel it whispers on for each table, so they don't kill objects on other tables
llSleep(0.25);//gives it time to go away
integer obj = llGetInventoryNumber(INVENTORY_OBJECT);//checks how many objects are in the list
llRezObject(llGetInventoryName(INVENTORY_OBJECT, x), llGetPos()+<0,0,1>*llGetLocalRot(),<0,0,0>,llGetLocalRot(),0);//rezzes the object 1 meter above the table
x++;//tells it to rezz the next one in the list
if (x == obj) x = 0;//if you were at the last one in the list, start over at the beginning
}

if( num == 1 )//what do to if the "back" button is pressed
{
llWhisper(5,"die";//again, tells the current object to go away,change the channel it whispers on for each table, so they don't kill objects on other tables
llSleep(0.25);//again, gives it time to go away
integer obj = llGetInventoryNumber(INVENTORY_OBJECT);//again, checks how many are in the list
llRezObject(llGetInventoryName(INVENTORY_OBJECT, x), llGetPos()+<0,0,1>*llGetLocalRot(),<0,0,0>,llGetLocalRot(),0);//again, rezzes the object 1 meter abaove the table
x--;//tells it to choose the previous item in the list
if (x < 0) x = obj-1;//if you were at the beginning, start over at the end
}
}
}

//Object Display Table next button
//Written by Ruthven Willenov.
//May be used freely however you want
//Drop this Script into the "next" button prim.
default
{
touch_start(integer total_number)
{
llMessageLinked( LINK_ROOT, 0, "", NULL_KEY ) ; //sends a blank link message to the root with the number "0", to represent, this is the next button
}
}

//Object Display Table back button
//Written by Ruthven Willenov.
//May be used freely however you want
//Drop this Script into the "back" button prim.
default
{
touch_start(integer total_number)
{
llMessageLinked( LINK_ROOT, 1, "", NULL_KEY ) ; //sends a blank link message to the root with the number "1", to represent, this is the back button
}
}

//Die Script, drop this into any prim of each object that is displayed so that it will go away before the next/previous item is displayed. or to clear it without showing another object simply say "die" on the correct channel ie "/5die"

default
{
state_entry()
{
llListen(5,"","","";//change the channel to match the table the object is in
}

listen(integer chan, string name, key id, string msg)
{
if(msg=="die"//if the message it hears is "die"
{
llDie();//kills the object containing this script
}
}
}

You might want to use the integer modulus (%) operator. It makes things look a little simpler.

// Increment (circularly)
x = (x+1)%obj;

// Decrement (circularly); +obj is for the case where x-1 < 0
x = (x-1+obj)%obj;

oh ok, i got it now, but oddly it still does that weird thing, and it's not only at the end of the list like i thought. click next rezzes item 1, click back, rezzes item 2, back again, item 1, back again, last item, and vice vera when going in the opposite direction. tried moving it to the beginning of each if statement and it gives me a run time and math error. i figured it would be better to call on the next/previous item right before it rezzes because it looks like what the previous script is doing is, rezzing the selected item, then figuring out which one is next/previous, and making that the one that's rezzed next no matter which button is pressed

[QUOTE[
if( num == 0 )
{
x = (x+1)%obj;
llWhisper(5,"die";
llSleep(0.25);
integer obj = llGetInventoryNumber(INVENTORY_OBJECT);
llRezObject(llGetInventoryName(INVENTORY_OBJECT, x), llGetPos()+<0,0,1>*llGetLocalRot(),<0,0,0>,llGetLocalRot(),0);

Your last change is doing that, but it helps to define 'obj' before actually using it

if( num == 0 )
{
integer obj = llGetInventoryNumber(INVENTORY_OBJECT);
x = (x+1)%obj;
llWhisper(5,"die";
llSleep(0.25);
llRezObject(llGetInventoryName(INVENTORY_OBJECT, x), llGetPos()+<0,0,1>*llGetLocalRot(),<0,0,0>,llGetLocalRot(),0);

Hmm. Are you certain there are NO other link messages being received. Maybe put a llOwnerSay() at the beginning of your 'link_message' handler for debugging. 'num' will ALWAYS either be zero or non-zero, so if ANY script sends ANY link message to your prim for ANY reason, your code will be incrementing or decrementing. Make sure you get as many messages as you expect.

//Object Display Table
//Written By Ruthven Willenov
//To Setup
//First you need to drop a seperate script inside each item you want to display that listens on the specified channel for the message "die".
//Second to set up the table, make the table, and make 2 buttons, name one "next" and the other "back" (without the quotes)
// link the buttons to your table and position them appropriately, drop this script into the table
//setup the channel, and offset correctly below
//drop in the items, and you're all set

integer obj = 0;//the first item in the list is always 0 not 1
integer chan = 5;//what channel to whisper on, make sure it matches the channel in the items displayed
vector offset = <0.0,0.25,0.010>;//where to rezz the object relative to the the rezzer
default
{
touch_start(integer total_number)
{
string button = llGetLinkName(llDetectedLinkNumber(0));//detects the name of the button that is pressed
if (button == "next"//if the next button was pressed
{
integer quant = llGetInventoryNumber(INVENTORY_OBJECT);//how many objects in my inventory
obj = (obj+1)%quant;//call the next object in the list
llWhisper(chan,"die";//tells the displayed object to go away
llSleep(0.25);//gives the current object time to go away
llRezObject(llGetInventoryName(INVENTORY_OBJECT, obj), llGetPos()+offset*llGetLocalRot(),<0,0,0>,llGetRot(),0);//rezz the object
}
else if (button == "back"//if the back button is pressed
{
integer quant = llGetInventoryNumber(INVENTORY_OBJECT);//how many objects in my inventory
obj = (obj-1+quant)%quant;//call the previous object in the list
llWhisper(chan,"die";//tells the displayed object to go away
llSleep(0.25);//gives the current object time to go away
llRezObject(llGetInventoryName(INVENTORY_OBJECT, obj), llGetPos()+offset*llGetLocalRot(),<0,0,0>,llGetRot(),0);//rezz the object
}
}
}

//On Rez Change Rotations & Die
// Put this script in any object that needs to change its rotations when rezzed on the rezzer table.
//This script will delete it self if the object it is in changes ownership.
// This will allow you to sell copies of the object directly off of the rezzer display item.
// This script will change the rezz rotations of the object it is in. It is often necessary if the base prim of the object does not have zero rotations.
//ALSO you must edit the <0,0,0> X,Y,Z numbers to say where you want the object to rezz in relation to the center of the rezzer table.

key owner;
rotation rot_xyzq;
default
{
state_entry()
{
owner = llGetOwner();
vector xyz_angles = <0,1.0,0>; // This is to define a 1 degRee change(use the rotation #'s you see in the edit window)
vector angles_in_radians = xyz_angles*DEG_TO_RAD; // Change to Radians
rot_xyzq = llEuler2Rot(angles_in_radians); // Change to a Rotation
}
on_rez(integer rez_num)
{
if(owner != llGetOwner())
llRemoveInventory(llGetScriptName());
else
llSetRot(llGetRot()*rot_xyzq); //Do the Rotation...
}
}

//Die Script, drop this into any prim of each object that is displayed so that it will go away before the next/previous item is displayed. or to clear it without showing another object simply say "die" on the correct channel ie "/5die"
default
{
state_entry()
{
llListen(5,"","","";//change the channel to match the table the object is going to be rezzed from
}
listen(integer chan, string name, key id, string msg)
{
if(msg=="die"//if the message it hears is "die"
{
llDie();//kills the object containing this script
}
}
}