Как сделать этот код более эффективным?

Question

У меня есть этот фрагмент кода, который перебирает по карте и фильтрует некоторые поля в зависимости от типа отношения, я должен запускать два цикла и чувствовать, что он будет медленным для больших карт. Есть ли способ, которым я могу использовать технику/рефакторинг, чтобы сделать этот код более эффективным.

func getFields(filter map[string]map[string]bool, msg *Message) (fs []Field) { 

    for k, _ := range filter { 
     if relationString(msg) == k { 
      if fieldFilter, ok := filter[k]; ok { 
       for _, f := range msg.Fields { 
        if _, ok := fieldFilter[f.Name]; ok { 
         fs = append(fs, f) 
        } 
       } 
      } 
     } 
    } 
    return 
} 

Answer 1

Вам не нужен внешний цикл как одно улучшения:

func getFields(filter map[string]map[string]bool, msg *Message) (fs []Field) { 
    if fieldFilter, ok := filter[relationString(msg)]; ok { 
    for _, f := range msg.Fields { 
     if _, ok := fieldFilter[f.Name]; ok { 
      fs = append(fs, f) 
     } 
     } 
    } 
    } 

    return 
}