Dbscroll Packing Command

[arneljan]

Can Some One Tell Me How To Make Command To Pack 10 DB or Meteor To DBScroll Or MetoerScroll

Can Some One Give Me Sample Of Command..

[Asphy×ia]

Wherever you've coded your item usages:

case xxxx:
{
if (player.inventory.contains(dbID, 10)
{
removeItem(DbId, 10);
addItem(DbscrollId, 1);
}
else
Text("You don't have enough to create a DBscroll.");

You don't have to use the else, but that is a simple version of doing it - not specifying which source you are using, it's plainly simple

same for met scroll.

[arneljan]

Ty For sharing this.. i use source 5530

Is This The Right Command To Pack

case "DBScroll":
{
if (client.Inventory.Contains(1088000, 10))
{
client.Inventory.Add(720027, 0, 1);//DBScroll
}
if (client.Inventory.Contains(1088000, 10))
{
client.Inventory.Remove(1088000, 10);
}
break;
}
case "MetScroll":
{
if (client.Inventory.Contains(1088001, 10))
{
client.Inventory.Add(720027, 0, 1);//Meteor Scroll
}
if (client.Inventory.Contains(1088001, 10))
{
client.Inventory.Remove(1088001, 10);
}
break;
}

[Asphy×ia]

Originally Posted by arneljan

Ty For sharing this.. i use source 5530

Is This The Right Command To Pack

case "DBScroll":
{
if (client.Inventory.Contains(1088000, 10))
{
client.Inventory.Add(720027, 0, 1);//DBScroll
}
if (client.Inventory.Contains(1088000, 10))
{
client.Inventory.Remove(1088000, 10);
}
break;
}
case "MetScroll":
{
if (client.Inventory.Contains(1088001, 10))
{
client.Inventory.Add(720027, 0, 1);//Meteor Scroll
}
if (client.Inventory.Contains(1088001, 10))
{
client.Inventory.Remove(1088001, 10);
}
break;
}

Sure but you're accumulating useless checks.

case "DBScroll":
{
if (client.Inventory.Contains(1088000, 10))
{
client.Inventory.Remove(1088000, 10);
client.Inventory.Add(720027, 0, 1);//DBScroll
}
break;
}

You were checking if the inventory contained 10 dbs twice, which isn't needed. Just throw the main check, if it's good: remove the 10 dbs and replace with a DBScroll. Same for mets.

[Spirited]

Wait, don't tell me that **** actually switches based on string comparisons. That can't be real source code. I feel ill.

[arneljan]

Ty Its Working Now Thanks For The Help

[Spirited]

NO. That was actual source code?! WHYY?? At best it's hashing strings, at worst it's a sequence of string compares! Why are you doing this?! Please don't do that.

[pro4never]

Originally Posted by Spirited

NO. That was actual source code?! WHYY?? At best it's hashing strings, at worst it's a sequence of string compares! Why are you doing this?! Please don't do that.

Halloween came early this year

[InsomniacPro]

Originally Posted by Spirited

Wait, don't tell me that **** actually switches based on string comparisons. That can't be real source code. I feel ill.

I already have a headache.

[Soulfly25]

Suggestion. Why u make a command for the Dbscroll?. why not make it directly right click the meteor in the inventory and wallah the 10meteor has become 1 meteorscroll!

Bravo!

[Asphy×ia]

Originally Posted by Soulfly25

Suggestion. Why u make a command for the Dbscroll?. why not make it directly right click the meteor in the inventory and wallah the 10meteor has become 1 meteorscroll!

Bravo!

He's not making a command, he's trying to do what you just stated