Since you found by yourself that you were adding a list containing the object, and not directly the object, I'll focus on the code you use to display the inventory :
Python:
screen inventory_screen:
[...]
default startPos = 0
$ y = 0
$ x = 515
# Only works with the [number of entries by page] entries starting at /startPos/.
for item in sorted( inventory.items )[startPos:startPos+9]:
if x >= 1055: # 515 + ( 270 * 2 )
$ y += 235
$ x = 515
else:
$ x += 270
$ pic = item.image
imagebutton idle pic hover pic xpos x+300 ypos y+110 action [Hide("gui_tooltip"), SetVariable("item", item), selectItem(item)] hovered [ Play ("sound", "sfx/click.wav") ] unhovered [Hide("gui_tooltip")] at inv_eff
if item.selected:
add "gui/selected.png" xpos x+200 ypos y+70 anchor(.5,.5)
# What are you trying to do ?
# "( variable, variable )" mean that you are creating a tuple.
# But creating a tuple of only one value should be wrote
# "( variable, )", else Ren'py will directly assign the value
# without putting it in a tuple.
# Is it that you wanted to build a list of items to sell ?
# Anyway, what's the value of "item_use", it's define nowhere
# in the screen, is it comming from /selectItem/ ? If yes, why
# not doing this directly inside it ?
$ selitem = (item_use)
showif startPos > 0:
# Starting pos, actual first entry - number of entries by page.
textbutton _("Prev Page"):
# You don't need to show a screen already shown. Screens are automatically
# updated by Ren'py.
action SetScreenVariable('startPos', startPos - 9)
# float ? You really want to place it as % of the actual size ?
xpos .375 ypos .83 # arbitrary pos for x
showif startPos < len( inventory.items ) + 9:
# Starting pos, actual first entry + number of entries by page.
textbutton _("Next Page")
action SetScreenVariable('startPos', startPos + 9)
xpos .475 ypos .83
You can even get ride of the computations if you decide to place the items in a
You must be registered to see the links
. And solve the problem of selected items with a
You must be registered to see the links
displayable.
Python:
screen inventory_screen:
[...]
default startPos = 0
# 3 columns of 3 rows.
grid 3 3:
# Only works with the [number of entries by page] entries starting at /startPos/.
for item in sorted( inventory.items )[startPos:startPos+9]:
$ pic = item.image
imagebutton:
idle pic
hover pic
# Arbitrary example. Build the in the fly "selected image".
# You can also create the /LiveComposite/ object directly
# in the item object and have something like /item.selectedImage/.
selected_idle LiveComposite( ( 154, 200 ),
( 0, 0 ), Frame( pic, size=( 154, 200 ) ),
( 0, 0 ), store.Frame( "gui/selected.png", size=( 154, 200 ) )
# Why not changing /selitem/ directly here ?
# Each time a new object will be selected, the value will change.
# And your code weren't reverting its value to a null one if nothing
# was selected, so it will change nothing.
# Still don't know what /item_use/ have as value.
action [Hide("gui_tooltip"), SetVariable("item", item), selectItem(item), SetVariable( "selitem", item_use)]
hovered [ Play ("sound", "sfx/click.wav") ]
# Really not sure of this one, what are you trying to achieve ?
unhovered [Hide("gui_tooltip")] at inv_eff
# Alternate code if what you wanted was to build a list of
# items to sell.
# action [Hide("gui_tooltip"),
# SetVariable("item", item),
# selectItem(item),
# If the item is on the list of item to sell, a click mean
# "remove it". Else a click mean "add it"
# If( item_use in selitem, RemoveFromSet( "selitem", item_use ), AddToSet( "selitem", item_use ) )]
# If the item is in the list of items to sell, then it's selected.
# selected item in selitem
showif startPos > 0:
# Starting pos, actual first entry - number of entries by page
textbutton _("Prev Page"):
# You don't need to show a screen already shown. Screens are automatically
# updated by Ren'py
action SetScreenVariable('startPos', startPos - 9)
# float ? You really want to place it as % of the actual size ?
xpos .375 ypos .83 # arbitrary pos for x
showif startPos < len( inventory.items ) + 9:
# Starting pos, actual first entry + number of entries by page
textbutton _("Next Page")
action SetScreenVariable('startPos', startPos + 9)
xpos .475 ypos .83
You must be registered to see the links
and
You must be registered to see the links
, if you're interested in the alternate code.
If the list was of text, you could do something like this :
Python:
# Iterate all the entries of /inventory/, and for each one, if it starts with
# the value passed as parameter, we add it to the list that we will return.
define inventoryHas = lambda x: [ i for i in inventory if i.startswith( x ) ]
label whatever:
# If an entry match, then remove it from the list before...
if inventoryHas( "sportdress" ):
# Note that /inventoryHas/ return a list. And the list can
# only have one entry since the purpose is to prevent duplicated
# entries. So, what you want to remove is the first entry of the
# returned list.
$ inventory.remove( inventoryHas( "sportdress" )[0] )
# Add the new entry to the list.
$ inventory.add( sportdress_1 )
But the list is a list of objects... Therefore you have to change a little your item object. Since I don't know your object, I'll present you the
You must be registered to see the links
meta method, and make it return the name of the item, instead of the usual
<class "whatever" object at 0x...>.
Here's a small example of use :
Code:
init python:
class Item( object ):
def __init__( self, what ):
self.what = what
def __repr__( self ):
return self.what
default inventory = [ Item( "sportdress_1" ), Item( "something else" ), Item( "nothing" ) ]
# Note: Now we works with the text representation of the object, not its direct value.
define inventoryHas = lambda x: [ i for i in inventory if str( i ).startswith( x ) ]
label start:
if not inventoryHas( "sportdress" ):
"do not have dress"
else:
"Already have dress"
if not inventoryHas( "sportsuit" ):
"do not have suit"
else:
"Already have suit"
Note that, by using this, you can have more generic substitutions. You can by example decide that there can be only one dress in the inventory, whatever the kind of dress :
Python:
init python:
class Item( object ):
def __init__( self, what, kind ):
self.what = what
self.kind = kind
# No need of __repr__ now that we have an attribute to directly address.
default inventory = [ Item( "sportdress_1", "dress" ), Item( "something else", "crap" ), Item( "nothing", "void" ) ]
# Note: We have an attribute to rely on, we test it directly.
# And like it's a dedicated value, we can test it's entire value.
define inventoryHas = lambda x: [ i for i in inventory if i.kind == x ]
label whatever:
# Did it have a dress ?
if inventoryHas( "dress" ):
# Yes, a sport one, so we remove it before...
$ inventory.remove( inventoryHas( "dress" )[0] )
# Add the floral dress to the list.
$ inventory.add( floraldress_2 )
