インスタンス変数の使い方

Question

前提・実現したいこと

初心者です。pythonの勉強をしながらDS18B20の温度センサーを使って温度を読みとろうと学習中です。printで関数を呼ぶと下記のようなエラーが発生します。どうすればよいでしょうか？お願いします。

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

AttributeError: type object 'DS18B20' has no attribute 'raw'

該当のソースコード

import linecache

PATH = '/sys/bus/w1/devices/xx-xxxxxxxx'
FILE_NAME = '/w1_slave'


class DS18B20:
    def __init__(self):
        self.raw = linecache.getline(PATH + FILE_NAME, 2)[29:]
        self.value = round(float(self.raw) / 1000.0, 1)


if __name__ == "__main__":
    print("raw value = " + DS18B20.raw)
    print("temperature = " + str(DS18B20.value))

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

python3.7.1

Answer 1

class DS18B20:

これは「クラス定義」ですね。DS18B20という名前のクラスを作ります。

    def __init__(self):
        self.raw = linecache.getline(PATH + FILE_NAME, 2)[29:]
        self.value = round(float(self.raw) / 1000.0, 1)

これは「コンストラクタ」と呼ばれるものですね。「インスタンス」を作るときの初期化処理を記述します。インスタンス化すれば自動的に呼ばれます。

if __name__ == "__main__":
    print("raw value = " + DS18B20.raw)
    print("temperature = " + str(DS18B20.value))

クラスのraw属性などを見ようとしても、「コンストラクタ」が呼ばれていないのでそんな属性はないですね。

たとえばこう修正すれば動くでしょう。

if __name__ == "__main__":
    sensor = DS18B20()
    print("raw value = " + sensor.raw)
    print("temperature = " + str(sensor.value))

ただし、これだけでは特に嬉しいことはないので、

• そもそも取得処理とかはコンストラクタに書くべきなのか。なんかそれ用のメソッドがあると良いのかもしれません（仕様を把握していないので実のところはよくわかりませんが）。
• PATHとFILE_NAMEはコンストラクタの引数で絶対パスを受け取る実装にでもすると多少良いかもしれません。

といった改良点はあるでしょう。

基礎知識不足だとうまく作るのはなかなか難しいので、pythonの勉強が先ですね。一通り終わってから応用に手を出しましょう。

Comments

[退会済みユーザー]

コンストラクタが原因だとはなんとなく気がついてはいたのですがif __name__の外に書いてました。中に書かないといけなかったのですね。ありがとうございました。もっと勉強します。