Лучший способ определить несколько переменных списка внутри класса в Python - PullRequest
1 голос
/ 20 сентября 2019

Я создал скрипт, который хранит и редактирует метаданные в системе.Сейчас я очищаю свой код, определяя класс и методы, ранее я использовал только отдельные функции.

В скрипте я сохраняю старые и новые значения определенных типов метаданных в списках, которые я распечатываю после завершения работы скрипта.Я определил несколько списков (16, если быть точным), что я понял довольно много, передавая их через метод.Мне было интересно, какой самый питонический способ подойти к этому.

Это следующие переменные списка, которые я определяю в начале.В функции / методе я добавляю значения к ним.В конце я печатаю сохраненное значение как отчет.

split_name = []
split_name_new = []
name = []
name_new = []
meta = []
meta_new = []
series = []
series_new = []
product = []
owner = []
desc = []
desc_new = []
keywords = []
keywords_new = []
no_edit_page =[]
no_edit_page_name = []

В классе, который я рассчитывал, это будет выглядеть примерно так (если я определю весь список отдельно)

class Metadata_editor():
    def __init__(self,url):

        self.split_name = [] 
        self.split_name_new = []
        self.name = []
        self.name_new = []
        self.meta = []
        self.meta_new = []
        self.series = []
        self.series_new = []
        self.product = []
        self.owner = []
        self.desc = []
        self.desc_new = []
        self.keywords = []
        self.keywords_new = []
        self.no_edit_page =[]
        self.no_edit_page_name = []

        #Ugly solution because the method gets crowded by all the variables passed through
        def data_edit(self, split_name, split_name_new, name, name_new,.. etc):
            #Not the whole method, but just to give some idea..
            #Selenium function that locates meta
            md = driver.find_element_by_xpath("//input[@name='metadata-name']") 
            meta_data = md.get_attribute("value")
            #replace_words translate the word using a dictionary object
            meta_data_new = replace_words(meta_data,c) 
            meta.append(meta_data)
            meta_new.append(meta_data_new)

Приведенное выше решение, которое я понял, не было бы идеальным.

Я нашел альтернативный способ, которым я мог бы воспользоваться, а именно определить список списков.Тогда решение будет выглядеть примерно так (см. Ниже).Однако «список данных [10]» не так очевиден, как, скажем, «владелец».У меня вопрос, это «лучший» способ решить эту проблему, или у вас есть другие предложения?На самом деле я ничего не имею против этого решения, но мне было интересно, есть ли более «питонический» способ подойти к этому.

class Metadata_editor():
    def __init__(self,url):
        self.data_list=[[] for _ in range(16)] #Creates a list, that contains 16 lists

    # More eloquent solution, as only one variable is passed through. However finding 
    # the right data from data_list is perhaps not as easy before 
    def data_edit(self, data_list): 

        md = driver.find_element_by_xpath("//input[@name='metadata-name']")
        meta_data = md.get_attribute("value")
        meta_data_new = replace_words(meta_data,c)
        data_list[5].append(meta_data)
        data_list[6].append(meta_data_new)

Ответы [ 3 ]

1 голос
/ 20 сентября 2019

Вы можете сохранить его как словарь.Это дает преимущество, заключающееся в возможности ссылаться на ключи по имени, а не запоминать индексы.

class Metadata_editor():
    def __init__(self, url):
       keys = [
            'split_name', 'split_name_new', 'name', 'name_new' 'meta', 'meta_new',
            'series', 'series_new', 'product', 'owner', 'desc', 'desc_new',
            'keywords', 'keywords_new', 'no_edit_page', 'no_edit_page_name',
        ]
        self.data_dict = dict((x, []) for x in keys)

    def data_edit(self):
        md = driver.find_element_by_xpath("//input[@name='metadata-name']")
        meta_data = md.get_attribute("value")
        meta_data_new = replace_words(meta_data,c)
        self.data_dict['meta'].append(meta_data)
        self.data_dict['meta_new'].append(meta_data_new)

Несколько дополнительных замечаний:

  • имена классов в целомследуйте соглашению UpperCaseCamelCase.Таким образом, Metadata_editor условно записывается как MetadataEditor
  • Использование self устанавливает атрибут в классе, к нему можно получить доступ в классе с помощью self, и атрибут не нужно передаватьв метод. Я показал это в примере выше, получая доступ к self.data_dict в методе data_edit .
  • Вы также можете использовать setattr для установки атрибутов класса, как показано в некоторых издругие ответы.
0 голосов
/ 20 сентября 2019

Вы можете инициализировать несколько списков, как показано ниже:

class Metadata_editor():
    def __init__(self,list_names):
        [setattr(self,name,[]) for name in list_names]
me = Metadata_editor(['split_name','split_name_new']) # initialize two lists
me.split_name.append(5) # add value to a list
print(me.split_name, me.split_name_new)

>> [5], []


После установки как частикласса через self.list_name, к списку (ам) можно обращаться глобально в пределах класса - больше не требуется, чтобы его передавали.Чтобы инициализировать списки для определенных значений, вы можете сделать:
def __init__(self,list_names,list_values):
    [setattr(self,name,value) for name,value in zip(list_names,list_values)]
0 голосов
/ 20 сентября 2019

Использование setattr:

...
def __init__(self, url):
    names = '''split_name split_name_new name
        name_new meta meta_new series series_new
        product owner desc desc_new keywords
        keywords_new no_edit_page
        no_edit_page_name'''.split()
    for name in names:
        setattr(self, name, [])
...
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...