2016-08-26 4 views
0

Я написал сценарий в Python, чтобы запросить API RESTful экземпляра Jira моей компании, чтобы определенная информация была загружена в Google Doc. По общему признанию, я не профессиональный программист, и в лучшем случае я до сих пор любитель. Как я могу очистить этот код и сделать его более Pythonic и элегантным?Как я могу сделать этот код Python лучше/более элегантным?

cell = 2 

for issue in issues: 
    title = issue.fields.customfield_xxxx 
    first_name = issue.fields.customfield_xxxx 
    last_name = issue.fields.customfield_xxxx 
    email = issue.fields.customfield_xxxx 
    username = first_name[0] + last_name 
    wks.update_acell('A{}'.format(cell), '{}'.format(first_name)) 
    wks.update_acell('B{}'.format(cell), '{}'.format(last_name)) 
    wks.update_acell('C{}'.format(cell), '{}'.format(title)) 
    wks.update_acell('I{}'.format(cell), '{}'.format(email)) 
    wks.update_acell('E{}'.format(cell), '{}'.format(
     username + "@company.com")) 
    wks.update_acell('F{}'.format(cell), '{}'.format(username)) 
    wks.update_acell('H{}'.format(cell), '{}'.format(
     first_name + " " + last_name)) 
    wks.update_acell('G{}'.format(cell), '{}'.format(
     first_name + " " + last_name)) 
    wks.update_acell('J{}'.format(cell), '{}'.format(x)) 

cell += 1 
+3

Этот вопрос, вероятно, будет работать намного лучше при просмотре кода: http://codereview.stackexchange.com/. Они принимают код, ** работающий **, и могут помочь вам улучшить его. Я предлагаю переместить его туда. –

+0

Отличная идея. Спасибо! – mpoggy

+0

Вы очень приветствуются :) Я нашел их обзорный сервис неоценимым. –

ответ

3

Для начала можно использовать для цикла, чтобы очистить ваш код и сохранить некоторые набрав

cellfields = [['a',first_name],['b',last_name]['c',title]] ... etc 
    for fields in cellfields: 
     wks.update_acell(fields[0] + str(cell), fields[1]) 

жаль бедного переменного наименование. :/

+0

Хорошее предложение, но этот тип вопроса действительно здесь не принадлежит, и поэтому их следует оставить в покое. –

Смежные вопросы