蒙特卡洛可靠性计算程序中RadioButton的s_val取值异常问题

核心错误原因 #

1. RadioButton的command参数绑定错误：创建按钮时写了command=self.get_val1()，这会在按钮初始化时直接执行get_val1函数，而不是在点击按钮时触发。此时用户还未选择任何选项，self.s_value.get()返回默认值0，之后点击按钮也不会再执行函数（因为绑定的是函数执行后的返回值None）。正确写法是去掉括号，绑定函数对象command=self.get_val1。
2. 全局变量未初始化：代码中仅注释了s_val的用途，但未在全局作用域定义，导致第一次调用get_val1时才创建该变量，初始值为0。
3. 逻辑运算符使用错误：条件判断中误用位运算符|、&，应使用逻辑运算符or、and。
4. 数值判断逻辑缺陷：isdigit()无法识别带小数点的数值，会导致合法的小数输入被判定为非法。
5. 计算函数的语法错误：cal_res函数中存在分支判断错误（将type2写成type1），以及生成数组时多余的,2导致变量类型错误。

修复后的完整代码 #

import tkinter as tk
import tkinter.messagebox as messagebox
import numpy as np

# 全局变量初始化
s_val = 0
r_val = 0
para1_1 = 0.0
para1_2 = 0.0
para2_1 = 0.0
para2_2 = 0.0

class basedesk:
    def __init__(self, master):
        self.root = master
        self.root.title('可靠性计算')
        self.root.geometry('400x300')
        initface1(self.root)

class initface1:
    def get_val1(self):
        global s_val
        s_val = self.s_value.get()

    def __init__(self, master):
        self.master = master
        self.initface1 = tk.Frame(self.master)
        self.initface1.pack()
        reminder = tk.Label(self.initface1, text='请选择强度分布类型', width=50, height=10)
        reminder.pack()
        
        self.s_value = tk.IntVar()
        self.radio()
        tk.Button(self.initface1, text='确定', command=self.change).pack()

    def radio(self):
        # 修复command参数，去掉括号，绑定函数对象
        radio_bt1 = tk.Radiobutton(self.initface1, text='正态分布', variable=self.s_value, value=1, command=self.get_val1)
        radio_bt2 = tk.Radiobutton(self.initface1, text='对数正态分布', variable=self.s_value, value=2, command=self.get_val1)
        radio_bt3 = tk.Radiobutton(self.initface1, text='指数分布', variable=self.s_value, value=3, command=self.get_val1)
        radio_bt4 = tk.Radiobutton(self.initface1, text='威布尔分布', variable=self.s_value, value=4, command=self.get_val1)
        radio_bt1.pack()
        radio_bt2.pack()
        radio_bt3.pack()
        radio_bt4.pack()

    def change(self):
        self.initface1.destroy()
        face1(self.master)

class face1:
    def __init__(self, master):
        self.master = master
        self.face1 = tk.Frame(self.master)
        self.face1.pack()
        global s_val
        print(f"当前选择的强度分布类型：{s_val}")
        
        # 修复逻辑运算符，用or替代|
        if s_val == 1 or s_val == 2:
            tk.Label(self.face1, text='请输入该分布的两个参数', width=50, height=10).pack()
            tk.Label(self.face1, text='均值').pack()
            self.parameter1 = tk.Entry(self.face1, relief='groove', width=20)
            self.parameter1.pack()
            tk.Label(self.face1, text='标准差').pack()
            self.parameter2 = tk.Entry(self.face1, relief='groove', width=20)
            self.parameter2.pack()
        else:
            tk.Label(self.face1, text='请输入该分布的参数', width=50, height=10).pack()
            self.parameter1 = tk.Entry(self.face1, relief='groove', width=20)
            self.parameter1.pack()

        tk.Label(self.face1, text='注意：仅允许输入数字').pack()
        tk.Button(self.face1, text='确认', command=self.com).pack()

    def is_number(self, s):
        # 替换isdigit，支持小数判断
        try:
            float(s)
            return True
        except ValueError:
            return False

    def com(self):
        global para1_1, para1_2, s_val
        if s_val == 1 or s_val == 2:
            # 修复逻辑运算符，用and替代&
            if self.is_number(self.parameter1.get()) and self.is_number(self.parameter2.get()):
                para1_1 = float(self.parameter1.get())
                para1_2 = float(self.parameter2.get())
                self.change()
            else:
                messagebox.showwarning('警告', '请输入有效的数字')
        else:
            if self.is_number(self.parameter1.get()):
                para1_1 = float(self.parameter1.get())
                para1_2 = 0.0
                self.change()
            else:
                messagebox.showwarning('警告', '请输入有效的数字')

    def change(self):
        self.face1.destroy()
        initface2(self.master)

class initface2:
    def get_val2(self):
        global r_val
        r_val = self.r_value.get()

    def __init__(self, master):
        self.master = master
        self.initface2 = tk.Frame(self.master)
        self.initface2.pack()
        reminder = tk.Label(self.initface2, text='请选择应力分布类型', width=50, height=10)
        reminder.pack()
        
        self.r_value = tk.IntVar()
        self.radio()
        tk.Button(self.initface2, text='确认', command=self.change).pack()

    def radio(self):
        # 修复command参数
        radio_bt1 = tk.Radiobutton(self.initface2, text='正态分布', variable=self.r_value, value=1, command=self.get_val2)
        radio_bt2 = tk.Radiobutton(self.initface2, text='对数正态分布', variable=self.r_value, value=2, command=self.get_val2)
        radio_bt3 = tk.Radiobutton(self.initface2, text='指数分布', variable=self.r_value, value=3, command=self.get_val2)
        radio_bt4 = tk.Radiobutton(self.initface2, text='威布尔分布', variable=self.r_value, value=4, command=self.get_val2)
        radio_bt1.pack()
        radio_bt2.pack()
        radio_bt3.pack()
        radio_bt4.pack()

    def change(self):
        self.initface2.destroy()
        face2(self.master)

class face2:
    def __init__(self, master):
        self.master = master
        self.face2 = tk.Frame(self.master)
        self.face2.pack()
        global r_val
        
        if r_val == 1 or r_val == 2:
            tk.Label(self.face2, text='请输入该分布的两个参数', width=50, height=10).pack()
            tk.Label(self.face2, text='均值').pack()
            self.parameter3 = tk.Entry(self.face2, relief='groove', width=20)
            self.parameter3.pack()
            tk.Label(self.face2, text='标准差').pack()
            self.parameter4 = tk.Entry(self.face2, relief='groove', width=20)
            self.parameter4.pack()
        else:
            tk.Label(self.face2, text='请输入该分布的参数', width=50, height=10).pack()
            self.parameter3 = tk.Entry(self.face2, relief='groove', width=20)
            self.parameter3.pack()

        tk.Label(self.face2, text='注意：仅允许输入数字').pack()
        tk.Button(self.face2, text='确认', command=self.com).pack()

    def is_number(self, s):
        try:
            float(s)
            return True
        except ValueError:
            return False

    def com(self):
        global para2_1, para2_2, r_val
        if r_val == 1 or r_val == 2:
            if self.is_number(self.parameter3.get()) and self.is_number(self.parameter4.get()):
                para2_1 = float(self.parameter3.get())
                para2_2 = float(self.parameter4.get())
                self.cal_res()
            else:
                messagebox.showwarning('警告', '请输入有效的数字')
        else:
            if self.is_number(self.parameter3.get()):
                para2_1 = float(self.parameter3.get())
                para2_2 = 0.0
                self.cal_res()
            else:
                messagebox.showwarning('警告', '请输入有效的数字')

    def cal_res(self):
        global s_val, r_val, para1_1, para1_2, para2_1, para2_2
        # 修复数组生成的语法错误，以及分支判断错误
        if s_val == 1:
            x1 = np.random.normal(para1_1, para1_2, size=1000)
        elif s_val == 2:
            x1 = np.random.lognormal(para1_1, para1_2, size=1000)
        elif s_val == 3:
            x1 = np.random.exponential(para1_1, size=1000)
        else:
            x1 = np.random.weibull(para1_1, size=1000)

        if r_val == 1:
            x2 = np.random.normal(para2_1, para2_2, size=1000)
        elif r_val == 2:
            x2 = np.random.lognormal(para2_1, para2_2, size=1000)
        elif r_val == 3:
            x2 = np.random.exponential(para2_1, size=1000)
        else:
            x2 = np.random.weibull(para2_1, size=1000)

        n1 = sum(x1 > x2)
        result = n1 / 1000
        messagebox.showinfo('计算结果', f'可靠性结果为：{result:.4f}')

if __name__ == '__main__':
    root = tk.Tk()
    basedesk(root)
    root.mainloop()

额外优化说明 #

• 将from numpy import *改为import numpy as np，避免命名空间污染。
• 新增is_number函数替代isdigit()，支持小数输入判断。
• 简化cal_res中的计数逻辑，用sum(x1 > x2)替代循环计数，更高效。
• 全局变量统一初始化，避免未定义的问题。
• 将界面文本改为中文，提升易用性。

内容的提问来源于stack exchange，提问作者mayayang