2017-02-08 12 views
0

は、それがこのコードを改善することは可能ですか?値をオプションにしますか?オプションの文字列形式

ありがとうございます!あなたが書くことができ

cms = '{} {}{}'.format(
    cms, usage_map["switches"][switch], 
    '' if isinstance(value, bool) and value else ' {}'.format(value) 
) 
+1

この質問は、[コードレビュースタックエクスチェンジ](http://codereview.stackexchange.com/)の方が適している可能性があります。 –

+0

複数の行がありますが、上記のコードは非常に明確です'value'の型チェック;-)。あなたはおそらくもっと短いものを見つけることができますが、私はあなたが何かをかなり見つけることは疑っています。 – mgilson

答えて

2

if value is True:

cms = "%s %s" % (cms, usage_map["switches"][switch]) 
if isinstance(value, bool) and value: 
    cms += " %s" % value 

をまた、私はあなたのように条件を書き換えることができると思います。

+2

何が価値あるのか、私はこれがOPがすでに持っているものよりも私がこれを考えているとは思っていません... – mgilson

0

:あなたは、この使用することができます