無駄なonloadを外したい

Question

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

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

$(function(){
		$(".test").on("change", function() {
			var str = $(this).val();
			str = str.replace( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/g, function(s) {
			return String.fromCharCode(s.charCodeAt(0) - 65248);
      });
    $(this).val(str);
  }).change();
});

の

 var str = $(this).val();
 str = str.replace( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/g, function(s) {
 return String.fromCharCode(s.charCodeAt(0) - 65248);
});

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

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

Answer 1

var str = $(this).val();
str = str.replace( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/g, function(s) {
  return String.fromCharCode(s.charCodeAt(0) - 65248);
});

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

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

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

Re: mobuocity さん

Comments

[mobuocity]

ありがとうございます。　前回は</body>にコードを置きましたが、今回は上記に書くように指示され　上記に書きました。説明不足でしたすみません。

[think49]

とりあえず、onload 関係の処理は全くありません。
「指示を出した方の勘違い」か「mobuocity さんが指示内容の解釈を間違っている」のいずれかだと思います。
指示を出した方にもう一度、指示内容を確認することをお勧めします。

[mobuocity]

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

[think49]

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

[think49]

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

Answer 2

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

Comments

[mobuocity]

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

Answer 3

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

$(document).on('change','.test', function() {
  console.log(1);
  var str = $(this).val();
  str = str.replace( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/g, function(s) {
    return String.fromCharCode(s.charCodeAt(0) - 65248);
  });
  $(this).val(str);
});

Comments

[mobuocity]

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

[yambejp]

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

Answer 4

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

$(function(){
        $(".test").on("change", function() {
            var str = $(this).val();
            str = str.replace( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/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( /[Ａ-Ｚａ-ｚ０-９－！”＃＄％＆’（）＝＜＞，．？＿［］｛｝＠＾～￥]/g, function(s) {
            return String.fromCharCode(s.charCodeAt(0) - 65248);
      });
    $(this).val(str);
console.log(0);
  });
});

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