我如何縮短這個if語句？

Question

我有這個函數來檢查我們在通過座標系旅行時面對的方式，通過查看元組中的值如何增加或減少與路點列表中的下一個值相比較。看代碼，它感覺複雜和笨拙：

a.facing = self.direction(a.travel_list[0], a.travel_list[1]) 

def direction(self, start, end): 
    s_width = start[0] 
    s_height = start[1] 
    e_width = end[0] 
    e_height = end[1] 
    # check directions 
    if s_height < e_height: 
     if s_width < e_width: 
      return 'right' 
     elif s_width > e_width: 
      return 'up' 
     else: 
      return 'up_right' 

    elif s_height > e_height: 
     if s_width < e_width: 
      return 'down' 
     elif s_width > e_width: 
      return 'left' 
     else: 
      return 'down_left' 

    elif s_height == e_height and s_width < e_width: 
     return 'down_right' 
    else: 
     return 'up_left' 

返回值被調整爲順時針旋轉一步。我的問題是，我們如何更改代碼以使功能更短，更高效？

編輯：請注意，只能在指定的8個方向上進行移動。

Answer 1

使用（）的基礎上CMP字典返回值：

def direction(self, start, end): 
    return table[cmp(start[0], end[0]), cmp(start[1], end[1])] 

構建字典來概括當前的邏輯：

table = { 
    (-1, -1): 'right', 
    (-1, 1): 'up', 
    (-1, 0): 'up_right', 
     ... 
} 

如果你使用Python 3，您需要定義自己CMP（）功能：

cmp = lambda x, y: -1 if x < y else 1 if x > y else 0 

Answer 2

步驟1：獲取y和x的距離

步驟2：呼叫ATAN2（X，Y），以獲得的角度（http://docs.python.org/2/library/math.html#math.atan2）

步驟3：PI/4和圓形的鴻溝最近的整數以獲得-4和4之間的值

第4步：現在您可以做一個case/switch，每個值都是不同的基本方向（例如-4和4將是相同的方向，例如兩者東）

編輯：我剛剛意識到這一點如果你考慮如比東仍東高10度。 XD讓我想到一個更好的...

編輯2：好了，試試這個：

步驟1：兩個值

步驟2的元組：值1 = CMP（X1， X2）

步驟3：值2 = CMP（Y1，Y2）

步驟4：在元組方向的一個表執行查找（例如0,1是北，0，-1是南， 1,1是東北部等）並返回結果

Answer 3

您可以使用and運營商：

a.facing = self.direction(a.travel_list[0], a.travel_list[1]) 

def direction(self, start, end): 
    s_width, s_height, e_width, e_height = start[0], start[1], end[0], end[1] 
    # multiple assignments 
    # check directions 
    if s_height < e_height and s_width < e_width: 
     return 'right' 
    elif s_height < e_height and s_width > e_width: 
     return 'up' 
    elif s_height < e_height: # this will get executed if the top two are false 
     return 'up_right'  # its similar to using the `else` in a nested conditional 
    elif s_height > e_height and s_width < e_width: 
     return 'down' 
    elif s_height > e_height and s_width > e_width: 
     return 'left' 
    elif s_height > e_height: 
     return 'down_left' 
    elif s_height == e_height and s_width < e_width: 
     return 'down_right' 
    else: 
     return 'up_left' 

Answer 4

在改變你的函數一點，你可以返回路線列表用一個簡單的函數定義：

def direction(self, start, end): 
    delta = (end[0] - start[0], end[1] - start[1]) 

    s = [] 

    if delta[1]>0 : s.append("up") 
    elif delta[1]<0 : s.append("down") 

    if delta[0]>0 : s.append("right") 
    elif delta[0]<0 : s.append("left") 

    return s 

下面就包含爲0，1或2名取決於方向旅行。在沒有移動的情況下，該列表將是空的。

如果您特別需要指定格式的返回值，那麼您可以簡單地從列表中提取值並根據需要進行分析。

---

我不完全理解你的順時針方向轉變的需要，但如果那是不能改變的話，我會建議使用一些短/簡單的值作爲佔位符開始，然後用字典來處理與佔位符的兩個替代所需的文字以及順時針轉換。

Answer 5

如果你的原始代碼是正確的，那麼你的映射有些東西我不明白。假設起點是（寬度，高度），如你所說的那樣，或者（x，y），正如我在下面所用的那樣，假設x增加到右邊，y增加（數學中通常的情況，但你的需求似乎是略有不同），這裏有一個簡潔的方式來告訴那裏到底是相對於啓動（使用字符串名稱的設置）：

DIR_STR_LIST = ("up", "down", "right", "left") 

def direction3((sx, sy), (ex, ey)): 
    conds = (sy < ey, ey < sy, sx < ex, ex < sx) 
    istr = DIR_STR_LIST.__iter__() 
    return '_'.join([istr.next() for c in conds if c or not istr.next()]) 

的or not istr.next()推進迭代時c是假的需要。

雖然這是簡潔，它比我想要的有點醜陋。我認爲它可能表現更好，然後原來的代碼（因爲列表解析是快），但這是簡單性更好的情況下。原始代碼快5倍（哎！）。