[SOLVED] Call Screen, Imagebutton SetVariable - Not Working?

[4listair]

I'm trying to make a sort of tarot card reading thing where you can select different cards and then the card will "flip" over to show you what you got, but I can't seem to get it working.

In screens.rpy:

Code: Select all

screen reading():

    default tarot = 1

    add "images/bg_tarot.png"

    imagebutton:
        auto "images/card_small.png"
        xalign 0.315
        yalign 0.2
        action SetVariable("tarot", "1")
    imagebutton:
        auto "images/card_small.png"
        xalign 0.685
        yalign 0.2
        action SetVariable("tarot", "2")
    imagebutton:
        auto "images/card_small.png"
        xalign 0.5
        yalign 0.7
        action SetVariable("tarot", "3")

In script.rpy:

Code: Select all

    $ tarot = renpy.call_screen("reading")
    if tarot == 1:
        show card1:
            xalign 0.315
            yalign 0.2
        with dissolve

    if tarot == 2:
        show card2:
            xalign 0.685
            yalign 0.2
         with dissolve

    if tarot == 3:
        show card3:
            xalign 0.5
            yalign 0.7
        with dissolve

[Kia]

what part is your problem? randomizing, animation or the click action?

[Alex]

Note

Code: Select all

action SetVariable("tarot", "1")

will set 'tarot' to text value '1', so the next check will fail

Code: Select all

if tarot == 1

Try

Code: Select all

action SetVariable("tarot", 1)

Note 2
When you 'call' a screen it must return a value. So instead of

Code: Select all

action SetVariable("tarot", 1)

try

Code: Select all

action Return(1)

https://www.renpy.org/doc/html/screen_a ... tml#Return

[dGameBoy101b]

Rather than calling the screen directly through python, you could use the renpy "call screen" function and then use the "_return" variable to read the returned value. Also you should use the "elif" conditional statement for the second and third tests to absolutely ensure that only one card is chosen at a time (even though, logically, no more than one of the conditions should be meet in a single instant).

Code: Select all

init:
    screen reading():
        add "images/bg_tarot.png"
        imagebutton:
            auto "images/card_small.png"
            xalign 0.315
            yalign 0.2
            action Return(1)
        imagebutton:
            auto "images/card_small.png"
            xalign 0.685
            yalign 0.2
            action Return(2)
        imagebutton:
            auto "images/card_small.png"
            xalign 0.5
            yalign 0.7
            action Return(3)
label start:
    call screen reading
    $ tarot = _return
    if tarot == 1:
        show card1:
            xalign 0.315
            yalign 0.2
        with dissolve
    elif tarot == 2:
        show card2:
            xalign 0.685
            yalign 0.2
         with dissolve
    elif tarot == 3:
        show card3:
            xalign 0.5
            yalign 0.7
        with dissolve

[4listair]

Thank you, everyone! The return script was what I needed. :")