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

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

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

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

解決済

JavaScriptのリファクタリングについて

ttt---aaa
ttt---aaa

総合スコア21

JavaScript

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

2回答

-5評価

1クリップ

379閲覧

投稿2022/07/13 00:49

前提及び実現したいこと

ここに質問の内容を詳しく書いてください。
(例)
JavaScriptでシステムを作っており、やりたいことを実装できているのですが
もっとスマートにコードを記述したいと考えています。
JavaScriptの知識が浅いため、該当コードのリファクタリングなどについてお教えいただければと思います。

該当のソースコード

//編集画面読込時のイベント $p.events.on_editor_load = function () { getParentData(); } //「伝票作成」の「コース(分類E)」の値が変更されるたびに呼び出されるイベント $(document).on('change', '#' + $p.tableName() + '_ClassE', function () { getParentData(); }); //「伝票作成」の「オプション(分類G)」の値が変更されるたびに呼び出されるイベント $(document).on('change', '#' + $p.tableName() + '_ClassG', function () { getParentData(); }); //Bテーブルの「分類H」の値が変更されるたびに呼び出されるイベント $(document).on('change', '#' + $p.tableName() + '_ClassH', function () { getParentData(); }); //「伝票作成」の「交通費(分類Q)」の値が変更されるたびに呼び出されるイベント $(document).on('change', '#' + $p.tableName() + '_ClassQ', function () { getParentData(); }); function getParentData() { //Bテーブルの「新コース(分類R」に値がセットされている場合のみ、Aテーブルの指定したレコードの値を取得する if ($('#' + $p.tableName() + '_ClassR').val()) { $p.apiGet({ //Bテーブルの「分類R」の値(レコードID)を指定 'id': $p.getControl('ClassR').val(), 'done': function (data) { //Bテーブルの「数値A」にAテーブルから取得した「数値A」の値をセット $p.set($p.getControl('NumA'), data.Response.Data[0].NumA); }, 'fail': function (data) { console.log(data); } }); } //Bテーブルの「分類G」に値がセットされている場合のみ、Aテーブルの指定したレコードの値を取得する if ($('#' + $p.tableName() + '_ClassG').val()) { $p.apiGet({ //Bテーブルの「分類G」の値(レコードID)を指定 'id': $p.getControl('ClassG').val(), 'done': function (data) { //Bテーブルの「数値E」にAテーブルから取得した「数値C」の値をセット $p.set($p.getControl('NumE'), data.Response.Data[0].NumC); }, 'fail': function (data) { console.log(data); } }); } //Bテーブルの「分類H」に値がセットされている場合のみ、Aテーブルの指定したレコードの値を取得する if ($('#' + $p.tableName() + '_ClassH').val()) { $p.apiGet({ //Bテーブルの「分類H」の値(レコードID)を指定 'id': $p.getControl('ClassH').val(), 'done': function (data) { //Bテーブルの「数値D」にAテーブルから取得した「数値C」の値をセット $p.set($p.getControl('NumD'), data.Response.Data[0].NumC); }, 'fail': function (data) { console.log(data); } }); } //Bテーブルの「分類Q」に値がセットされている場合のみ、Aテーブルの指定したレコードの値を取得する if ($('#' + $p.tableName() + '_ClassQ').val()) { $p.apiGet({ //Bテーブルの「分類Q」の値(レコードID)を指定 'id': $p.getControl('ClassQ').val(), 'done': function (data) { //Bテーブルの「数値F」にAテーブルから取得した「数値C」の値をセット $p.set($p.getControl('NumF'), data.Response.Data[0].NumC); }, 'fail': function (data) { console.log(data); } }); } }

良い質問の評価を上げる

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

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

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

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

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

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

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

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

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

yambejp

2022/07/13 00:57 編集

見た感じjQueryっぽいですが、jsにしたいのですか?それともjQueryとしてコードを整理したいのですか? $pとか定義がないですが、なにかのライブラリを使用していますか?
m.ts10806

2022/07/13 01:09

質問じゃなく作業依頼になっていると思います。 取り組んだ結果で起きている問題について質問してください。
ttt---aaa

2022/07/13 11:24

yambejp様 コメントありがとうございます。 こちらあるアプリを使っており動きだけスクリプトを打てる仕様となっています。 m.ts10806様 コメントありがとうございます。 おっしゃる通り、質問ではないこと理解しております。大変申し訳ございません。

まだ回答がついていません

会員登録して回答してみよう

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

ただいまの回答率
87.20%

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

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

質問する

関連した質問

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

JavaScript

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