Hrm...I know people hate this

but what am i doing wrong?
NPC Code:
if (created)
{
setshape 1,32,32;
}
if(actiontime)
{
triggeraction x,y,timer,#v(int(timevar2));
}
if(actionsell)
{
removeweapon #p(0);
setstring serverr.bid_#p(2)_item,#p(0);
setstring serverr.bid_#p(2)_acct,#p(3);
setstring serverr.bid_#p(2)_img,#p(4);
setstring serverr.bid_#p(2)_price,#p(1);
setstring serverr.bid_#p(2)_time,#v(int(timevar2));
}
if(actionreclaim2)
{
with(getplayer(#s(serverr.bid_#p(1)_acct)))
{
addweapon #s(serverr.bid_#p(0)_item);
}
setstring serverr.bid_#p(0)_item,0;
setstring serverr.bid_#p(0)_acct,0;
setstring serverr.bid_#p(0)_img,0;
setstring serverr.bid_#p(0)_price,0;
}
//#CLIENTSIDE
if(created)
{
timeout=1;
}
if(playerchats)
{
tokenize #c;
//selling
//item name, price, table number, acct, img
if(strequals(#t(0),/sell))
{
if(hasweapon(#t(1)) && !strcontains(#t(1),staff) && strtofloat(#t(2))<100001 && strtofloat(#s(serverr.bid_#t(3)_item))==0 && strtofloat(#t(3))<23 && !strcontains(#t(1),system))
{
triggeraction x,y,sell,#t(1),#t(2),#t(3),#a,#W;
}
else
{
say2 Illegal!;
}
}
}
if(timeout)
{
triggeraction x,y,time,time;
if(actiontimer)
{
for(this.i=0;this.i<20;this.i++)
{
if(strtofloat(#s(serverr.bid_#v(this.i)_time))+(10 )==#p(0))
{
//reclaim2 :this.i, seller account
triggeraction x,y,reclaim2,#v(this.i),#s(serverr.bid_#v(this.i)_ acct);
}
}
}
timeout=1;
}