2016-08-26 4 views
0

Ich habe ein Skript in Python geschrieben, um die REST-API meiner Jira-Instanz für bestimmte Informationen abzufragen, die in ein Google-Dokument hochgeladen werden sollen. Zugegeben, ich bin kein professioneller Programmierer und bin bestenfalls noch Amateur. Wie kann ich diesen Code aufräumen und ihn pythonischer und eleganter machen?Wie kann ich diesen Python-Code besser/eleganter machen?

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

Diese Frage würde wahrscheinlich viel besser auf Codeansicht arbeiten: http://codereview.stackexchange.com/. Sie akzeptieren den ** Arbeitscode ** von Personen und können Ihnen helfen, ihn zu verbessern. Ich schlage vor, es dorthin zu bringen. –

+0

Tolle Idee. Vielen Dank! – mpoggy

+0

Sie sind herzlich willkommen :) Ich habe ihre Überprüfung Service von unschätzbarem Wert gefunden. –

Antwort

3

Für den Anfang können Sie eine for-Schleife verwenden, um Ihren Code zu bereinigen und für die Armen Variablennamen etwas eingeben

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

leider speichern. :/

+0

Guter Vorschlag, aber diese Art von Frage gehört wirklich nicht hierher, und so sollten sie in Ruhe gelassen werden. –

Verwandte Themen