getScriptLockでロックできなくて困っています【GoogleAppsSprict】

Question

前提・実現したいこと

【実現したいこと】
・GoogleAppsSprictの同時実行を防ぎたい
getScriptLockを使用したいのですが、
うまくロックがかからずご教授いただきたいです。

【前提】
・スプレッドシートを複数人で使用
・セル上にボタンを設置しスプリクトを設定

発生している問題・エラーメッセージ

【問題】
・同じPC、2つの違うアカウントで実行しても同時実行されてしまいます
（エラーにいかない）

該当のソースコード

function reserved_entry() {

//ロックを取得する
  var lock = LockService.getScriptLock();

try {
//30秒間のロックを実施する
lock.waitLock(30000);

    Browser.msgBox("実行中");

//終了メッセージ表示
  Browser.msgBox("処理が終了しました。");

} catch (e) {

Browser.msgBox("誰か使ってます");

}　finally　{
//ロックを開放する
lock.releaseLock();

}

}

試したこと

getDocumentLockでもロックがかかりませんでした。

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

ここにより詳細な情報を記載してください。

Answer 1

あまり詳しくないけど、興味あって調べて試してみたので回答しますね。

まずロック中に、Browser.msgBox を使うとロックが解除されてしまうようです。

https://stackoverflow.com/questions/43223774/how-to-understand-lockservice-and-implement-it-correctly

また、非常にわかりずらいのですが、

//30秒間のロックを実施する
lock.waitLock(30000);

このwaitLockやtryLockというメソッドは、指定したミリ秒の間 ロックの取得を試みる という動きをするものみたいです。

https://developers.google.com/apps-script/reference/lock

これらを踏まえて、以下のようなコードだとイメージ通りの排他処理が動作しました。
メッセージボックスの箇所は全てログに出力でも良いです。

function reserved_entry() {

  //ロックを取得する
  var lock = LockService.getScriptLock();

  try {
    //5秒間のロックの取得を試みる
    lock.waitLock(5000);
    console.log("実行中"); //ここはメッセージボックスはNG

    Utilities.sleep(20000); // 実際の処理の実行時間を20秒と想定

    //最後にロックを開放する
    lock.releaseLock();

    //終了メッセージ表示 ロック解除後なのでOK
    Browser.msgBox("処理が終了しました");

  } catch (e) {
    //既にロック済みで5秒後タイムアウトした場合エラーとなり以下のメッセージが返る
    Browser.msgBox("誰か使ってます"); 　
  }
}

ご参考までに。

Comments

[chikaMM]

とても分かりやすいご回答ありがとうございます。
Browser.msgBox を使うとロックが解除されてしまうこと大変驚きました。
設置する場所が大切ですね。
またwaitLockやtryLockの指定したミリ秒は、
後から処理を実行した人がロック取得するまでの制限時間なのですね。
こちらも勘違いしており、大変勉強になりました。

記載していただいたコードで私もロックの確認ができました。
大変助かりました。本当にありがとうございます。