alinazakharova1997 / tokenizator Goto Github PK

В документации класса приведены параметры метода, так быть не должно

Ответ:
Множество лучше, потому что дальше делаются проверки с использованием оператора in, за которым в случае массива стоит полный перебор элементов массива и сравнение их с искомым, а в случае множества -- одна операция проверки наличия ключа в хеш-таблице.

А зачем Вам обращаться к элементу массива?

Этот экземпляр сейчас создаётся при каждом вызове get_window(). Смысл записывать его не в обычную переменную, а в класс окна, при такой реализации непонятен. В класс окна его, может быть, и имеет смысл записать, но тогда не в методе get_window(), а непосредственно в определении класса окна:

class Window(object):
    ...
    tokenizer = Tokenizer()

и далее обращаться к нему как cls.tokenizer, не пересоздавая.

Совершенно не понятно, зачем нужен этот атрибут у класса контекстного окна. В коде он нигде не используется.

В python обычно классы именуются в единственном числе, за исключением классов некоторых коллекций (множеств, массивов, словарей), экземпляры которых являются чем-то, что можно назвать существительным во множественном числе, ср.:

class Cat(Animal):
     def meow(self):
         print("Meow!")


 class Cats(list):
     def meow(self):
         for cat in self:
             cat.meow()

token.position + len(token.s) -- это действительно позиция начала окна, но в перевёрнутой подстроке, которая была построена ранее (string[position.end::-1]), а не в строке string, в которой будет выделяться цитата. Эта перевёрнутая подстрока начинается в строке string в позиции position.end и заканчивается в начале string. Соответственно, чтобы перевести любую позицию p в этой перевёрнутой подстроке в позицию в строке string, необходимо вычесть p из position.end. Соответственно, позиция начала окна в строке string выглядит как position.end - (token.position + len(token.s))