Minecraft Forums

Thanks to the efforts of KsyMC and MrARM, inventory editing methods for editing inventories of chests and players were added to BlockLauncher.

However, I don't really know how to name them.

The names used by KsyMC and MrARM are:

getItemChest(par1int, par2int, par3int, par4int);
getItemCountChest(par1int, par2int, par3int, par4int);
getItemDataChest(par1int, par2int, par3int, par4int);

Player.getSlotInventory(par1int);
Player.getSlotInventoryCount(par1int);
Player.getSlotInventoryData(par1int);
Player.clearSlotInventory(par1int);

Which, as you can see, are a bit inconsistent (Slot vs Item, for example).

Can someone suggest better names for these methods?

Quote from Thecactigod

First of all I don't really like the prefixs (prefixes?) Like player.whatever. but that's just my opinion so whatever. As for the names, how about getInventorySlot(whatever) for player slots and getChestSlot(whatever) for chests?

I like the prefixes (I call them namespaces; Treebl calls them OOP-Styled), as it tells the script writer what type of method one is calling, so I'll be moving all new methods into that style.

Quote from Thecactigod

OK whatever you say. Your the king anyway.

Treebl's the actual king. If he wants to rename all methods in ModPE in the Swahili language, I would have no choice but to follow.

Quote from InusualZ

xD

if (kupataBidhaaKatikaMkono() == 46) {
    kulipuka(x, y, z, 4.0)
}

Quote from michaeldarker172

I get the chest functions but not the inventory functions. What do those do?

Quote from Creepig01

I agree although I do not have an Android device and BlockLauncher is not avalible to me, I believe the prefixes are good.

The Player inventory functions allows script users to see what you are carrying in your inventory. Currently, scripts can only see what you have in your hand.

I'm not mainly concerned with the prefixes: I am worried about the names themselves: i.e. getChestItem? getChestSlot? GetChestSlotItemId?

Quote from Connor4898

I think this:

Level.getChestSlot
Level.getChestCountSlot
Level.getChestDataSlot
Level.setChestSlot
Player.getInvSlot
Player.getInvSlotCount
Player.getIntSlotData
Player.setInvSlot
Player.clearInvItem

Why clearInvItem instead of Slot?