GoogleAppsScriptでスプレッドシートに書いたユーザー定義関数がうんともすんとも言わない

Question

英単語について文字ごとに点数を設定し、その単語が何点になるか計算する関数を作ろうとしています。以下の通り書いたのですが、セルを指定しても=wscore("apple")と指定しても、Loadingと表示された後エラーどころか何の反応も返ってこないままセルが真っ白です。どうしてでしょうか？
ちなみにデバッグすると8行目でtoLowerCase()が呼び出せないといわれます。しかし対処法がわかりません。
よろしくお願いいたします。

function wscore(w){
var i;
var s;
var wd;
var ws;

//入力文字列を大文字に変換
wd = w.toLowerCase();

//文字数（ループ回数）を取得
for (i=1;i<=wd.length;i++){

  //i番目の文字を取得
  ws = wd.substring(i,1);

  //点数を検索
    switch (ws){
      case "a", "e", "i", "l", "n", "o", "r", "s", "t", "u":
        s = 1;
      case "d", "g":
        s = 2;
      case "b", "c", "m", "p":
        s = 3;
      case "f", "h", "t", "v", "w":
        s = 4;
      case "k":
        s = 5;
      case "j", "x":
        s = 8;
      case "q", "z":
        s = 10;
    }

//得点に足す
wscore = wscore + s;

//次の処理へ
  }
}
以上

fujillin · Accepted Answer

No6です

＞上図のR2セルに=hoge(B2)と入れてみたところ、stringと表示されました。
おやっ！　ちゃんと文字列が引数に代入されているようですね・・・？
エラーメッセージのundefinedはなんだったんだろうか・・・

＞デバッグの段階ではwには何も入っていない状態なのでundefinedになるということでしょうか。
えっ！　実行環境でデバッグしてはいないってことですか？
引数を与えずに関数を実行してみたってこと？？
それで、undefinedのエラーが出てたってことでしょうか！？
（それなら、そうなりますよねぇ…）
実行時のエラーだとばかり、てっきり勘違いをしていました。（大変、失礼いたしました）

ようやく理解できて来たようです。（遅くてすみません）
もし上記の状態であるなら、ご提示のコードの問題点を修正するだけで（多分）機能するのではないかと思います。

試みに、ご提示のものとほぼ同様の内容で、簡易的に作成してみました。
一度、試してみてください。
※ 元のコードでは、switch～case で場合分けしていますがまとめて配列で処理しています。
※ アルファベットのA~Z、a～z以外の文字は、スコアの対象としません。
　（ご提示のコードでは「y」の得点がないので、yも無得点扱いにしてあります）
※ 引数が文字列以外の場合は、""を返します。
　（対象セルに未だ文字が入力されていない場合等に、0表示にならない意味も含めています）
※ 変数名等はかなりいい加減なので、設定し直した方が良いかも。

function hoge(v){
if (typeof(v) != "string" || v.length == 0) return "";
const val = [1,3,3,2,1,4,2,4,1,8,5,1,3,1,1,3,10,1,1,1,1,4,4,8,,10];

let t = 0, vL = v.toLowerCase();
for(let i=0, c; c = vL.charCodeAt(i++);){
 t += val[c - 97]?val[c - 97]:0;
}
return t;
}

fujillin · Answer

No5です

＞wからwdを取り出す必要があるということですね？
かなり違います。

No4の回答以降は、「関数の問題以前に呼び出し側にも問題があるのでは？」と書いています。
（だから、関数には問題が無いというわけではなく、いろいろとありますが…）
とは言え、引数がundefinedで渡されているという確証がないので、まずは、それを確認してみてはいかがという意味です。
No1の回答からずっと「引数（=ｗ）を調べてみては？」と申し上げていますが、そのままの内容です。

もしも、確認がとれたととするなら、
＞wからwdを取り出す必要があるということですね？
いくら関数側で何をしようとも、値を取得することはできないでしょう。
なぜなら、必要な情報（＝引数）が渡されていないからです。

ですので、「それよりも前」に問題がありそうと申し上げているのですが、その部分に関しては情報も無く、仕様も知らないので、私にはわかりかねますという意味です。

fujillin · Answer

No4です

＞ユーザー定義関数のような引数が入るスクリプトはデバッグができない～
なぜそのようなことになるのかわかりません。
No2のお礼にご提示の画像は、多分エディタだと思いますが、見たところ、簡単なデバッグ機能が付属しているように思われますが、そうではないのでしょうか？

何度もお断りしますが、私はAppsScriptは存じませんので、ユーザ関数の呼び出しや、戻り値のセット方法がどのような仕組みがどのようになっているのか知りません。
再三書いているように、受け取った引数を調べれば問題の箇所を絞り込むことが可能と思います。
（No3のお礼からすでに、undefinedらしいことはうかがえますが…）
また、このようなステップをデバッグの一環とはお考えではないのでしょうか？

例えば、
function hoge(v){
return v===null?"null":v===""?"blanc":!isNaN(v)?"number":typeof(v);
}
などとしておいて、hogeを呼出せば、渡された引数の大雑把なタイプ分けが返されます。
もしも、こちらでもundefinedが返されるようなら、引数の内容がundefinedになっていることになるので、ご質問文に示された「ユーザ定義関数の内容」の問題ではなく、それ以前の「シートでのユーザ関数の設定方法」の問題になるものと思います。
この場合は、シート側の仕様がどうなっているのかをよくご確認ください。

まずは、それに適合した使用法をしない限りは、関数の利用はできないものと思いますので・・・

fujillin · Answer

No3です。

＞「TypeError: undefinedのメソッド「toLowerCase」を呼び出せません。
ようやく、原因のヒントが少しは得られてきたようです。
引数がundefinedと評価されているということなので、質問文にご提示の関数ではなく、それ以前に、呼出し側で引数の内容がセット出来ていないことが疑われます。
これ以上は、全く情報が記されていないのでこちらではわかりかねます。

＞返却値について書きこまなければいけないようなので～
それに関しても気にはなっていましたが、ご質問のエラーとは直接関係が無さそうなので、ひとまずスルーしていました。

No2にも書きましたが、私はAppsScriptを存じませんので、仕様はわかりませんが……
＞wscore = wscore + s;
これって、functionを上書きしてしまっていませんか？
（処理内容としても怪しげな内容になっていますけれど・・）
普通に考えると、特殊なケースでもない限りは行わないことのように思います。
また、AppsScriptにおけるユーザ関数の戻り値は、returnで戻す仕様ではないのでしょうか？

……などなど、正しい仕様に合わせた関数にしておかないと、なかなか意図する結果は得られないものと思います。

fujillin · Answer

No2です

＞wは上記の例では"apple"やご指摘の通りRangeにあたるものです。
"apple"は文字列ですが、RangeはObjectでしょうから同じには扱えないでしょう。

＞エラーですが、メッセージが秒速で消えるので、～～
エラーメッセージがどこにあるのかわかりませんが、下の方の変数欄を右に見ていくと（多分、内容が表示されているのではないかと想像）、ｗはundefinedになっていませんか？
どのタイミングで、どのようなスコープによる参照なのかわからないので、実行時にもundefinedとは限らないのかも知れませんが、undefined.toLowerCase() でも、当然エラーが発生するでしょう。

毎回書いていますように、wのタイプが何で、内容がどうなっているかを確認するのが先ではないでしょうか？

fujillin · Answer

No1です。

AppsScriptは知りませんので、はっきりとはわかりませんが・・・

＞wはスプレッドシートから引数として渡してくるので、～
何が渡されるかは、スクリプト側で制御しているのではないのですか？
(いきなり渡されることはないでしょうから、何らかの要求をしているはずです）
もしも、不定のものが渡されてくるような状況であるなら、受け取った時点でのチェックは不可欠と思いますが、実際はそのようなことはなくて、引数の内容は決まっているのではないのでしょうか？

根拠のないまったくの想像ですが、エクセルで言うところの「Rangeオブジェクト」になっていそうな気がしますが、そんなことはありませんか？

＞これが文字列であることを予め設定しておくことができるんでしょうか？
そのような機能は（多分）ないと思います。
関数を再定義して事前にチェックするような処理を加えれば、見かけ上はそのような機能を作成することは可能だと思います。
とは言っても、結局のところ、その内部で以下と同じような処理を行うことになるのではないでしょうか。

通常は、受け取った側で内容をチェックして、想定外の内容の場合は
　・無視する（＝何もしない）　とか
　・エラー表示する　　　　　　とか
　・実際にエラーを発生させる
などの処理を行うのではないでしょうか？

いずれにしろ、typeof w でwの種類がわかるのではないかと思いますので、チェックしてみるのが宜しいかと。あるいは、エラーメッセージに書いてあったりしませんか？

fujillin · Answer

こんにちは

＞デバッグすると8行目でtoLowerCase()が呼び出せないといわれます。
単純に、wが文字列ではないのでは？
wの内容を確認してみるか、エラーメッセージをよく読むと、もう少し情報が増えるのではないでしょうか？

GoogleAppsScriptでスプレッドシートに書いたユーザー定義関数がうんともすんとも言わない

No6です

No5です

No4です

No3です。

No2です

No1です。

こんにちは

似たような質問が見つかりました

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング