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)));
        }

        }
        

}

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

Comments

[退会済みユーザー]

コードは ``` と ``` で囲ってインデントされるようにするとか、開発環境を書くとか読んでもらえる努力をしましょう。

[dodox86]

> private static void main(string[] args)
が呼ばれている様子もないので、要望どおり動くはずが無いのですが、、、動いたというコンソールアプリケーションの断片かもしれませんが、コンソールアプリケーションの場合、通常はmainではなくMainなので、それも謎です。とりあえず基本的なところから学びなおした方が良いと思います。そうでないと回答をいただいても理解できません。

[Visual Studio で C# を使用して Windows フォーム アプリを作成する]
https://docs.microsoft.com/ja-jp/visualstudio/ide/create-csharp-winform-visual-studio?view=vs-2019

[退会済みユーザー]

返信ありがとうございます。質問内容自体がわかりにくくなってしまい申し訳ございません。ご指摘がありました"main"に関しましては、最初”Main”とうっていたのですがエラーが出てしまい小文字に変更するとそのエラーが解消されたのでこのような形となっています。

[退会済みユーザー]

「コードは ``` と ``` で囲って」さえもやる気はない？　何なんだろうね・・・　ゆとり教育の成果かな？

[退会済みユーザー]

申し訳ございません。修正し忘れていました。

[退会済みユーザー]

コードを拝見しましたが、はっきり言ってメチャクチャで、少々修正すれば動くというものではなく、議論できるレベルに達してないです。また、

> "main"に関しましては、最初”Main”とうっていたのですがエラーが出てしまい小文字に変更するとそのエラーが解消されたのでこのような形となっています。

という、訳も分からずテキトーにいじってそれを他人に見てくれと投げてくるやり方も考え直さないと何ともならないと思います。

ここのような文章でやり取りするサイトで質問して課題を解決するには、質問者には最低限話が通じるレベルの基本的な知識と、自分の課題を他人が読んで理解できるように文書化するスキルが必要です。

そこを考えて、質問を書き直すことをお勧めします。

あと、これを読むこともお勧めします⇒ https://teratail.com/help/question-tips

Answer 1

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

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

using System;
using System.Windows.Forms;

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

        static double s;
        private static void main(string[] args)
        {
            s = 0.0038;
        }

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

追記です。

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

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

using System;
using System.Windows.Forms;

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

        public void button1_Click(object sender, EventArgs e)
        {
            double s;
            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;

            textBox1.Text = Convert.ToString(s);
        }
    }

}

Comments

[TN8001]

質問者が退会しているうえこんなことする人はあんまりいないのでほっておくつもりでしたが、ちょっと勘違いされているようなのでコメントします。

> static変数としてしてあるため、変更されず、初期値の0が表示されているということです。

sがstaticかどうかは無関係です。
問題は呼ばれてると思い込んでいるmainが呼ばれていないことです。

static void Mainは、WinFormでは通常Program.csにあります。
もし質問コードのMainが呼ばれるのであれば、Form1は表示されません（mainも小文字ですし）

> C#では、変数は、クラスメンバーとして扱わないと

（インスタンス）フィールド（インスタンス変数・メンバ変数）のことだと思いますが、「クラスメンバー」とするとフィールド・メソッド・プロパティ・イベント等々（静的かどうかを問わず）すべて含まれるように感じます。
[Classes - C# language specification | Microsoft Docs](https://docs.microsoft.com/ja-jp/dotnet/csharp/language-reference/language-specification/classes#static-and-instance-members

> Cなどの関数型のプログラム

手続き型かなんかの間違いでは？
[関数型プログラミング - Wikipedia](https://ja.wikipedia.org/wiki/%E9%96%A2%E6%95%B0%E5%9E%8B%E3%83%97%E3%83%AD%E3%82%B0%E3%83%A9%E3%83%9F%E3%83%B3%E3%82%B0

[kkokuke]

ご指摘ありがとうございます。勉強になります。