どちらのコードが優れているか？

Question

前提・実現したいこと

python3を勉強しています。
python3で2種類のコードを書きました。
どちらも期待している出力をします。
2種類のコードを比べて、良い点と悪い点を教えてください。
また、どちらのコードを書けるように練習した方がいいかアドバイスを下さい。

ご回答よろしくお願いします。

該当のソースコード

class Enemy:
    def __init__(self, name):
        self.name = name

    def attack(self):
        print(self.name + "は勇者を攻撃した")

enemies = ["スライム","モンスター","ドラゴン"]
enemies_num = []

for i,j in enumerate(enemies):
    #オブジェクト名を作る
    a = "number" + str(i)
    enemies_num.append(a)
    #enemies_numリストにnumber0、number1・・・と格納されていく
    
    #オブジェクト名を与える、メソッドで出力する
    enemies_num[i] = Enemy(j)
    enemies_num[i].attack()

該当のソースコード2

class Enemy:
    def __init__(self, name):
        self.name = name

    def attack(self):
        print(self.name + "は勇者を攻撃した")

enemies = []
enemies.append(Enemy("スライム"))
enemies.append(Enemy("モンスター"))
enemies.append(Enemy("ドラゴン"))

for enemy in enemies:
    enemy.attack()

Comments

[hentaiman]

２つ目のコードでモンスター数が1000匹いたらソース直書きで1000行書く事になるけどそれは想定済みですか？ドラクエで300～500のようですけど

[tmp]

優秀とか曖昧で条件が書いてないので希望する回答が得られないかもしれないですね。
個人的には、プログラム内にモンスター名は、書きたくないですね。

[Tay99301]

○この投稿の内容について
　ソースコード2が模範解答になっている問題を解いていて、模範解答を見たときに違和感を感じました。ソースコード1は自分で書いたものです。私からすると、モンスター名を一度リストに入れてしまえば次から書かなくて済むためソースコード1の方が便利だと思いました。なんでソースコード2が模範解答になっているんだろうと気になったので、質問を投稿させてもらいました。
　初心者向けに分かりやすいコードを模範解答にしたのかもしれません。
○投稿について
　なぜこの質問を投稿したのかもっと書けばよかったです。
　プログラムが動作していないというわけではないので、そもそもこのサイトを利用する以外で解決するべきでした。

　質問、意見をしていただき、感謝します。
　回答者のかたもありがとうございました。

[hentaiman]

なるほど
オブジェクト指向の概念を学習させる為でしょうね
たしかに質問に至った背景を書いていればスマートな回答を得られたかもしれませんね

学習中なら、これ無駄じゃないか？という疑問は持ったまま次に進むといいかもしれません
何故かって言うと次のステップの学習内容では模範解答のやり方をベースにしている可能性があるからです
とはいえ教材によるけど

Answer 1

1つめのコードの、

    a = "number" + str(i)
    enemies_num.append(a)

は、その直後に上書きされて消え去るので無意味です。
また、"number" + str(i)という文字列を作るという処理も目的不明です。

比べるとしたら、下記と2つ目のコードですが、

enemy_names = ["スライム","モンスター","ドラゴン"]
enemies = []

for name in enemy_names:
    enemy = Enemy(name)
    enemy.attack()
    enemies.append(enemy)

これは、名前がどこから来るのかに依るでしょう。