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

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

ただいまの
回答率

89.07%

無駄なonloadを外したい

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,116

mobuocity

score 18

前回解決したのとよく似ているのですが、またわからないところがでてきてしまい、再度投稿させていただきます。

このコードはjQueryで全角入力された英数字を半角変換を参考にしてコードを書きました。
ある人にこのコードを見てもらうと

$(function(){
        $(".test").on("change", function() {
            var str = $(this).val();
            str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
            return String.fromCharCode(s.charCodeAt(0) - 65248);
      });
    $(this).val(str);
  }).change();
});


var str = $(this).val();
 str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
 return String.fromCharCode(s.charCodeAt(0) - 65248);
});


という部分の中に「平時読み込み時に実行されているからonloadを外してね。」と言われました。またスクリプトはできるだけhead内に書き加えてとも言われました。
しかし自分でも調べてみたのですがどこにonloadを実行しているのかわかりません。

改善点をご指摘いただけませんか?

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

+2

var str = $(this).val();
str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
  return String.fromCharCode(s.charCodeAt(0) - 65248);
});

このコードに onload はありません。

$(function(){
  // 処理
});

このコードは onload ではありませんが、DOMContentLoaded に近い動作をします。
前質問で </body> にコードを置くことにしたようですので、このコードは削除して構いません。

Re: mobuocity さん

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/21 07:41

    今一度確認しましたが、同じところを指摘されました。

    キャンセル

  • 2016/09/21 11:27

    その方に「該当コードにonloadはなかった」と伝えてその方の意図を確認してください。
    私達はコードから問題点を洗い出すことは出来ますが、「その方がどういう意図でmobuocityさんに指示を出したのか」は第三者である私達には分かりません。
    yambejp さんが指摘されたように「$(function(){}) を onload と解釈した」のかもしれませんが、mobuocityさんが確認した場所と違いますし、厳密にはこれも onload ではないんですよね…。
    後はmobuocityさんとその方との話し合いで解決すべき問題であると思います。

    キャンセル

  • 2016/09/21 11:41

    onloadがあるとするなら、「mobuocity さんがここにコードを書き移す際に必要な部分を削除して掲載してしまった」という可能性はありますね…。

    キャンセル

checkベストアンサー

+1

// コードが切れていて正しく動かないはず
    $(this).val(st
  }).change(); // 切れているコード類がどうなっているのかによっては、これがonloadで実行される。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/21 07:37

    失礼しました。修正します。

    キャンセル

+1

以下のようにすれば$(function(){});で呼び出さなくても.testに作用します

$(document).on('change','.test', function() {
  console.log(1);
  var str = $(this).val();
  str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
    return String.fromCharCode(s.charCodeAt(0) - 65248);
  });
  $(this).val(str);
});

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/09/21 07:40

    ありがとうございます。head内にいれた場合だとどうなりますか?

    キャンセル

  • 2016/09/21 09:17

    HTML上どこで宣言しても大丈夫です
    $('.test').change()の場合、onload処理をしない場合は
    HTMLの後半(testクラスのオブジェクトがすべて表示されてから)でないと
    オブジェクトをつかめませんが、documentから引っ張れば後から追加された
    オブジェクトでも対象になります

    キャンセル

0

様々な意見、ご指摘ありがとうございます。
自分でエラーログを出しながら、あることに気づき、そのコードを消すとOKいただきました。
今からそのコードと説明をいたします。その前にたくさんの説明不足や返信が遅れたことをここでお詫びします。すみませんでした。

$(function(){
        $(".test").on("change", function() {
            var str = $(this).val();
            str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
            return String.fromCharCode(s.charCodeAt(0) - 65248);
      });
    $(this).val(str);
console.log(0);
  }).change();
});


のコードの下にある

$(function(){
        $(".test").on("change", function() {
            var str = $(this).val();
            str = str.replace( /[A-Za-z0-9-!”#$%&’()=<>,.?_[]{}@^~¥]/g, function(s) {
            return String.fromCharCode(s.charCodeAt(0) - 65248);
      });
    $(this).val(str);
console.log(0);
  });
});


の.change()を消すことが必要だったみたいです。
これがついてるときにページを開きconsole.log(0)見るとフォーマットを選択してない時点で
console.log(0)1 となっておりこれが「平時読み込み時に実行されているからonloadを外してね。」ということでした。 なので.change()を消すとページを読み込んでもconsole.log(0)は出ず、フォーマットを選択し、changeを行うとconsole.log(0) 1 となりました。
ちょっと説明があやふやなのですが、どう説明したらいいのか自分でもわかりません、またちょっと、「ん?」っと思ってるところがあるのでこれで精いっぱいです。 以上です。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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