pythonにおける辞書の作成方法について

Question

前提・実現したいこと

「かけられる数」をキー、掛け算の結果最も大きな値を出す「かける数」を要素
として持つ辞書を作成しようとしています。
以下のような結果を出力したいです。

#１なら９、-4なら-2、7なら9、-2なら-4、5なら9、９なら７
result = {1:9, -4:-2, ....}

発生している問題・エラーメッセージ

現状問題になっているのは
①combinationsを用いることで-2*(-4)のような計算ができていない
②現在できていることから、実行したいことをするためにはどうしたらよいのかわからない

該当のソースコード

from itertools import combinations
data = [1, -4, 7, -2, 5, 9]
ans = []
for x in combinations(data, 2):
    ansNum = x[0] * x[1]
    print(str(x[0]) + '*' + str(x[1]) + '=' + str(ansNum))
    ans.append(ansNum)

試したこと

掛け算の結果が最も大きくなる「かける数」をmax(data)を用いずに見つける方法は
以下のコードで可能でした。

num = 2

data = [4, 7, 2, 3, 9, 0, 8] 
result = [0] * 7

for i in range(0,len(data),1):
    ans = data[i] * num
    result[i] = ans
print(result)
maxnum = max(result)
print(maxnum)
searchindex = result.index(maxnum)
print(data[searchindex])

補足情報（FW/ツールのバージョンなど）

Python 3.6.3
Mac OS High Sierra

Comments

[fuzzball]

「掛け算の結果最も大きな値を出す」というのは何の話をしているのでしょうか？

[fuzzball]

あぁ、「dataの中で」ということでしょうか？自分自身は対象にならないのでしょうか？対象が複数あるときはどうするのでしょうか？

[退会済みユーザー]

「dataの中で」です。ご指摘いただきましてありがとうございます。自分自身は除きます。対象が複数ある場合（９が二つ配列に含まれている場合など）は、配列の前から数えて小さい方を採用したいです。

[fuzzball]

「９が二つ配列に含まれている場合」はどっちでもいいと思いますが‥。具体的に書くと、自分が0のときの話です。「配列の前から数えて小さい方」でいいでしょうか？

[退会済みユーザー]

自分が0の時は正直考えていなかったのですが、「配列の前から数えて小さい方」にします。

Answer 1

かけられる数の符号に注目します。
最大値を求めたいので、正なら、かける数（＝自身を除く）のうち最大値、負なら最小値を採用でよいと分かります。
なお、かけられる数が０なら何をかけても同じですが、とりあえず最大値を採用しています。

data = [1, -4, 7, -2, 5, 9]
#data = [1,1,-1]
ans = {}
for i,v in enumerate(data):
    mul = data[:i] + data[i+1:] # 自身を除く
    if v >= 0:
        ret = max(mul)
    else:
        ret = min(mul)
    ans[v] = ret

for k,v in ans.items():
    print(k,v,k*v)

Comments

[退会済みユーザー]

ご回答いただきましてありがとうございました。

Answer 2

眠気覚ましに。
実直に実装しました。

data = [1, -4, 7, -2, 5, 9, 0]
result = {v: max(data[:i] + data[i+1:], key=lambda x: x * v) for i, v in enumerate(data)}
print(result)
# {0: 1, 1: 9, 5: 9, 7: 9, 9: 7, -4: -2, -2: -4}

0の場合は配列の前から数えて小さい方

は実装していません。（たまたまそうなってるだけ）

謝辞

最初、自身を取り除くのに、

tmp = data[:]
del tmp[i]

としていましたが、can110さんの回答を参考にさせていただきました。

Comments

[can110]

なるほど。結果から最大とる方が分かりやすいですね。
key=lambda x: x * vは目から鱗です。

[fuzzball]

質問に沿って素直に書きました。
符号で切り分けるのは「やられた」って感じですねｗ
can110さんのおかげで一行に出来ました。ありがとうございました。

[退会済みユーザー]

ご回答いただきましてありがとうございました。

Answer 3

質問にそって素直にかいてみました。
data.py

data = [1, -4, 7, -2, 5, 9]

dict = {}
for x in data:
    ys = data[:]
    ys.remove(x)
    dict[x] = int(max([x * y for y in ys]) / x)

print(dict)

実行結果

$ python3 data.py
{1: 9, -4: -2, 7: 9, -2: -4, 5: 9, 9: 7}