Можно ли в питоне использовать следующую конструкцию?

Question

[Михаил Болеев]

Есть такая конструкция:

def func_2(x):
    if x == 1:
        return 'единица'
    if x == 2:
        return 'двойка'
    if x == 3:
        return 'тройка'

def func_1(x):
    return func_2(x)

def func_3():
    print('завершение')

n = 2

if func_1(n) == 'единица':
    print('единица')
if func_1(n) == 'двойка':
    print('двойка')
if func_1(n) == 'тройка':
    print('тройка')
func_3()

Казалось бы, программа успешно работает. Но на практике (в более сложном проекте) это происходит не всегда. Иногда не срабатывает условие внизу, иногда - последняя функция. Не пойму, в чем причина?
P. S. Функции являются методами класса.

Comments

[longclaps]

Ты написал какую-то удивительную чепуху, которая "успешно работает", и спрашиваешь о чем-то другом, которое работает не всегда. Возможно, ты неаккуратен с отступами. Определённо, ты страдаешь ерундой.

[Михаил Болеев]

longclaps, Если интересно, я по просьбе создал аналогичный вопрос с реальным кодом. Его можно посмотреть и здесь, в комментариях ниже. ))

Answer 1

[Ivan Yakushenko]

numbers = {
    1: 'единица',
    2: 'двойка'
}

x = 1
y = 3

print(numbers.get(x, 'Неизвестное число'))
print(numbers.get(y, 'Неизвестное число'))

Comments

[Михаил Болеев]

При чем здесь это? Я же не спрашивал как реализовать задачу. Просто привел банальный пример.

[Ivan Yakushenko]

Михаил Болеев, ты написал кусок бреда, а не кода. Мало того, что на питоне так писать не нужно, так еще и оказывается что ты привел не тот код, который по факту есть у тебя. Или пиши код как есть с указанием в каких ситуациях не работает, или пиши правильно. Второй вариант лучше.

[Михаил Болеев]

Ivan Yakushenko, Почему бред? Вопрос был про вызов функции, которая возвращает результат другой функции.

[Ivan Yakushenko]

Михаил Болеев, ну голову включи, ты привел пример, который не имеет никакого отношения к твоему реальному коду, плюс не написал при каких обстоятельствах он не работает.
Более того даже в этом "примере":
- зачем тебе func_1?
- зачем ты каждый раз вызываешь func_1?
- зачем ты используешь if-else если тут явно словари напрашиваются?

[Михаил Болеев]

У меня есть функция (метод), из которой запускается другая функция, которая возвращает результат. Я пытаюсь проверять результат при помощи описанной конструкции, но срабатывает не всегда (описано как).

[Ivan Yakushenko]

Михаил Болеев, мне сейчас нужно в голове с твоих слов пытаться код написать? В чем проблема привести конкретный фрагмент кода, которого касается вопрос?

[Михаил Болеев]

Могу привести, конечно. Не хотел городить огород. Нужно будет в коде разбираться.

[Михаил Болеев]

Реальный код:

class Shot:

    def __init__(self):
        self.x = None
        self.y = None

    def shot_by_ships(self):
        xy = input('\033[1;32mВаш выстрел (например, d3): ')
        if len(xy) == 2:
            input_x = xy[0]
            if xy[1].isdigit():
                input_y = int(xy[1])
            else:
                input_y = 10
            if input_x in Board.letters and 0 <= input_y < 10:
                self.x = Board.letters.index(input_x)
                self.y = int(xy[1])
                if radar.board[self.x][self.y] == Cell.miss_cell:
                    print('Эта клетка занята! Еще разок...')
                    self.shot_by_ships()
                return self.receive_shot()

            else:
                print('Неправильный ввод. Еще разок... ')
                self.shot_by_ships()
        else:
            print('Должно быть две координаты. Еще разок...')
            self.shot_by_ships()


    def receive_shot(self):
        sx = self.x
        sy = self.y
        # radar.count_shot += 1

        if type(comp.board[sx][sy]) == Ship:
            ship = comp.board[sx][sy]
            ship.len_ship -= 1


            if ship.len_ship <= 0:
                radar.mark_destroyed_ship(ship)
                print('\033[1;33mКорабль уничтожен!')
                Ship.count_ship -= 1
                return ship

            radar.board[sx][sy] = Cell.damaged_ship
            print('\033[1;33mПопадание!')
            return 'get'

        else:
            radar.board[sx][sy] = Cell.miss_cell
            print('\033[1;33mПромах!')
            return 'miss'


while Ship.count_ship > 0:
    shot = Shot()
    # shot.shot_by_ships()
    if shot.shot_by_ships() == 'miss':
        print('\033[1;33mПромах!', 'Внешняя запись')
    elif shot.shot_by_ships() == 'get':
        print('\033[1;33mПопадание!', 'Внешняя запись')
    elif type(shot.shot_by_shipst()) == Shot:
        print('\033[1;33mКорабль уничтожен!', 'Внешняя запись')
    # radar.clear_screen()

    print('Отрисовка')
    radar.print_board()

[Михаил Болеев]

Вот результат:

Ваш выстрел (например, d3): e5
Попадание!
Ваш выстрел (например, d3): d5
Промах!
Отрисовка

Как видно, после попадания отрисовка не происходит. Второй принт не запускается.
После промаха не запускается второй принт.
Может быть все наоборот.

[Ivan Yakushenko]

Михаил Болеев, это должно быть в вопросе, а не в комментарии, желательно в новом.

[15432]

Михаил Болеев,
вы забыли пару возвратов:

print('Неправильный ввод. Еще разок... ')
            return self.shot_by_ships()
        else:
            print('Должно быть две координаты. Еще разок...')
            return self.shot_by_ships()

[Михаил Болеев]

Это не отражается на результате. Представьте, что я правильно ввожу координаты. А без ретёрна все работает (скрипт запускается по новому).

Ваш выстрел (например, d3): g5
Промах!
Ваш выстрел (например, d3): l3
Неправильный ввод. Еще разок...
Ваш выстрел (например, d3): h4
Промах!
Отрисовка

[o5a]

Михаил Болеев, проблема в том, что в каждой if/elif используется вызов функции, соответственно она каждый раз вызывается заново даже при elif. Результат нужно считать один раз, а затем использовать в проверке уже его.

shot_check = shot.shot_by_ships()
if shot_check == 'miss':
    print('\033[1;33mПромах!', 'Внешняя запись')
elif shot_check == 'get':
    print('\033[1;33mПопадание!', 'Внешняя запись')

[Михаил Болеев]

o5a, Да. Вы правы. Уже использовал такой вариант. Но почему правильно не работает, если вызывается каждый раз функция, остается непонятным. Спасибо!

[o5a]

Михаил Болеев, что ж непонятного? Каждое условие if/elif должно знать значения используемых в выражении элементов. Значение функции неизвестно, пока ее не запустишь. shot.shot_by_ships() и вызывается каждый раз заново. Использование вызова функции нигде не кэшируется, если так понятнее.

[Михаил Болеев]

o5a,

Использование вызова функции нигде не кэшируется, если так понятнее.

Да, теперь понятно. Спасибо!