任何方式來改善這個python函數？

Question

def __number(): 
    # This line returns the number of the latest created object 
    # as a "Factura" object in format "n/year" 
    last = Factura.objects.filter(f_type__exact=False).latest('number') 

    # We convert it into a string and split it to get only the first number 
    spl = str(last).split('/')[0] 

    # Convert it into integer so we can do math 
    n = int(spl) 

    # Get the current year 
    y = date.today().strftime('%y') 

    # If none return 1/year 
    if n == None: 
     return str(1) + '/' + str(y) 

    # Else we increment the number in one. 
    else: 
     n = n + 1 
     return str(n) + '/' + str(y) 

它做什麼：它會自動生成格式爲'1/year''2/year'的數字等等。如果用戶引入了其他數字，p.e. 564/10功能跟隨它，下一個將是565/10。

即使用戶介紹p.e.在與564/10的輸入之後的34/10功能將遵循最大數量。

我做對了嗎？還是有更好的方法來做到這一點？

---

更新：

def __number(): 
    current_year = date.today().strftime('%y') 
    try: 
     facturas_emmited = Factura.objects.filter(f_type__exact=False) 
     latest_object = facturas_emmited.latest('number').__str__() 
     first_number = int(latest_object.split("/")[0]) + 1 
    except Factura.DoesNotExist: 
     first_number = 1 
    return '%s/%s' % (first_number, current_year) 

Answer 1

這真的只是一個開始，但我會開始用自我記錄代碼替換一些註釋。

def __number(): 
    # "Factura" object in format "n/year" 
    latest_object = Factura.objects.filter(f_type__exact=False).latest('number') 

    # Better name can be available if you explain why the first number is important and what it means 
    # Do Factura objects not have a __repr__ or __str__ method that you must cast it? 
    first_number = int(str(latest_object).split('/')[0]) 
    current_year = date.today().strftime('%y') 
    # Use "is None" rather than "== None" 
    if first_number is None: 
     return '1/%d' % current_year 
    # No else needed because of return above 
    # Why do we add 1 to first number? Comments should explain _why_, not how 
    return '%d/%d' % (first_number + 1, current_year) 

Answer 2

能last是None？如果是這樣的話，檢查一下會很好：

# get last as before 

if last: 
    n = int(str(last).split("/")[0]) + 1 
else: 
    n = 1 

# get y as before 

return str(n) + "/" + str(y) 

這裏的另一個改進是你只在一個地方建立結果字符串。

我不知道Factura對象是什麼，但是通過調用它的某些方法，您不能得到n的值嗎？這會比將其轉換爲字符串更好，將其拆分並取出最後一部分。

Answer 3

我前段時間通過使用object.id/year（其中object/id是數據庫id）解決了類似的問題。

它保證這將是唯一的，自動增量（你不需要做n = n + 1，理論上可以導致重複的數據庫值）。

你可以通過覆蓋save方法來做到這一點，唯一的竅門是你需要先保存一個對象（id被分配），然後再創建id/year數字並再次保存（也許有更好的方法來做到這一點比雙倍保存）。

def save(self, force_insert = False, force_update = False): 
    super(Factura, self).save(force_insert, force_update) 
    current_year = date.today().strftime('%y') 
    self.identifier = '%s/%s'%(self.id, current_year) 
    super(Factura, self).save(force_insert, force_update)