Q&A
期待する動作はどういうもので、現状の動作はどうなっているのでしょうか？
期待する動作：7秒後に画面遷移
現状の動作：2～3秒でリロードが入る。
以下コメント...
setTimeoutでは7秒を指定。
しかし、2～3秒でリロードが入る。
恐らく、このリロードまでの秒数が定まらないのは、PGを読み込んでいる時間なのだろうか・・・？
色々コード割愛しました
<HTML> <HEAD> <script language="JavaScript"> function myrefresh(_CF_Data) { var LAST = document.frm.LAST.value; document.frm.RANK.value = _CF_Data; if(LAST == 1){ document.frm.method="POST"; document.frm.action="ooo.cfm"; document.frm.submit(); }else{ document.frm.method="POST"; document.frm.action="otitiue.cfm"; document.frm.submit(); } } //1000ミリ秒 * 〇 ＝ 〇秒 setTimeout('myrefresh()', 1000 * 7); </script> </HEAD> <BODY> <CENTER> <cfform name="frm"> <cfoutput> <input type = "hidden" name = "LAST" value = "#LAST#"> <input type = "hidden" name = "RANK" valuye = ""> </cfoutput> <SCRIPT Language="JavaScript">myrefresh('#RANK#');</SCRIPT> </cfform> </CENTER> </BODY> </HTML>
回答3件
0
ベストアンサー
<SCRIPT Language="JavaScript">myrefresh('#RANK#');</SCRIPT>
があるので、setTimeoutを待つまでもなく、即時submitしていませんか？
投稿2018/09/25 05:12
総合スコア116734
0
coldfusionですかね？
まだ、現役なんですね！
と、関係ない話は置いておいて・・・
yamabejpさんの、回答が、すぐ実行されてしまう原因だと思います。
すぐ実行されないよう対策した上で、setTimeoutを生かすなら、下記のような記述で良いと思います。
myrefreshには、引数_CF_DATAがあるので、それを渡してあげる必要があります。
(ソースから該当するデータが読み取れないので、あくまで例としてvalueFor_CF_DATAとしました)
setTimeout
（IE9以前では、動作しないようですが・・・）
setTimeout(myrefresh, 1000 * 7, valueFor_CF_DATA);
投稿2018/09/25 05:25
総合スコア715
0
直接の回答になるかはわかりませんが、どのタイミングで更新しているかを見てみるとよいかと思います。
（実際はページにアクセスしてから7秒経過しているが、目視確認ではページが表示されてから7秒を数えていないか）
setTimeoutの前に現在時刻を保存しておき、myrefresh()の処理終了時にも
現在時刻を取得し、setTimeout前の時刻との差分を計ってみる。
（引数にsetTimeout前の時刻を追加するなどで計ってみてください。）
また、setTimeout('myrefresh()', 1000 * 7);とありますが、setTimeout(myrefresh('xxx'), 1000 * 7);ではないでしょうか？
→ 関数を''で囲む必要はない、引数を受け取る必要がある（受け取らなくてもいいなら別ですが）
[追記]setTimeoutは関数を文字列で渡してもokのようです。（あまり推奨されませんが）
[追記2] 提示した方法が良くない手法だったので訂正します。コールバック関数にするなどの方法を取るべきですね。
投稿2018/09/25 05:05
編集2018/09/25 05:38総合スコア5158
一応、setTimeoutには実行コードを文字列で渡すという機能が存在します。
例えばtextarea等に入力してもらった関数を文字列でぶっこんで実行するなんていう使い方が可能です。
が、eval()と同じでリスクありまくりなんで推奨されない書き方ではあります。
なんと！そんな機能があったんですね。知りませんでした、、、。
昔はコールバック関数を使わなかったので文字列で関数を指定することが多かったですね
いまは何をするにもコールバックするのでそれはそれでプログラマのストレスになっていますが・・・
myrefresh('xxx')の時点で、実行されちゃう気がしますが、どうなんでしょう？？
> yambejp様
なるほど、昔の名残みたいな感じですか。
> Meganezaru様
どうなんでしょうとは何に対してどのような意味でしょうか？
ちなみにコールバックするならこんな感じ？
setTimeout(()=>{myrefresh('xxx')}, 1000 * 7);
setTImeout(myrefresh('xxx'), 1000 * 7);のステップが実行されたときに、
myrefresh('xxx')が実行されて、その戻り値が引数として渡される、と思いました。
実際に実行したわけではないので、「どうなんでしょう？？」と・・・^^;
> yamabejpさん
シンプルで、よいですね！
ああなるほど。yambejp様のコメントで納得しました。
たしかにコールバックにしないと即時実行になってしまいますね、、、失念、、、。
あなたの回答
tips
プレビュー
