最大和最小频率都返回相同的值

Question

我已经创建了一个使用wxFormBuilder的GUI，它应该允许用户在列表中输入“业务访问者”的名称，然后单击两个按钮之一以返回最多频繁和最不频繁的业务访客。这里看一下GUI：http://imgur.com/XJnvo0U

不幸的是，最大和最小按钮都抛出相同的值（大多数）。我认为这与频率如何加权输入有关，也许与clkFindMost和clkFindLeast事件的定义重叠有关。任何帮助将不胜感激。

**编辑：我非常新的Python，和学习曲线陡峭

import wx 
import myLoopGUI 
import commands 

class MyLoopFrame(myLoopGUI.MyFrame1): 
    def __init__(self, parent): 
     myLoopGUI.MyFrame1.__init__(self, parent) 

    def clkAddData(self,parent): 
     if len(self.txtAddData.Value) != 0: 
      try: 
       myname = str(self.txtAddData.Value) 
       self.listMyData.Append(str(myname)) 
      except: 
       wx.MessageBox("This has to be a name!")    
     else: 
      wx.MessageBox("This can't be empty") 




    def clkFindMost(self, parent): 
     name_list = set(self.listMyData.GetStrings()) 
     unique_names = set(name_list) 
     frequencies = {} 
     for name in unique_names: 
      if frequencies.get(name): 
       frequencies[name] += 1 
      else: 
       frequencies[name] = 0 

     counts = list(frequencies.values()) 
     names = list(frequencies.keys()) 
     max_count_index = counts.index(max(counts)) 
     min_count_index = counts.index(min(counts)) 
     most_frequent = names[max_count_index] 
     least_frequent = names[min_count_index] 
     self.txtResults.Value = most_frequent 



    def clkFindLeast(self, parent): 
     name_list = set(self.listMyData.GetStrings()) 
     unique_names = set(name_list) 
     frequencies = {} 
     for name in unique_names: 
      if frequencies.get(name): 
       frequencies[name] += 1 
      else: 
       frequencies[name] = 0 

     counts = list(frequencies.values()) 
     names = list(frequencies.keys()) 
     max_count_index = counts.index(max(counts)) 
     min_count_index = counts.index(min(counts)) 
     most_frequent = names[max_count_index] 
     least_frequent = names[min_count_index] 
     self.txtResults.Value = least_frequent 

    def clkClear (self, parent): 
     self.txtResults.SetValue("") 

#Needed to ensure the program runs   
myApp = wx.App(False) 
myFrame = MyLoopFrame(None) 
myFrame.Show() 
myApp.MainLoop() 

Answer 1

编辑：在我原来的答案，我专注于错误的问题。

首先，迭代一组唯一名称来确定频率是没有意义的。而是应该将遍历所有的名字，如：

name_list = self.listMyData.GetStrings() # presuming this is an iterable 
    frequencies = {} 
    for name in name_list: 
     if frequencies.get(name): 
      frequencies[name] += 1 
     else: 
      frequencies[name] = 0 

即使你的任务不允许它，我会离开这个对于熏陶：
你想做的事在collections.defaultdict提供什么样的一流解决方案如果它不存在则创建一个关键字，如果它存在则创建一个关键字。由于collections.defaultdict的子类dict您以后在代码中使用的dict方法仍然有效。

import collections as co # at top of script 
# ... 
    name_list = self.listMyData.GetStrings() # presuming this is an iterable 
    frequencies = co.defaultdict(int) 
    for name in name_list: 
     frequencies[name] += 1