クラスの機能を引き継げない

Question

クラスの機能を引き継ぎたい

クラスの機能を引き継ぐという練習していました。

class Person:
    def __init__(self, name, height, weight):
        self.name = name
        self.height = height
        self.weight = weight

    def bmi(self):
        result = self.weight / (self.height * self.height)
        print(self.name,'のBMIは', result,'です。')

というファイルをまず作成し、myclass.pyで保存したのち、別フォルダに

class BusinessPerson(Person):
    def __init__(self, name, height, weight, title):
        super().__init__(name, height, weight)
        self.title = title

    def work(self):
        print(self.title, 'の', self.name,'は働いていてる。')

とファイルを作成し、myclass.pyで保存しました。

###作成したコード

import myclass

bp = myclass.BusinessPerson('モミジ', 1.26, 24, 'freshman')
bp.bmi()
bp.work()

以上のようにコードを作成し実行しても、以下のようなエラーが表示されます。

表示されるエラー

class BusinessPerson( Person):
NameError: name 'Person' is not defined

スペルや文法ミスを探しても見つからないので、教えていただけると助かります。
よろしくお願いします、

ファイル構造をスクショしました。
一番下の方の1002に1つ目のmyclass.py、1003に2つ目のmyclass.pyを保存し、1003のclass.client.pyにコードを書き実行しました。

###変更してやってみた
ここで以下のように1002に保存してあった1つ目のmyclass.pyの名称をmyclass2.pyとして切り取り、1003に移動しました。
1003に保存してあった2つ目のmyclass.pyの名称をmyclass3.pyに変更して、

import myclass3

bp = myclass3.BusinessPerson('モミジ', 1.26, 24, 'freshman')
bp.bmi()
bp.work()

class.client.pyを実行したのですが、結果は変わりませんでした。

ややこしくてすみません。よろしくお願いします。

maciOS 10.14.1
visual studio code

Comments

[quickquip]

作成したファイルを格納したフォルダの構成と、実行しているファイルの場所が大事。

[azuapricot]

↑の人の通りで BusinessPersonで呼び出してる元のクラスの Personなんてしらねーぞーっていうエラーなのでファイル構造のスクリーンショットなどをご提示ください～

[gunmed]

回答ありがとうございます。変更してみました。よろしくお願いします。

Answer 1

myclass.pyはモジュール検索パスに記載されているルールおよび場所から検索されますが、見つからないとimportできません。
一番単純な解決策は３つのファイルをすべて同じ場所に配置することです。
この場合、重複しているmyclass.pyファイル名はどちらか変更する必要があります。

追記

書いてみました。なおファイル名は分かりやすいように書き換えました。

# persion.py
class Person:
    def __init__(self, name, height, weight):
        self.name = name
        self.height = height
        self.weight = weight

    def bmi(self):
        print(self.weight / self.height / self.height)

# buisiness_persion.py
from person import Person
class BusinessPerson(Person):
    def __init__(self, name, height, weight, title):
        super().__init__(name, height, weight)
        self.title = title

    def work(self):
        print(self.title, 'の', self.name,'は働いていてる。')

# client.py
from buisiness_persion import BusinessPerson
bp = BusinessPerson('モミジ', 1.26, 24, 'freshman')
bp.bmi()
bp.work()

実行結果

15.11715797430083
freshman の モミジ は働いていてる。

Comments

[gunmed]

とりあえず頂いたアドバイス通り、ファイルを同じ場所にして、名称を変えてみましたが、なかなかうまくいきません。質問内容を変更したので、何かわかることがあれば教えていただけると幸いです。

[gunmed]

素早い回答を丁寧にありがとうございました。解決しましたので、ベストアンサーにしたいと思います。

Answer 2

やりたいことに対して作業内容がややこしすぎるような・・・

• 3つもファイルを作ると互いにどうimportするかで（この質問のように）ハマりがち
• 別フォルダに同名のファイルとして置くとimportがややこしくなる

単に1つのファイルにまとめて書くんじゃ駄目ですか。

---

直接のエラー原因は（若干憶測含みですが）2つめのmyclass.pyがもう一方のmyclass.py（の中のPersonクラス）をimportしないといけないのに、それを書いていないことです。

ディレクトリ構造がわからないとアドバイスのしようがないので、とりあえずこれ以上のことは書かないでおきます。

Comments

[gunmed]

初プログラミングで1冊目の入門書の指示通りに勉強しているので、hayataka2049さんがおっしゃってる通り、1つのファイルにまとめて書くのがいいと思いますが、クラスの機能を引き継ぐという練習でやっているという体でお願いします、、、。
ディレクトリ構造というのかわからないですが、ファイルの構造をスクショしてみたので、参照していただき、何かあれば教えていただけるとありがたいです。
ちなみに、myclass3.pyのコード文頭に
import myclass2
を追加して実行してみたのですが、同じ結果になってしまいました、、、

[hayataka2049]

class BusinessPerson(Person):
を定義しているファイルでPersonを参照するために、
from Personを定義しているモジュール import Person
を書く必要があります。
ファイル移動後のディレクトリ構成なら、myclass3.pyの先頭にfrom myclass2 import Personで通るかな？

[gunmed]

はい、それで通ることができました。ありがとうございました。
もう一人の方の方でもできてしまったので、ベストアンサーを決めかねました。そこで、公正に時間が早かったもう一人の方をベストアンサーにしようと思います。気分を害してしまったらすみません。わかりやすい回答でしたので、また機会があればよろしくお願いします。