VBとC#でのボタンイベントの違い

Question

■WebForms
■OS:Windows7
■VisualStuido2017

ASP.netにてボタンクリック処理をVB.netとC#で記載しています。

Private Sub searchbtn_Click(sender As Object, e As EventArgs) Handles searchbtn.Click
End Sub

<asp:Button  ID="Searchbtn" runat="server" Text="サーチ" />

protected void Searchbtn_Click(object sender, EventArgs e) {
}

<asp:Button  ID="Searchbtn" runat="server" Text="サーチ" OnClick="Searchbtn_Click" />

vbの場合、OnClick="Searchbtn_Click"を記載しなくても良かったのですが、
C#だと記載しないとボタンクリック処理が動作しません。
C#では、aspx側に「OnClick="Searchbtn_Click"」の記述を記載しないと
クリックイベントは処理されないのでしょうか。
記載しなくてもいい場合、どのようにすれば良いか教えて下さい。

宜しくお願いします。

Comments

[退会済みユーザー]

タグの ASP は Teratail ではクラシック ASP を意味します。ASP.NET に訂正してください。

何を作っているか（Web Forms? MVC? その他？）と開発環境（OS. Visual Studio のバージョンなど）ぐらいは書けませんか？

> ボタンクリック処理をVB.netとC#で記載しています。

どういう意味？　Web サイトプロジェクトであるページは VB.NET で、別のページは C# で書いているということ？　Web アプリケーションプロジェクトでそんなことはできませんけど。

[退会済みユーザー]

https://teratail.com/questions/375376 ←これと同じことを聞いているんだよね？

[kumakumatan]

VB.NETで作成したものをC#で作り変える学習をしています。

[kumakumatan]

はい。VBからC#への移行学習をしています。

[退会済みユーザー]

イベントへのイベントハンドラのアタッチの仕方という同じこと質問なのだからスレッドを乱立しないでまとめてください。

[退会済みユーザー]

タグの ASP は Teratail ではクラシック ASP を意味します。ASP.NET に訂正してください。

[退会済みユーザー]

タグの ASP は Teratail ではクラシック ASP を意味します。ASP.NET に訂正してください。

これで 3 回目

Answer 1

VBの方はHandles searchbtn.Click があるのでイベントの捕捉がてきています。
Handles句はVBだけの糖衣構文になります。
C#でaspx側に書きたくなければクラスのコンストラクタに追加してください。

searchbtn.Click += Searchbtn_Click;

これはVBでコンストラクタに

AddHandler searchbtn.Click, AddressOf searchbtn_Click

と書いたのと同じになります。
VBの方はHandles句がそれをやってくれているということです。
追記
コンストラクタでなくても実行時に一度だけ呼ばれる関数の中ならどこに書いてもだいたい問題ないです。
ただし、OnClick="Searchbtn_Click"　と同じ動作を期待するなら関数の呼び出し順に注意が必要です。
ちなみに「一度だけ呼ばれる関数」と限定しているのは複数回イベントを追加すると追加した分だけイベントが処理されるためです。例えばVBでHandles searchbtn.Click, searchbtn.Click とすると2回イベントが処理されるようになります。
コンストラクタなら必ず一度だけ呼ばれるので都合がいいです。

Comments

[kumakumatan]

ありがとうございます。
コンストラクタの部分ですが実際はどの部分に記載するのでしょうか。
namespace C_test
{
public partial class L1 : System.Web.UI.Page
{
}
}
namespace内、public partial class L1内に記載しても、エラーになってしまいます。
宜しくお願いします。

[vann_2921]

コンストラクタは関数ですよ。
コンストラクタについて調べてください。
無いなら自分で書く必要があります。
VBの方だとイベントの追加と同じ要領でInitialize?かNew?があるので追加してください。そのコードを元にC#に書き直せばいいと思います。

[kumakumatan]

protected void Page_Load(object sender, EventArgs e){
}
内に記載してできました。ありがとうございました。
今後とも宜しくお願いします。

[kumakumatan]

public void InitializeComponent() //コンストラクタ
{
Searchbtn.Click += new EventHandler(Searchbtn_Click);
}
と作成して、
protected void Page_Load(object sender, EventArgs e){
InitializeComponent();
}
にて呼び出してできるようになりました。
ありがとうございます。

[退会済みユーザー]

> コンストラクタでなくても実行時に一度だけ呼ばれる関数の中ならどこに書いても大丈夫です。

「どこに書いても」というのは間違いです。タイミングの問題でイベントが発生したときハンドラがまだアタッチされてないということがあり得ます。

正解は .aspx ファイルのコードで質問者さんが書いたとおり、

<asp:Button ID="Searchbtn" runat="server" Text="サーチ" OnClick="Searchbtn_Click" />

です。

[vann_2921]

そこを制御(いつからイベントの捕捉を始めるか)はプログラムを書く人が考えて下さい。
イベント追加する処理自体はどこに書いても正しく追加できます。

[退会済みユーザー]

間違っているところ、

> コンストラクタでなくても実行時に一度だけ呼ばれる関数の中ならどこに書いても大丈夫です。

は直してください。（そもそもあなたが言う「コンストラクタ」というのが何だか不明ですがそこはとりあえず置いとくとして）

[退会済みユーザー]

上のコメントで、

> そもそもあなたが言う「コンストラクタ」というのが何だか不明ですがそこはとりあえず置いとくとして

と書きましたが、そこを明確にしておきます。コメント欄では狭いし画像が貼れないので回答欄に書きます。

Answer 2

vann_2921 さんの回答欄の下の 2021/12/27 10:07 の私のコメントで書いた件です。

vann_2921 さんの言う「コンストラクタ」というのが何だか不明ですが、もしそれが Page のコンストラクタのことだとするとそこでは設定できません。タイミングが早すぎます。

例えば、Button1 を Page に静的に配置して（デザイン画面でツールボックスから Page にドラッグ＆ドロップして）、以下のように .aspx.cs で Page のコンストラクタで Button1.Click イベントにハンドラをアタッチしようとすると、

using System;

namespace WebForms1
{
    public partial class WebForm1 : System.Web.UI.Page
    {
        public WebForm1()
        {
            Button1.Click += Button1_Click;
        }

        private void Button1_Click(object sender, EventArgs e)
        {
            
        }
    }
}

コンストラクタに制御が飛ぶ時点では Button1 は存在しないので NullReferenceException がスローされます。

Page のライフサイクルは以下の記事の通りで、ライフサイクルの各タイミングでその記事の Life-Cycle Events セクションに書いてあるイベントが発生します。

ASP.NET Page Life Cycle Overview　（日本語版は無くなってました）
https://docs.microsoft.com/en-us/previous-versions/ms178472(v=vs.140)

どうしても Button1.Click += Button1_Click; でハンドラをアタッチしたいなら、それらのイベントのハンドラで可能ですが、タイミング的に全てのケースで問題ないか検証が必要です。質問者さんの回答の Load イベントのハンドラ Page_Load では遅い可能性があります。

なので、そんなことで悩むより、正解は .aspx ファイルのコードで質問者さんが書いたとおり、

<asp:Button ID="Searchbtn" runat="server" Text="サーチ" OnClick="Searchbtn_Click" />

とすることです。

Answer 3

public void InitializeComponent() //コンストラクタ
{
Searchbtn.Click += new EventHandler(Searchbtn_Click);
}
と作成して、
protected void Page_Load(object sender, EventArgs e){
InitializeComponent();
}
にて呼び出してできるようになりました。

Comments

[退会済みユーザー]

タイミングの問題でイベントが発生したときハンドラがまだアタッチされてないということがあり得ます。

Page_Load のタイミングで全てのケースで問題ないかは要検証。

なので、そんなことで悩むより、正解は .aspx ファイルのコードで質問者さんが書いたとおり、

<asp:Button ID="Searchbtn" runat="server" Text="サーチ" OnClick="Searchbtn_Click" />

とすることです。