Game in Pythonista App

Here are some general tips and guidelines:

• Do read the PEP8 guide – Loads of good information in that one

• Use docstring, not comments for functions – Docstrings will possibly show up in your IDE when you hover or read documentation on functions, so move your function comments to docstrings like this:

  def turn_on(sender):
      """Switch button to 'on' state."""
      ...
  

• Allow vertical space between functions – Between functions it is recommended to add two new lines, which helps separate the functions, and allows for using single new lines within functions to group code statements there.

• Be consistent in naming – You vary your function naming style from button_tapped() to game_Description(). I would strongly suggest to be consistent in naming (and other style aspect as well), and the most pythonic way for functions is snake_case.

• Switch to new style string formatting – Instead of the % operator, you are better of using 'Score: {}'.format(x). The former is depreceated, and might be removed in the future.

• Don't intermix top level code and functions – You have a section of top level code before the play_game() (aka play_Game()), which is kind of hidden. A really good advice is to not have any top level code besides the following at the bottom of the file:

  if __name__ == '__main__':
      main()
  

  That is group most of the top level code in a main() function, and gather the view initialisation in another function. (Not quite sure how this works in Pythonista, but alternatively, you'll have a single call to main() at the top level.)

• Don't use global variables – If possible, it is always better to pass values to functions, instead of using them globally. When restructuring into a view initialisation function, this introduces possibly a new concept of returning multiple values from a function (see code below)
• Feature: Your code always increases score, tapped or not – Your current code always gives you a increase in score, independent on whether you actually tapped anything or not. If however you didn't tap anything, you'll get a false test on the check_lights() test. This is kind of hiding the game logic. I'd move the test around a little, to increase understanding of game logic.

• Indentation errors in presented code – I'm surprised no-one commented on this yet, but the code as presented doesn't work, there is a faulty indentation on the x = random.randint(0, 3) line, and likewise on the speed -= .07 line. Both lines should be indented. A trick to remember for next post here on Code Review is to mark the entire code block (when editing your question/answer, and hit Ctrl+K. (Do not edit the code now, though, as you've gotten some answers)

• Reorganise buttons into a list – The other answers has touched upon this already, but to take it one step further, you could use a list holding all of your buttons, and this would simplify much of your logic.

• Include the 0 before floats – It is not neccessary, and is a minor detail, but I recommend keeping the 0 in front of floats. That is 0.07, instead of .07. (I kept misreading it as 0.7, and didn't quite see the use of increasing the speed twice before stopping the speed increase... )

Implementing all of these changes (and removing the increase score function, as it kind of hid a main concept of the game) we arrive at the following code:

import random
import ui
from time import sleep
import console
import sys


def turn_on(button):
    """Turn button to "on" state."""
    button.title = 'on'
    button.tint_color = ('yellow')


def turn_off(button):
    """Reverts button to off state."""
    button.title = 'off'
    button.tint_color = 'light blue'


def blink(button):
    """Briefly turns switches button state "on" and then "off"."""
    turn_on(button)
    sleep(.5)
    turn_off(button)


def button_tapped(button):
    """When the button is tapped, it turns the button "off"."""
    turn_off(button)


def game_description():
    """Describe game objective in alert box."""
    console.alert("Game objective:", 
       "Click the button to turn the   light off before the next light turns on", 
       "Ready")


def game_over_alert():
    """Alert box stating game over, and asking for a new game."""

    return console.alert('Game Over!','','Play Again?')


def check_lights(buttons):
    """Check if all buttons are turned off."""
    return all(button.title == 'off' for button in buttons)


def all_off(buttons):
    """Turn off all buttons."""
    for button in buttons:
        turn_off(button)


def add_button(view, width_percentage, height_percentage,
               button_action=None, title='off'):
    """Adds a button positioned by percentages, with given title."""

    button = ui.Button(title)
    button.center = (view.width * width_percentage, view.height * height_percentage) 
    button.flex = 'LRTB'
    button.action = button_action
    view.add_subview(button)
    return button


def initialise_view():
    """Initialise view, and return view and buttons."""

    view = ui.View()

    # Default settings for view

    view = ui.View()
    view.name = 'Light Panel'
    view.background_color = 'black'

    # Create "mole" buttons
    mole_buttons = [add_button(view, 0.2 * (i + 1), 0.5,
                               button_action=button_tapped) for i in range(4)]

    score_button = add_button(view, 0.5, 0.75, 'Score: 0')

    return view, mole_buttons, score_button


def play_game(view, mole_buttons, score_button):
    """Main game loop. Handles game, and restarting of game."""

    score_count = 0
    speed = 2.55
    game_description()
    random.seed() 

    while True:
        turn_on(mole_buttons[random.randint(0, 3)])

        # Allow player to wack a mole        
        sleep(speed)

        # Check if they hit the mole, that is check all lights are off
        if check_lights(mole_buttons):
            score_count += 1
            score_button.title = 'Score: {}'.format(score_count)
        else:
            # They failed hitting the mole
            break

        # Change speed a little
        if speed >= 1.5:
            speed-= 0.07


    if game_over_alert():
        all_off()
        play_game(view, mole_buttons, score_button)


def main():
    view, mole_buttons, score_button = initialise_view()
    view.present('sheet')

    play_game(view, mole_buttons, score_button)

    view.close()

if __name__ == '__main__':
    main()

I haven't installed Pythonista, yet, so I'm not able to test this code, but hopefully it should work nicely.