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

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

ただいまの
回答率

90.48%

  • JavaScript

    16991questions

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

関数内から複数の関数を呼び出したい

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 1,781

touch

score 35

onclick時に、以下の三つの関数を、番号順で呼び出したいと考えています。
①未入力チェックを行う関数 checkName()
②文字数チェックを行う関数 checkByte()
③確認ダイアログを表示する関数 checkUpdate()

これら三つの関数を、一つの関数内にまとめて記述し、かつ上記の順で評価を行えるようにするには、どのように記述したらよいのでしょうか?

イメージを下記コードとともに記します。


<%--ボタン押下時に、callFunction()関数を呼ぶ。returnはいる? --%>
<input type ="submit" value="登録" name="regist_btn" onclick="return callFunction();">

<%--idの値をhiddenで保持。 --%>
<input type ="hidden" id="id" value"<%=id%>">



<%--callFunction関数内 --%>
funcrtion callFunction(){


       <%--名前が未入力ならアラートをだし、falseを返す --%>
       return checkName(); ①

       <%--名前が入力されているが、文字数が制限を超えている場合にアラートをだし、false--%>
      return checkByte(); ②

      <%--idの値を取得 --%>
       var id = document.getElementByte('id')


     <%-- ①と②がtrueで、idがnullでなければ確認ダイアログを出す関数を呼ぶ --%>
     if(id != null){
       updateChk(); ③
     }
}

/*
* 商品名のテキストボックスが空白もしくはスペースの場合、
* アラート表示をするcheckForm関数です.
*/
function checkName(){

    /*コードを見やすくするため、formプロパティを変数に取得します*/
    var f = document.form;
                
    /*あらかじめ商品名のテキストボックスに含まれる空白文字を取り除いておきます*/
    f.item.value = f.item.value.replace(/^\s+|\s+$/g, "");

    /*空白文字を取り除いた上で、未入力チェックを行ないます
    AllBrankの場合も、未入力としてアラートを出します*/
                
    //商品名が入力されていない場合、アラート表示します
    if(f.item.value == "") {
        window.alert('「商品名を入力して下さい」');

        //商品名のテキストボックスにフォーカスを合わせます
        f.item.focus();
        return false;
    }
    return true;
}

/*
* 文字列のバイト数を求めるgetByte関数です.
* @param text テキストボックスに入力された値
* @return count バイト数
*/
function getByte(text){
    
    //バイト数を代入する変数
    var count = 0;
                
    /*escape()メソッドにより、値の長さ分
    文字列を1文字ずつ取り出しエンコードします。*/
    for (i = 0; i < text.length; i++){
        
        //エンコードされた文字をcharaに代入
        var chara = escape(text.charAt(i));
                    
        /*もしcharaの長さが、4文字未満ならば1バイト、
        4文字以上ならば2バイトとしてカウントします。*/
        if (chara.length < 4){
            count++; 
        }else{
            count += 2;
        }
    }
    return count;
}
            
/*
* バイト数をチェックし、最大値を超えて入力できないようにするcheckByte関数です.
*/
function checkByte(){
    
    /*テキストボックスの値を引数にgetByte関数を呼び出し、バイト数を取得*/
    var countByte = getByte(document.getElementById('item').value);

    /*もしバイト数が最大値以上になった場合、
    アラート表示する処理です。*/
    if(countByte >= 50){
        
        window.alert('「商品名は全角25文字、半角50文字以内で入力して下さい」');
        return false;
    }
    return true;
}

/*
*確認ダイアログ
*/
function updateChk(){
 var flag = window.confirm('更新しますか?');
return flag;
}




個々の関数については動作の確認が取れています。
現在このような実装になっていますが、名前の未入力でアラートを出した後は、登録処理は中断されますが、文字数をこえた入力をした場合は、アラートが出ず、そのまま処理が走ってしまいます。おそらく、callFunction内の実装が正しくないからだと思いますが、調べても解決策が見つかりませんでした。どなたか正しい実装方法を教えてはいただけないでしょうか?よろしくお願いいたします。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

return checkName(); ① で結果がtrueでもfalseでも処理が終了しています。
if文を追加して、処理結果がfalseの場合はreturnで処理を終了し、trueの場合は後続の処理を行うよう記述してみてください。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/07/04 20:38

    ご回答ありがとうございます。
    処理結果でif分岐させる方法が大変参考になり、無事思った通りに動くようになりました。
    誠にありがとうございました。

    キャンセル

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

  • ただいまの回答率 90.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    JavaScriptの確認ダイヤログについて

    JavaScriptの確認ダイヤログについての質問です。以下のコードにて、チェックボックスをチェックした値について、削除ボタンを押すと「本当に削除しますか?」というダイヤログが表示

  • 解決済

    指定したインデックスの要素を削除するにはどうしたら良いでしょうか?

    フォームデータをページを更新せず送信し、下記のようにformからsubmitクリックで指定したインデックスの要素を削除したいのですがうまくできません。 「<h3>タイトル</h3>

  • 受付中

    JavaScriptでのValueセット

    お世話になります。 つたないコードで申し訳ありませんが、 JavaScriptの処理がうまくいかず悩んでおります。 どこをどう直せば良いのか、アドバイス頂けると幸いです。 ●何

  • 解決済

    javascriptとbodyの順を逆にしたら動かなくなった。

    作成したプルダウンメニューについての評価を頂いた際に、 ヘッダの中にscriptを書いたほうが良いと言われたので移植してみたのですが、 なぜか動かなくなりました。 とても慎重に行

  • 解決済

    javascriptをjspに組み込む

    こんにちは。 質問です。 下記のJSPにjavascriptを組み込みました。 そこで1点問題が生じまして、1番下のほうの登録や更新のOKボタン の部分で、どうしてもボタンの名称を

  • 解決済

    JavaScriptでギャラリーを作りたい

    現在JSでギャラリー的なものを作ろうとしています 仕様としては 画面上段に画像(原寸より小さいもの)を表示し、クリックされると下にその画像の原寸大が表示されるというものです ですが

  • 解決済

    datalistタグの子要素optionタグに仕込んだ自作の属性をgetAttributeしたいがで...

    html5から使用できるようになったdatalistタグによるサジェストリストですが、optionタグに仕込んだ自作の属性fidをgetAttributeできません。 selec

  • 解決済

    複数のselectのうちどれか1つは必須項目にしたい

    下記コード、div.classlistの中に複数のセレクトメニュー3つとテキストボックスがあります。 この中で、3つのセレクトメニューのうち、未選択または複数選択している場合は入

同じタグがついた質問を見る

  • JavaScript

    16991questions

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