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

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

ただいまの
回答率

90.40%

  • JavaScript

    18136questions

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

数字が一つずつ上がっていく以外には内容が同じコードのまとめ方。

解決済

回答 5

投稿

  • 評価
  • クリップ 0
  • VIEW 817

GoheiKusumi

score 1

学校の課題で、javascriptを使ってlocalstrorageに多数のメールアドレスやパスワードをまとめて保存できるものをつくっています。

質問:数字が一つずつ増えていく以外には、コードの内容が同じ場合、どのような方法でまとめることができますか?(”//ボタンを押すとkeyとvalueが各テキストエリアに表示されるように”のところなどです。)
データを保存したり、クリックして呼び出すところまではできたのですが、単純なコードを繰り返している部分があり、もう少しうまくまとめたいと考えています。特にデータ量が増えた時などを考えたコードの書き方を学びたいです。

<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>myMemoPad</title>
<script src="js/jquery-2.1.3.min.js"></script>
<link rel="stylesheet" href="css/pass.css">

<script>
$(function(){

//ボタン名を各keynameに  
var keyname_0 = localStorage.key(0);
document.querySelector("#btn0").innerHTML = keyname_0;

var keyname_1 = localStorage.key(1);
document.querySelector("#btn1").innerHTML = keyname_1;

var keyname_2 = localStorage.key(2);
document.querySelector("#btn2").innerHTML = keyname_2;


//ボタンを押すとkeyとvalueが各テキストエリアに表示されるように
$("#btn0").on("click", function(){
    var keyname_0 = localStorage.key(0);
    document.querySelector("#key_area").innerHTML = keyname_0;
    var Data0 = localStorage.getItem(localStorage.key(0));
    var pmData0 = JSON.parse(Data0);
    vala = pmData0.pas;
    document.querySelector("#password_area").innerHTML = vala;
    valb = pmData0.mai;
    document.querySelector("#mail_area").innerHTML = valb;
    });

$("#btn1").on("click", function(){
    var keyname_1 = localStorage.key(1);
    document.querySelector("#key_area").innerHTML = keyname_1;
    var Data1 = localStorage.getItem(localStorage.key(1));
    var pmData1 = JSON.parse(Data1);
    vala = pmData1.pas;
    document.querySelector("#password_area").innerHTML = vala;
    valb = pmData1.mai;
    document.querySelector("#mail_area").innerHTML = valb;
    });


$("#btn2").on("click", function(){
    var keyname_2 = localStorage.key(2);
    document.querySelector("#key_area").innerHTML = keyname_2;
    var Data2 = localStorage.getItem(localStorage.key(2));
    var pmData2 = JSON.parse(Data2);
    vala = pmData2.pas;
    document.querySelector("#password_area").innerHTML = vala;
    valb = pmData2.mai;
    document.querySelector("#mail_area").innerHTML = valb;
    });    


//テキストボックスでkeyとvalueを入れる
$("#save").on("click", function(){
    var keyname = $("#key_area").val();
    var data = {pas:$("#password_area").val(),mai:valuem = $("#mail_area").val()}
    if (keyname.length == 0){
        alert("サイト名を入力してください");
    }else{
        alert("保存しました");
        localStorage.setItem(keyname, JSON.stringify(data));
        $('#select').append($('<option>').html(keyname).val(value));
    }
});
});
</script>

</head>
<body>
  <header>
    <h1>Hard password recorder</h1>
  </header>
   <main>
      <p>サイト名</p>
      <textarea id="key_area" value="new"></textarea>
      <p>パスワード</p>
      <textarea id="password_area"></textarea>
        <p>メール</p>
          <textarea id="mail_area"></textarea>

              <div id="save">Save</div>
              <div id="clear">Clear</div>


      <p id="btn0"></p>
      <p id="btn1"></p>
      <p id="btn2"></p>
  </main>

  <footer><small></small></footer>

</body>

</html>

試したこと

forやthisを使うことが鍵になりそうだと言うことはわかったのですが、今回の場合どのように関数をつくれば良いのか理解できませんでした。

どうぞよろしくお願いいたします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 5

checkベストアンサー

+1

$("#btn0").on("click", function(){
    var keyname_0 = localStorage.key(0);
    document.querySelector("#key_area").innerHTML = keyname_0;
    var Data0 = localStorage.getItem(localStorage.key(0));
    var pmData0 = JSON.parse(Data0);
    vala = pmData0.pas;
    document.querySelector("#password_area").innerHTML = vala;
    valb = pmData0.mai;
    document.querySelector("#mail_area").innerHTML = valb;
    });

//1、2と続く


のところを

$("#btn0").on("click",hoge(0));
$("#btn1").on("click",hoge(1));
$("#btn2").on("click",hoge(2));

function hoge(i){
 return function(){
   var keyname = localStorage.key(i);
  //処理続く
 }
}


のような形にすればいいと思います。

なお、ちょこちょこkeyname_0みたいな感じの変数が出てきますが、スコープを考えればそれらはkeynameでいいです。そうした方がコードの再利用性が上がります。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/13 12:16

    関数hogeが function を返さないといけない書き方ではありませんか?
    もしくは下記のような書き方にするか。
    $("#btn0").on("click", function(){hoge(0);});

    キャンセル

  • 2017/05/13 12:37

    そうですね。
    言いたかったことはちょっとだけ違う同じような処理は、関数と引数を使ってまとめられるってことでしたが、正確な書き方ではなかったです。

    キャンセル

  • 2017/05/13 14:45

    oskbt様
    ご回答ありがとうございました!
    一番知りたかった箇所について、具体的にご回答をくださり大変勉強になりました。
    早速実践してみます。
    ありがとうございました!

    キャンセル

+1

大変いい質問です。先生に聞いて下さい。先生は生徒に次に何を教えるべきかわかります。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/13 07:00

    あるいは同じ課題をやっている友達に聞いて下さい。一人では気づかないことを補い合ってともに成長できます。

    ローカル変数、引数の二つが重要です。

    キャンセル

  • 2017/05/13 14:49

    早急なご回答ありがとうございます。
    周りのメンバーにも聞いてみたいと思います!
    ローカル変数、引数について早速調べてみます!
    ありがとうございました。

    キャンセル

+1

「数字が一つずつ上がっていく」というよりlocalStorageの中身からボタンを作ってそれをクリックさせるほうが良い気がします。

const $_s = $( '#select' ), $_k = $( '#key_area' ), $_p = $( '#password_area' ), $_m = $( '#mail_area' );
const func = key => {
    let d = JSON.parse( localStorage.getItem( key ) );
    $_s.append( $( '<option>' ).html( key ).val( d.mai ) );
    $( '#clear' ).after( $( '<button></button>' ).text( key ).addClass( 'from-storage' ).on( 'click', _=> {
        $_k.val( key );
        $_p.val( d.pas );
        $_m.val( d.mai );
    } ) );
};
Object.keys( localStorage ).forEach( func );
$( '#clear' ).on( 'click', _=> {
    $_s.html( '' );
    $_k.val( '' );
    $_p.val( '' );
    $_m.val( '' );
    $( '.from-storage' ).remove();
    localStorage.clear();
} );
$( '#save' ).on( 'click', _=> {
    let key = $_k.val();
    let data = { pas : $_p.val(), mai : $_m.val() };
    if ( !key ){
        alert( 'サイト名を入力してください' );
    }else{
        localStorage.setItem( key, JSON.stringify( data ) );
        func( key );
        alert( '保存しました' );
    }
} );
<main>
    <select name="select" id="select"></select>
    <label for="key_area">サイト名</label>
    <textarea id="key_area" value="new"></textarea>
    <label for="password_area">パスワード</label>
    <textarea id="password_area"></textarea>
    <label for="mail_area">メール</label>
    <textarea id="mail_area"></textarea>

    <button id="save">Save</button>
    <button id="clear">Clear</button>
</main>

動くサンプル:https://jsfiddle.net/pzob8mvc/


【JavaScript: localStorage のすべてのキーを列挙する - Sarabande.jp】
http://blog.sarabande.jp/post/73643047325

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

1,2,3など同じ数字が出てきて、それが同じ処理の場合、forでその数字をvar iにしてi++などをすれば連続で出来ます。"あ"+i があ1、あ2、あ3などのように。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/13 14:47

    早急なご回答をありがとうございました!
    早速forとi++を使ってみます!
    ありがとうございました!

    キャンセル

0

      <p id="btn0"></p>
      <p id="btn1"></p>
      <p id="btn2"></p>

まず、ここをid属性ではなくclass属性に変更しないと、それぞれのボタンに対して個別でクリックイベントを設定するハメになってしまいます。

      <p class="btn"></p>
      <p class="btn"></p>
      <p class="btn"></p>

このように変更すれば、$(".btn")で同じクラス名を持つボタンを全て取得できるので、それらの要素に対して、ループ処理でクリックイベントを紐づけることができるようになります。

あとはループの中で、他の方が回答されているように対応されればよろしいかと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/13 14:52

    ご回答くださりありがとうございます!
    そもそもidで分けているところが長いコードの原因だったのですね。
    参考になりました!
    ありがとうございます。

    キャンセル

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

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

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

  • JavaScript

    18136questions

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