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

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

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

Q&A

解決済

1回答

2157閲覧

javascriptでsetTimeoutがclearTimeoutで止まらないです。

jam79iy

総合スコア6

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

0グッド

0クリップ

投稿2021/01/10 07:39

編集2021/01/10 07:41

前提・実現したいこと

Discordでゲームのスタミナがマックスなことを教えてくれるbotをjavascriptで作成してみようと思い作り始めて見たのですが、完成したはいいもののキャンセル機能がなく不便に感じキャンセル機能もつけてみようと思いつけてみようと思いました。
そこでタイマーをsetTimeoutで時間を計算してメッセージを送るようにしてたので、clearTimeoutを使いキャンセルしようと思い書いてみたのですが、止まらず通知してしまいます。
なぜ止まらないのでしょうか?

読みにくいコードですがご教授いただけると幸いです。

該当のソースコード

javacript

1const Discord = require('discord.js'); 2 3const { prefix, token } = require('./config.json'); 4const client = new Discord.Client(); 5 6client.on('ready', () => { 7 console.log(`${client.user.username} でログインしています。`); 8}) 9 10client.on('message', async msg => { 11 var jushimax=160*8; 12 var jushi=0,nokorih=0,nokorim=0; 13 var time=((jushimax-(jushi*8))-80)*60*1000; 14 var jushiTimer 15 16 if(!msg.content.startsWith(prefix) || msg.author.bot) return; 17 18 const args=msg.content.slice(prefix.length).split(' '); 19 const command=args.shift().toLowerCase(); 20 21 if (msg.content === `${prefix}hello`) { 22 msg.channel.send(`${msg.author}!\nこんにちは、よろしくね。`); 23 }else if(msg.content === `${prefix}server`) { 24 msg.channel.send(`サーバー名:${msg.guild.name}\n人数:${msg.guild.memberCount}`); 25 } 26//ここから教えてくれるコマンドのコードです。 27 else if(command === 'jushi'){ 28 if (!args.length){ 29 return msg.channel.send(`${msg.author}!、樹脂を !jushi 160 のように書いてください。`); 30 } 31 32 jushi=args[0]; 33 nokorih=Math.floor((jushimax-(jushi*8))/60); 34 nokorim=(jushimax-(jushi*8))%60; 35 36 37 if(0<=args[0] && args[0]<=160){ 38 msg.channel.send(`今、${msg.author}の樹脂は${args[0]}です。\n残り${nokorih}時間${nokorim}分で樹脂が全回復です。`); 39 }else{ 40 msg.channel.send('樹脂の入力は0~160まででお願いします…'); 41 } 42 43 if(args[0]<150){ 44 jushiTimer=setTimeout(function(){ 45 msg.channel.send(`今、${msg.author}の樹脂は150です。\n残り1時間20分で樹脂が全回復してしまいます。`); 46 },10000) 47 } 48 }else if(msg.content === `${prefix}timerc`){ 49 clearTimeout(jushiTimer); 50 msg.channel.send(`${msg.author}さんの樹脂タイマーをキャンセルしました。`); 51 } 52}) 53 54client.login(token);

試したこと

・初心者なのでなにをしたらいいかわからず手をつけられていません。

補足情報(FW/ツールのバージョンなど)

・スタミナ=樹脂

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2021/01/10 08:37 編集

様々な分岐があるようですがclearTimeoutのあるブロックは確実に通ってるのでしょうか
ku__ra__ge

2021/01/10 08:40

まず問題を切り分けましょう。10秒ごとにconsole.logでログを出力し、ボタンを押すとそれを止めるというだけの機能のプログラムを作ってみて、本当に問題はclearTimeoutにあるのか確かめましょう。
guest

回答1

0

ベストアンサー

中にざっくり説明書きました。
(1/10 19:47 編集) 簡単な内容に書き換えました

※javascriptの変数のライフサイクル、スコープについて勉強する場合、クロージャというキーワードで検索して勉強してみてください。

client.on('ready', () => { console.log('ready...'); }); // ↓のtmが 「client.on('message', message => { ... });」の外にある事が重要。 // setTimeoutしている部分は、内部関数を10秒後に実行するように設定して、そのまま先に進み、 // イベント関数は終了します。その時点で、イベント関数内で定義された変数は消えてしまうため、 // tmは外のグローバルな位置に定義しておきます。cancel呼び出し時に見るためです。 let tm; client.on('message', message => { if (message.author.bot) { return; } if (message.content === 'start') { tm = setTimeout(function(){ message.channel.send('10秒経ったよ') }, 10000) } else if (message.content === 'cancel') { clearTimeout(tm) message.channel.send('やめ') } }); client.login(token);

投稿2021/01/10 10:16

編集2021/01/10 11:52
umau

総合スコア805

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

jam79iy

2021/01/10 11:15

回答ありがとうございます。 とても内容がわかりやすかったです。 質問があります。 的を得ていない可能性がありますがご容赦ください。 今回のtmという変数はなぜグローバル変数でないといけないのでしょうか? ローカル変数では関数はtmを見れないんですか?
umau

2021/01/10 11:45

少し難しい所なので、上手く伝わるか分からないですが、、 ローカル変数は、関数が終了すると、基本的にメモリから削除されます。 1, メッセージstartを受信し、「message => { ... }」関数が始まる。 2, 「message.content === 'start'」 の分岐に入り、tm にsetTimeoutを止めるためのキーが設定される 3, else if (message.content === 'cancel') には入らず、そのまま関数は終了する。← もし、tmがローカル変数だった場合、ここでメモリから消えて無くなる。 4, 10秒以内にメッセージcancelを受信したとします。またイベント関数が呼ばれて新しく始まる。 5, 今度は「message.content === 'cancel'」の分岐に入り、clearTimeoutを呼びますが、tmは残っていないので止まらない。 メッセージを受信するたびに、「message => { ... }」が新たに開始され、まっさらな状態から始まるので、前回の呼び出しの際に取得した値を使いたい場合、それをどうやって受け取るか、と言うのがポイントになります。 回答した例では、グローバルな位置に定義することで、「message => { ... }」が終了しても消されないようにしているという事です。
jam79iy

2021/01/11 02:46

なるほど とてもわかりやすかったです。スッキリしました。 cancelを受信したときはまた新しく「message => { ... }」関数が呼び出されるという発想がありませんでした。 なのでローカル変数でもできると勘違いしていました。 わかりやすい回答、質問への返答ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問