質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

91.00%

  • jQuery

    5824questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

alertが表示されない。

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 543

tkmnusr

score 122

前提・実現したいこと

alertが表示されないので、ご教授いただけませんか?

該当のソースコード

<head>    
<script type="text/javascript" src="jquery-3.2.1.min.js"></script>
</head>

<body>
<form name="check" action="sample1.html" method="post">
  <button type='submit' name='myaction' value='download'>ダウンロード</button>
</form>


<script type="text/javascript">
$(function()
{
  alert("test1");

  $("#myaction").on("click",function(){
  {
    alert("test2");
  });
});
</script>

試したこと

下記のようにコメントアウトするとtest1のalertが表示されます。

<script type="text/javascript">
$(function()
{
  alert("test1");

  //$("#myaction").on("click",function(){
  //{
    //alert("test2");
  //});
});
</script>

追記。

今度はクリック処理でalertが出ないです。今度はカッコの数は合ってそうです。

<script type="text/javascript">
$(function()
{

  $("#myaction").on("click",function()
  {
    alert("test2");
  });
});
</script>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

checkベストアンサー

+2

{ が一つ多いような気がしますが?

$("#myaction").on("click",function(){
  {
    alert("test2");
  });

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/20 20:46

    ご回答ありがとうございます。
    { の数が1個多かったです。
    すみません、再びクリック処理の方でalertが出なくなりました。

    キャンセル

  • 2017/10/20 21:17

    name='myaction' ⇒ id='myaction' に変更してください。

    キャンセル

  • 2017/10/20 21:48

    ご回答ありがとうございます。
    できました。
    id属性は重複してはいけないのですね。
    name属性ってあまり使わないですか?
    PHPで受け取るときもid属性で取得できますか?

    キャンセル

  • 2017/10/20 22:51

    jQuery のセレクタで # の意味をググるなりして調べてください。「PHPで受け取るとき」というのは意味が分かりませんが、少なくともこの質問に書いてある jQuery / JavaScript の話に限っては何の関係もないと思うのですが?

    キャンセル

  • 2017/10/20 22:57

    name 属性はもちろん使いますよ。form を送信するときを考えてみてください。今回の話は、それとは関係なく jQuery のセレクタで # がどういう意味を持つかです。何かと混同してませんか?

    キャンセル

+1

字下げスタイルが混同しているのがミスの原因でしょうね。

function hoge(){
    //処理
}


これはK&Rスタイルというそうです。※

私はこのスタイルが好きです。

他に

function hoge()
{
    //処理
}


というBSDスタイルなるものもあるそうですね。

あなたのコードはどちらでしょうか?今一度確認してください。

そうすればSurferOnWww様がおっしゃるように、括弧の数がおかしいことに気づくはずです。

どちらかのスタイルに統一しましょう。

※関数の{}はBSDスタイルみたいにするそうなので、正確には違うのかもしれませんが。。(^ ^;

今更ながら、コメントに対し追記:

SurferOnWww様のおっしゃるようにname="myaction"ではなくid="myaction"にする必要がありますね。またはbutton要素が一つしかないならば、下記のように$("button")を指定することでも設定できますが、classかidで指定する方が無難でしょう。

$(function(){
  alert("test1");

  $("button").on("click",function(){
    alert("test2");
  });
});


○備考

  • $("#hoge") -> id="hoge"となった要素を取得
  • $(".hoge") -> class="hoge"となった要素を取得
  • $("hoge") -> hoge要素を取得

PHPについては私は全然なのでなんとも言えませんが、フォームを送るときには基本的に値を送信したいフォーム部品にはname属性が必要だったと記憶しています。

javascriptでAjax等を使ったりして値を送信する場合はその限りではなかったはずです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/10/20 20:46 編集

    ご回答ありがとうございます。
    { の数が1個多かったです。
    すみません、再びクリック処理の方でalertが出なくなりました。

    キャンセル

  • 2017/10/20 21:54

    無事解決してよかったです。一応追記しました。

    キャンセル

  • 2017/10/20 21:58

    ご回答ありがとうございます。
    PHP側の取得ではname属性が必要となると、id属性とname属性の2つが必要ということですね。
    ボタンが2つ以上の場合、idは重複しないようにして、name属性は重複してもいいということですね。

    キャンセル

  • 2017/10/20 22:07 編集

    ラジオボタン等の複数の部品からなるものはグループ化のために同一のname属性を指定しますが、基本的にname属性も重複してはいけなかったような。。?
    断言はできませんが分けた方が無難だと思います。

    またjavascriptの方でもdocument.getElementsByName("hoge")[0];のように書けばname属性のみで要素を取得できますが、idやclassを指定する方がいいと思います。

    キャンセル

  • 2017/10/20 22:11

    ご回答ありがとうございます。
    グループ化以外では、name属性も分けたほうが無難なんですね。
    $("#hoge")の書き方で取得したいので、idを使っていこうと思います。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 91.00%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • jQuery

    5824questions

    jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。