空リストのPythonバグ

Question

私はPythonを使ってInform 7（inform7.com）の基本機能をエミュレートしようとしています。室内にすべてのオブジェクトを印刷するように部屋を設定しようとしていて、部屋に何もオブジェクトがない場合は、まだIn this room there is aが印刷されます。コードの関連部分はここにあります。

def __repr__(self): 
    if self.room_firstrun: 
     for i in things: 
      if things[i] == self.name: 
       self.objects_in_room.append(i) 
     self.room_firstrun = False 

    fullstring = 'In this room, there is a ' 
    for i in self.objects_in_room: 
     if not self.objects_in_room: 
      fullstring = '' 
      break 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) > 2: 
      stringtoappend = ', and a ' + i + '.' 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 2: 
      stringtoappend = ' and a ' + i + '.' 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 1: 
      stringtoappend = i + '.' 
     elif i == self.objects_in_room[0]: 
      stringtoappend = i 
     else: 
      stringtoappend = ', a ' + i 
    fullstring += stringtoappend 
    stringtoappend = '' 
    return '\n----------' + self.name + '----------\n\n' + self.desc + '\n\n' + fullstring 

。すべてのアイテムを削除してリストが空であることを確認した場合でも、if not self.objects_in_room: fullstring = '' breakという行には決して届かない。私はしばらくそれを修正しようとしてきましたが、それは動作していません。また、部屋に2つのアイテムがあると、最初のものが表示されません。多分私は別の質問にそれを置くべきです。事前にお返事ありがとう！

Answer 1

if not self.objects_in_roomは、forループ内にあるため、決して到達されませんでした。また、forループは、self.objects_in_roomが空でない場合にのみ実行されます。この問題を解決するには、ループ

def __repr__(self): 
    if self.room_firstrun: 
     for i in things: 
      if things[i] == self.name: 
       self.objects_in_room.append(i) 
     self.room_firstrun = False 

    fullstring = 'In this room, there is a ' 
    if not self.objects_in_room: 
      fullstring = '' 
    for i in self.objects_in_room: 

     if i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) > 2: 
      fullstring += ', and a {0}.'.format(i) 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 2: 
      fullstring += ' and a {0}.'.format(i) 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 1: 
      fullstring += '{0}.'.format(i) 
     elif i == self.objects_in_room[0]: 
      fullstring += i 
     else: 
      fullstring += ', a {0}'.format(i) 
    return '\n----------' + self.name + '----------\n\n' + self.desc + '\n\n' + fullstring 

外if not self.objects_in_roomを置くか、行うことができます：あなたはfor..else文を使ってPython 3を使用している場合。

def __repr__(self): 
    if self.room_firstrun: 
     for i in things: 
      if things[i] == self.name: 
       self.objects_in_room.append(i) 
     self.room_firstrun = False 

    fullstring = 'In this room, there is a ' 

    for i in self.objects_in_room: 

     if i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) > 2: 
      fullstring += ', and a {0}.'.format(i) 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 2: 
      fullstring += ' and a {0}.'.format(i) 
     elif i == self.objects_in_room[len(self.objects_in_room) - 1] and len(self.objects_in_room) == 1: 
      fullstring += '{0}.'.format(i) 
     elif i == self.objects_in_room[0]: 
      fullstring += i 
     else: 
      fullstring += ', a {0}'.format(i) 
    else: 
     fullstring = '' 
    return '\n----------' + self.name + '----------\n\n' + self.desc + '\n\n' + fullstring 

なぜ、stringtoappendを使用していますか？なぜそれを直接フルストリングに追加しないのですか？