为何无法在方法中使用ComboBox选中索引？——库纳转欧元转换器

一、核心问题：变量作用域限制 #

• first和second是comboBox1_SelectedIndexChanged、comboBox2_SelectedIndexChanged方法内的局部变量，作用域仅局限于各自方法内部，外部的Currencies.Convert方法根本无法访问到这两个变量。
• 你代码里写的Converter.comboBox1.first完全不符合C#语法——控件对象本身没有first这个属性，也不能通过这种方式访问局部变量。

二、其他明显代码问题 #

• 类型错误：SelectedIndex是整数类型（int），用double接收完全没必要，应该声明为int first = comboBox1.SelectedIndex;。
• 拼写错误：第二个分支里的comboBox2.seocnd是笔误，正确写法是second。
• 未定义标识符：HRK、EURO这些没有定义，你应该把它们对应成ComboBox的索引值（比如假设HRK是索引0，EURO是索引1），或者用ComboBox的SelectedItem获取文本后判断。
• 逻辑冗余：两个分支的计算逻辑完全一样，应该合并；而且currencies参数看起来没用到，转换逻辑只需要金额和货币类型，不需要传入数组。

三、修正后的代码示例 #

1. 在Form类中定义全局变量存储选中的货币索引 #

// 在Form类的字段区域声明（不要写在方法内部）
private int _fromCurrencyIndex;
private int _toCurrencyIndex;

// 事件方法中给全局变量赋值
private void comboBox1_SelectedIndexChanged(object sender, EventArgs e)
{
    _fromCurrencyIndex = comboBox1.SelectedIndex;
}

private void comboBox2_SelectedIndexChanged(object sender, EventArgs e)
{
    _toCurrencyIndex = comboBox2.SelectedIndex;
}

2. 修正Currencies类的转换方法 #

internal class Currencies
{
    // 定义常量对应货币索引，方便后续维护
    private const int HRK_INDEX = 0;
    private const int EURO_INDEX = 1;
    private const double HRK_TO_EURO_RATE = 7.4;

    public double Convert(int amount, int fromCurrency, int toCurrency)
    {
        if (fromCurrency == HRK_INDEX && toCurrency == EURO_INDEX)
        {
            return amount / HRK_TO_EURO_RATE;
        }
        else if (fromCurrency == EURO_INDEX && toCurrency == HRK_INDEX)
        {
            return amount * HRK_TO_EURO_RATE;
        }
        else
        {
            throw new NotSupportedException("不支持当前货币转换组合");
        }
    }
}

3. 在Form中调用转换并显示结果（比如通过按钮触发） #

private void btnConvert_Click(object sender, EventArgs e)
{
    // 假设输入金额的控件是txtAmount，显示结果的控件是lblResult
    if (int.TryParse(txtAmount.Text, out int amount))
    {
        var currencyConverter = new Currencies();
        try
        {
            double result = currencyConverter.Convert(amount, _fromCurrencyIndex, _toCurrencyIndex);
            lblResult.Text = $"转换结果：{result:F2}";
        }
        catch (NotSupportedException ex)
        {
            lblResult.Text = ex.Message;
        }
    }
    else
    {
        lblResult.Text = "请输入有效的金额";
    }
}

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