C# textBoxに変数を表示させたい

Question

現在ボタンを押すと、textBoxに計算された答えを表示するアプリを作成しているのですが、表示される答えがいつも「０」になってしまいます。
アプリとしての機能はBottonをclickするとtextBoxに変数が表示されるという簡単なものなのです。しかし、textboxに「0,0038」と表示させたいところが毎回「０」となってしまします。c#にグローバル変数はないということがわかりましたので、表示させたい変数ｓはグローバル変数のような形で使っています。
コードは以下のようになっています。
ご教授していただければありがたいです。
よろしくお願い致します。

visual studio Ver.2019　　を使用しています。

【コード】

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace keisann_tamesi_aplication
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();


        }

          static double s ;
         
            private static void main(string[] args)


            {

                var Val = new int[] { 20,21,24,324,345,344,23,22,21,327,344,322,21,21,21,329,344,321,21,21,21,323,345,344,21,21,21,324,344,344,21,21,21,21,
                326,344,345,21,21,21,21,344,322,321,324,21,21,21,21,21 };

                //Valの隣り合う要素の値の差を要素値とする配列 A を作る
                var A = new int[Val.Length - 1];

                int[] C = new int[101];
                for (int i = 0; i < A.Length; ++i)
                {
                    A[i] = Val[i + 1] - Val[i];
                }
                //配列Aの要素の中から295より大きいものを配列retに代入していく
                //   int[] ret = Array.FindAll(A, num => num > 295);

                int n = 0;
                for (int i = 0; i < A.Length; i++)
                {

                    if (A[i] >= 295)
                    {
                        C[n] = i; n = n + 1;
                    }
                }


                //配列 ret の隣り合う要素の値の差を要素値とする配列 hip を作る
                var hip = new int[C.Length - 1];
                for (int i = 0; i < hip.Length; ++i)
                {

                    hip[i] = C[i + 1] - C[i];
                }

                //配列 hip の要素から奇数番目の要素を取り出す
　　　　　　　　int[] B = new int[3];
           　　 double l = 0;
                for (int i = 0; i < B.Length; ++i)
                {
                    B[i] = hip[i * 2];
                }

                for (int i = 0; i < B.Length; ++i)
                {
                    l += B[i];
                }

            s = l / 50 * 0.01;

                

        }

            public void button1_Click(object sender, EventArgs e)
            {
            this.Invoke(new MethodInvoker(() => textBox1.Text = Convert.ToString(s)));
        }

        }
        

}

``` 　
【試したこと】
上の計算自体が間違っているのかと考え、いったんコンソールアプリケーションのほうで動かしてみると問題なくいきました。

Answer

端的に問題を指摘すると、計算結果を格納しようとしている変数 sが、static変数としてしてあるため、変更されず、初期値の0が表示されているということです。static変数を紹介する例みたいになっています。

コードをスッキリさせると以下のようになります。sが変更されずに0のままであることが理解できると思います。

C#
1
2using System;
3using System.Windows.Forms;
4
5namespace ShowDecimals
6{
7    public partial class Form1 : Form
8    {
9        public Form1()
10        {
11            InitializeComponent();
12        }
13
14        static double s;
15        private static void main(string[] args)
16        {
17            s = 0.0038;
18        }
19
20        public void button1_Click(object sender, EventArgs e)
21        {
22            textBox1.Text = Convert.ToString(s);
23        }
24    }
25}
26

追記です。

C#では、変数は、クラスメンバーとして扱わないと、スコープの問題で苦労するので、使い回すプロジェクトであれば、現実的には、クラス設計をし直すことになります。
一方、Cなどの関数型のプログラムを、C#で処理したい、その場でしか使わないプログラムなど、極力手を加えたくない場合があります。

その場合は、作法的にはよろしくないと思いますが、単純に、クリックイベントにすべてを格納する方法が一番ラクです。

C#
1using System;
2using System.Windows.Forms;
3
4namespace ShowDecimals
5{
6    public partial class Form1 : Form
7    {
8        public Form1()
9        {
10            InitializeComponent();
11        }
12
13        public void button1_Click(object sender, EventArgs e)
14        {
15            double s;
16            var Val = new int[] { 20,21,24,324,345,344,23,22,21,327,344,322,21,21,21,329,344,321,21,21,21,323,345,344,21,21,21,324,344,344,21,21,21,21,
17                326,344,345,21,21,21,21,344,322,321,324,21,21,21,21,21 };
18
19            //Valの隣り合う要素の値の差を要素値とする配列 A を作る
20            var A = new int[Val.Length - 1];
21
22            int[] C = new int[101];
23            for (int i = 0; i < A.Length; ++i)
24            {
25                A[i] = Val[i + 1] - Val[i];
26            }
27            //配列Aの要素の中から295より大きいものを配列retに代入していく
28            //   int[] ret = Array.FindAll(A, num => num > 295);
29
30            int n = 0;
31            for (int i = 0; i < A.Length; i++)
32            {
33
34                if (A[i] >= 295)
35                {
36                    C[n] = i; n = n + 1;
37                }
38            }
39
40
41            //配列 ret の隣り合う要素の値の差を要素値とする配列 hip を作る
42            var hip = new int[C.Length - 1];
43            for (int i = 0; i < hip.Length; ++i)
44            {
45
46                hip[i] = C[i + 1] - C[i];
47            }
48
49            //配列 hip の要素から奇数番目の要素を取り出す
50            int[] B = new int[3];
51            double l = 0;
52            for (int i = 0; i < B.Length; ++i)
53            {
54                B[i] = hip[i * 2];
55            }
56
57            for (int i = 0; i < B.Length; ++i)
58            {
59                l += B[i];
60            }
61
62            s = l / 50 * 0.01;
63
64            textBox1.Text = Convert.ToString(s);
65        }
66    }
67
68}
69

関連した質問