C# と VB.NET の質問掲示板

ボタンを押すと計算を行うプログラムを作成していますが、
コンボボックスの選択で計算を行う関数を切替させることをしたいです。
下記のようなコードでコンボボックスで選択されているインデックスに応じた
繰り返し計算を行わせていますが、どうしてもfor文の中にif文を入れてしまうと
処理時間が大きくなってしまいます。



public partial class Form1 : Form
{

   double function1(double x)
   {
     return x*x;
   }

   double function2(double x)
   {
     return x*x*x;
   }
   

・・・

private void button6_Click(object sender, EventArgs e)
{
   for(int a=0; a<1000000; a++)
   {
      if(comboBox1.SelectedIndex==1)
      {
          y = function1(x);
      }
      else
      {
          y = function2(x);
      }
   }
}


for文の中にif文を入れ込まずに、
前段のところでコンボボックスで選択した関数を使う、
のようなことはできないのでしょうか？


public partial class Form1 : Form
{

      if(comboBox1.SelectedIndex==1)
      {
         double function(double x)
         {
           return x*x;
         }
      }
      else
      {
         double function(double x)
         {
           return x*x*x;
         }
      }

素人的質問で申し訳ないです。

C/C++ では関数ポインタを使う案件だね。 C# だと delegate だろうか。

public delegate double Funcd(double d);
void TestFunc()
{
    Funcd f = new Funcd(function1);
    Debug.WriteLine(f(1.0));
}

提示ソースだと Math.Pow() を使うだけのような気のせいがするけど・・・
イベントハンドラ内にて乗数を記憶すればよい

double function2(double x)
{
    return x * x;
}

double function3(double x)
{
    return x * x * x;
}

みたいにメソッドを定義して、

Func<double, double> function =
    (comboBox1.SelectedIndex==1) ?
    function2 :
    function3;

あとは、
     for(int x=0; x<1000000; x++)
     {
         function(x);
     }
かな。

Funcっていうのは、あらかじめ用意されているdelegateです。
  >  public delegate TResult Func<in T, out TResult>(T arg);
これ使えば、わざわざ定義しなくてもいいので楽ちんです。

■No86358 (ぶなっぷ さん) に返信
> Func<double, double> function =
>     (comboBox1.SelectedIndex==1) ?
>     function2 :
>     function3;
> 
> あとは、
>      for(int x=0; x<1000000; x++)
>      {
>          function(x);
>      }
> かな。

ぶなっぷさん、これだとfor文の中にif文が入ったままではないですか？


Func<double, double> function1 => x =>
{
    return x*x;
};
		
Func<double, double> function2 => x =>
{
    return x*x*x;
};

と定義して

    var function =
        (comboBox1.SelectedIndex==1) ?
            function1 :
            function2;

     for(int x=0; x<1000000; x++)
     {
         function(x);
     }

※日本語の間違いを修正しました。

この質問に興味があったので、横から失礼します。

ループで毎回comboBox1.SelectedIndexを参照するのは
もしかしたらコストが累積しそうに思いますが、
単純なif文でもそれなりにコストがかかるのでしょうか？

例えば、(comboBox1.SelectedIndex==1)の条件を外だしして、
以下のように（あるいは三項演算）しても、
やっぱり遅くなるものなのでしょうか？

private void button6_Click(object sender, EventArgs e)
{
   bool f = (comboBox1.SelectedIndex==1);
   for(int a=0; a<1000000; a++)
   {
      if(f)
      {
          y = function1(x);
      }
      else
      {
          y = function2(x);
      }
   }
}

遅くなってすいません。

furuさん
  いえいえ、お互い切磋琢磨しましょう。

魔界の仮面弁士 さん
  コードを書いて確認してみました。
  確かに、暗黙のキャストしてくれないんですね。
  動かして確認していないのがもろばれですね（笑）