「eval」を使用することが悪い習慣であるのはなぜですか？

次のクラスを使用して、曲のデータを簡単に保存しています。

class Song:
    """The class to store the details of each song"""
    attsToStore=('Name', 'Artist', 'Album', 'Genre', 'Location')
    def __init__(self):
        for att in self.attsToStore:
            exec 'self.%s=None'%(att.lower()) in locals()
    def setDetail(self, key, val):
        if key in self.attsToStore:
            exec 'self.%s=val'%(key.lower()) in locals()

これは、if/elseブロックを書き出すよりもはるかに拡張可能だと思います。ただし、これevalは悪い習慣と見なされ、使用するのは安全ではないようです。もしそうなら、誰かが私に理由を説明し、上記のクラスを定義するより良い方法を私に示すことができますか？

はい、evalの使用は悪い習慣です。いくつかの理由を挙げれば：

1. ほとんどの場合、それを行うにはより良い方法があります
2. 非常に危険で安全でない
3. デバッグを困難にする
4. スロー

あなたの場合、代わりにsetattrを使うことができます：

class Song:
    """The class to store the details of each song"""
    attsToStore=('Name', 'Artist', 'Album', 'Genre', 'Location')
    def __init__(self):
        for att in self.attsToStore:
            setattr(self, att.lower(), None)
    def setDetail(self, key, val):
        if key in self.attsToStore:
            setattr(self, key.lower(), val)

編集：

evalまたはexecを使用しなければならない場合があります。しかし、それらはまれです。あなたのケースでevalを使用することは確かに悪い習慣です。evalとexecは間違った場所で頻繁に使用されるため、私は悪い習慣に重点を置いています。

編集2：

OPの場合、evalが「非常に危険で安全でない」という意見には、いくつかの意見の相違があります。これはこの特定のケースには当てはまるかもしれませんが、一般的には当てはまりません。質問は一般的であり、私が挙げた理由は一般的なケースにも当てはまります。

編集3： ポイント1と4を並べ替え

使用evalは弱く、明らかに悪い習慣ではありません。

1. 「ソフトウェアの基本原理」に違反しています。ソースは、実行可能ファイルの合計ではありません。ソースに加えて、への引数evalがあります。これは明確に理解する必要があります。このため、これは最後の手段です。

2. それは通常、軽率なデザインの兆候です。オンザフライで構築された動的ソースコードを使用する正当な理由はめったにありません。委任やその他のオブジェクト指向の設計手法を使用して、ほとんど何でも実行できます。

3. 小さなコードのオンザフライでのコンパイルが比較的遅くなります。より良い設計パターンを使用することで回避できるオーバーヘッド。

脚注として、混乱した社会主義者の手の中では、うまく機能しない可能性があります。ただし、混乱した社会性のあるユーザーまたは管理者に直面した場合、そもそも解釈されたPythonを彼らに与えないことが最善です。本当に悪の手に渡って、Pythonは責任を負うことができます。evalリスクはまったく増加しません。

この場合、はい。の代わりに

exec 'self.Foo=val'

組み込み関数を使用する必要がありますsetattr：

setattr(self, 'Foo', val)

はい、そうです：

Pythonを使用したハック：

>>> eval(input())
"__import__('os').listdir('.')"
...........
...........   #dir listing
...........

以下のコードは、Windowsマシンで実行されているすべてのタスクをリストします。

>>> eval(input())
"__import__('subprocess').Popen(['tasklist'],stdout=__import__('subprocess').PIPE).communicate()[0]"

Linuxの場合：

>>> eval(input())
"__import__('subprocess').Popen(['ps', 'aux'],stdout=__import__('subprocess').PIPE).communicate()[0]"

問題となっている特定の問題については、を使用する代わりにいくつかの方法がありますeval。

指摘したように、最も簡単なのは以下を使用することsetattrです。

def __init__(self):
    for name in attsToStore:
        setattr(self, name, None)

それほど明白ではないアプローチは、オブジェクトの__dict__オブジェクトを直接更新することです。属性をNoneに初期化するだけの場合、これは上記より簡単ではありません。しかし、これを考慮してください：

def __init__(self, **kwargs):
    for name in self.attsToStore:
       self.__dict__[name] = kwargs.get(name, None)

これにより、キーワード引数をコンストラクタに渡すことができます。例：

s = Song(name='History', artist='The Verve')

また、次のように、locals()より明示的に使用することもできます。

s = Song(**locals())

...そして、None名前が次の場所にある属性に本当に割り当てたい場合locals()：

s = Song(**dict([(k, None) for k in locals().keys()]))

オブジェクトに属性のリストのデフォルト値を提供するもう1つの__getattr__方法は、クラスのメソッドを定義することです。

def __getattr__(self, name):
    if name in self.attsToStore:
        return None
    raise NameError, name

このメソッドは、名前付き属性が通常の方法で見つからない場合に呼び出されます。このアプローチは、単にコンストラクターで属性を設定するか、を更新するよりも簡単__dict__ではありませんが、属性が存在しない限り実際には作成されないというメリットがあり、クラスのメモリ使用量を大幅に削減できます。

このすべてのポイント：一般に、回避すべき多くの理由がありますeval-制御できないコードを実行することのセキュリティ問題、デバッグできないコードの実際的な問題など。しかし、さらに重要な理由つまり、通常は使用する必要はありません。Pythonは、内部メカニズムの多くをプログラマーに公開しているため、コードを記述するコードを記述する必要はほとんどありません。

他のユーザーは、コードに依存しないようにコードを変更する方法を指摘しましたeval。を使用するための正当なユースケースを提供します。これevalは、CPythonでも見つかるテストです。

がをレイズするtest_unary.pyかどうかのテストで見つかった1つの例を次に示します。(+|-|~)b'a'TypeError

def test_bad_types(self):
    for op in '+', '-', '~':
        self.assertRaises(TypeError, eval, op + "b'a'")
        self.assertRaises(TypeError, eval, op + "'a'")

ここでの使用法は明らかに悪い習慣ではありません。入力を定義し、動作を観察するだけです。evalテストに便利です。

evalCPython gitリポジトリで実行されたこのの検索を見てください。evalを使用したテストは頻繁に使用されます。

eval()がユーザー指定の入力を処理するために使用される場合、ユーザーは次のようなものを提供してDrop-to-REPLを実行できます。

"__import__('code').InteractiveConsole(locals=globals()).interact()"

あなたはそれでうまくいくかもしれませんが、通常あなたはあなたのアプリケーションで任意のコードを実行するためのベクトルを望んでいません。

@Nadia Alramliの回答に加えて、私はPythonに不慣れで、使用evalがタイミングにどのように影響するかを確認したいと思っていたので、小さなプログラムを試してみました。

#Difference while using print() with eval() and w/o eval() to print an int = 0.528969s per 100000 evals()

from datetime import datetime
def strOfNos():
    s = []
    for x in range(100000):
        s.append(str(x))
    return s

strOfNos()
print(datetime.now())
for x in strOfNos():
    print(x) #print(eval(x))
print(datetime.now())

#when using eval(int)
#2018-10-29 12:36:08.206022
#2018-10-29 12:36:10.407911
#diff = 2.201889 s

#when using int only
#2018-10-29 12:37:50.022753
#2018-10-29 12:37:51.090045
#diff = 1.67292