lambda式で2つの数をswapさせたい

Question

Pythonで搭載されているlambda式で2つの数 a,b があり a<b の時に a,b を入れ替える関数を作りたいです。

n,m=map(int,input().split())
swap=lambda x,y: x,y=y,x if x<y else x,y=x,y
print(swap(n,m))

上のような関数を定義して実行したところ

line 2
swap=lambda x,y: x,y=y,x if x<y else x,y=x,y
    ^
SyntaxError: can't assign to lambda

というエラーが出ました。
どこを直せば良いでしょうか？ご指摘のほどよろしくお願いいたします。

Answer 1

問題がいくつかあります。

問題1: 関数内の代入は、関数外部には影響しません。

>>> def func(a):
...     a = 0
...
>>> b = 10
>>> func(b)
>>>
>>> b
10

global宣言をすれば可能な場合もありますが、あまり良い方法とは言えないでしょう。

問題2: ラムダ式内には、文を書くことはできません。
今のPythonには代入式が無く、代入文しかありません。

問題3: ラムダ式を変数にそのまま束縛するのは推奨されません。

Always use a def statement instead of an assignment statement that binds a lambda expression directly to an identifier.

Yes:

def f(x): return 2*x

No:

f = lambda x: 2*x

引用元： PEP 8 -- Style Guide for Python Code／参考： 日本語訳

---

2つの数 a,b があり a<b の時に a,b を入れ替える関数を作りたい

値を入れかえて返す関数を書くのが落としどころでしょう。

def swap(x, y):
    if x < y:
        return y, x
  
    return x, y

...
n, m = swap(n, m)

Comments

[can110]

Pythonでは代入は式ではなく文なのですね。盲点＆勉強になりました。

[LouiS0616]

if(a = 10) みたいなあるあるバグを防ぐためらしいですね。
Python3.8では代入演算子 := の導入が予定されているようです。

[Naru123]

問題3でreturnを入れていない書き方はよく見るのですがあまりよくないのですね、参考にさせていただきます。
素直に関数定義して書くとします。ありがとうございました

[can110]

あるあるバグですね…別に演算子を用意するというアプローチ、面白いですね。

Answer 2

lambdaではx,y=y,xのような代入はできません。以下のようにそのままタプルを返すとよいかと思います。

swap = lambda x,y: (y,x) if x<y else (x,y)
print(swap(1,2)) # (2, 1)
print(swap(3,2)) # (3, 2)

Comments

[Naru123]

タプルを返すことはできるのですね、勉強になります。ありがとうございます