PythonsのAttributeError： 'NoneType'オブジェクトに属性 'errors'がありません

Question

私はそれが何度もここにあったことは知っていますが、私の場合は正しい答えが見つかりませんでした。
最初に：私はハッシュなどで実行されない簡単なデータベースシステムを作っています（少なくとも今のところは）。今私は立ち往生した。

import sys 
import os 

filename = "" 
database = "" 
path = "" 
table = "" 

class Nollty: 
    returns = 0 
    errors = 0 

    def __init__(self, filename, database): 
     self.filename = filename 
     self.database = database 
     self.path = self.filename + "databases/" + self.database 
     openfile = open(self.path + "/db_required", "r") 
     if not openfile.errors: 
      self.returns = 1 
     if not os.path.exists(self.path + "/db_required"): 
      self.returns = 0 
     openfile.close(); 

    def select(self, table): 
     errors = 0 
     self.table = table 
     openfile = open(self.path + "/" + self.table, "r") 
     if not openfile.errors: 
      errors = 1 
     if not os.path.exists(self.path + "/" + self.table): 
      errors = 0 
     openfile.close(); 


nollty = Nollty("", "test") 
if nollty.returns == 1: 
    print "Successfully connected to the database!" 

query = nollty.select("aaa_auto") 
if query.errors == 0: 
    print "Successfully chosen the table!" 

エラー出力は次のようになります。

Traceback (most recent call last): 
File "/home/spotrudloff/Python/Nollty/nollty.py", line 40, in <module> 
if query.errors == 0: 
AttributeError: 'NoneType' object has no attribute 'errors' 

問題は、私はPHPプログラマーだと思われる、と私は（そう、私の思考はまだある「PHPy」）数時間で今日のPythonを学びました。

すべての回答ありがとうございます。

Answer 1

select()は明示的な値を返さないため、戻り値はNoneTypeです。 select()がコードの成功または失敗に応じて1または0を返すようにコードを変更します。

Answer 2

select（）が値を返さないため、デフォルトでNoneType（null）に設定されているようです。あるいは（おそらく何をするつもりだったのか）、クエリを無制限に変更してください。

Answer 3

クラス変数としてreturnsとerrorsを使用することはお勧めできません。 Nolltyのインスタンス数にかかわらず、これらの変数のそれぞれ1つだけインスタンスがあります。代わりに、この操作を行います。

def __init__(self, filename, database): 
    self.returns = 0 
    self.errors = 0 
    # rest of __init__ 

次に、戻り値を示すためreturnsの使用は、どちらか良いアイデアのように見えるしていません。 Pythonでは、通常、コンストラクタの問題を示すために例外が発生します。そうすれば、呼び出し側はreturnsのチェックを忘れることによって問題を無視することはできません。

同様に、select()の例外を使用して、パラメータに問題があることを示します。私のお勧めは、に、を取り除くことです。returnsとerrorsです。

selectから値を返さないので、はNoneになります（PythonではNoneが特別な値です）。 select()から有用なものを返すか、有用な値を返さない場合は結果に何も割り当てないでください。

Answer 4

selectメソッドのerrors変数はローカル変数です。クラスエラー変数を明示的に設定する必要があります。また、他の人があなたのselectメソッドにreturn文がないことを示唆しています。あなたのコードから、あなたはnolltyのerrors変数にアクセスしようとしているように見えますが、論理的にはNoneの問い合わせ変数でそれをチェックしています。

目的に応じて2つのオプションがあります。 selectメソッドからエラーを返すか、selectメソッドが戻る前にself.errors = errorsを設定します。または、以下のように、好きな場合は両方を行います。通常、選択が失敗した場合はFalseを返し、成功した場合はTrueを返しますが、必須ではありません。

class Nollty: 
    ... 
    def select(self,table): 
     ... 
     self.errors = errors 
     return errors 

errs = nollty.select("aaa_auto") 
if nollty.errors == 0: 
    print "Successfully chosen the table!" 

##functionally equivalent 
#if errs==0: 
# print "Successfully chosen the table!"