dell_OKのプロフィール

Linuxをゼロから始めるにはプログラムの知識がないと駄目でしょうか？ 閲覧ありがとうございます。回答宜しくお願い致します。

共有ボタンをクリックすると SNSシェアリンクとテキストボックス（中にページURL）とページURL をコピーするボタンをポップアップで表示させ、✕ボタンで閉じる機能を HTML で書いてみました。 希望通りの動作にすることが出来たのですが、コードをもう少しまとめてみてはどうかとアドバイス頂きました。 頂いたアドバイス ➀ と ➁ をどのように修正すべきか悩んでおります。 ➂ の一貫性に関して dell_ok さんであれば addEventListener で統一されますでしょうか？ innerHTML は HTMLエスケープを行っていない場合にクロスサイト・スクリプティング攻撃の対象になるようなので、下記コードに修正致しました。 //旧コード divShapeText.onclick = function() { divShapeText.innerHTML = "コピーされました"; }; ↓ //新コード divShapeText.onclick = function() { divShapeText.textContent = "コピーされました"; }; ※ 最新コード https://github.com/i6Grja3R/sample_theme.git ※ 参考サイト https://qiita.com/RYO_nami/items/4543edfc7febe88 … https://dubdesign.net/javascript/execcommand-copy/ ※ 頂いたアドバイス ➀ const textboxHref = location.href; let url = location.href と同じ内容を、別の変数に代入しているのは無駄に思えます。 ➁ jQueryを読み込んでいるようですが、最後の数行を除いて、ほとんど利用していませんよね？ ➂ イベントのバインドも、「addEventListener」であったり「onclick」であったりと一貫性が感じられません。 ご提示のスクリプトに限って言えば、「textContent」と「innerHTML」も同様です。 要素取得のためだけに、jQueryを読み込むのであれば、例えば自前で 　const QS = s => document.querySelector(s); のような関数を定義しておけば、読み込みは不要になりますし、「要素を取得する関数」とだけ覚えておけば、個別に要素の変数を多数作成するよりもわかりやすくなる可能性がありますし、スクリプトも全体的に短縮化が可能です。 （jQueryに慣れているなら、関数名を$にしておいてもよい）

共有ボタンをクリックすると SNSシェアリンクとテキストボックス（中にページURL）とページURL をコピーするボタンをポップアップで表示させ、✕ボタンで閉じる機能を HTML で書いてみました。 希望通りの動作にすることが出来たのですが、コードをもう少しまとめてみてはどうかとアドバイス頂きました。 頂いたアドバイス ➀ と ➁ をどのように修正すべきか悩んでおります。 ➂ の一貫性に関して dell_ok さんであれば addEventListener で統一されますでしょうか？ innerHTML は HTMLエスケープを行っていない場合にクロスサイト・スクリプティング攻撃の対象になるようなので、下記コードに修正致しました。 //旧コード divShapeText.onclick = function() { divShapeText.innerHTML = "コピーされました"; }; ↓ //新コード divShapeText.onclick = function() { divShapeText.textContent = "コピーされました"; }; ※ 最新コード https://github.com/i6Grja3R/sample_theme.git ※ 参考サイト https://qiita.com/RYO_nami/items/4543edfc7febe88 … https://dubdesign.net/javascript/execcommand-copy/ ※ 頂いたアドバイス ➀ const textboxHref = location.href; let url = location.href と同じ内容を、別の変数に代入しているのは無駄に思えます。 ➁ jQueryを読み込んでいるようですが、最後の数行を除いて、ほとんど利用していませんよね？ ➂ イベントのバインドも、「addEventListener」であったり「onclick」であったりと一貫性が感じられません。 ご提示のスクリプトに限って言えば、「textContent」と「innerHTML」も同様です。 要素取得のためだけに、jQueryを読み込むのであれば、例えば自前で 　const QS = s => document.querySelector(s); のような関数を定義しておけば、読み込みは不要になりますし、「要素を取得する関数」とだけ覚えておけば、個別に要素の変数を多数作成するよりもわかりやすくなる可能性がありますし、スクリプトも全体的に短縮化が可能です。 （jQueryに慣れているなら、関数名を$にしておいてもよい）

共有ボタンをクリックすると SNSシェアリンクとテキストボックス（中にページURL）とページURL をコピーするボタンをポップアップで表示させ、✕ボタンで閉じる機能を HTML で書いてみました。 希望通りの動作にすることが出来たのですが、コードをもう少しまとめてみてはどうかとアドバイス頂きました。 頂いたアドバイス ➀ と ➁ をどのように修正すべきか悩んでおります。 ➂ の一貫性に関して dell_ok さんであれば addEventListener で統一されますでしょうか？ innerHTML は HTMLエスケープを行っていない場合にクロスサイト・スクリプティング攻撃の対象になるようなので、下記コードに修正致しました。 //旧コード divShapeText.onclick = function() { divShapeText.innerHTML = "コピーされました"; }; ↓ //新コード divShapeText.onclick = function() { divShapeText.textContent = "コピーされました"; }; ※ 最新コード https://github.com/i6Grja3R/sample_theme.git ※ 参考サイト https://qiita.com/RYO_nami/items/4543edfc7febe88 … https://dubdesign.net/javascript/execcommand-copy/ ※ 頂いたアドバイス ➀ const textboxHref = location.href; let url = location.href と同じ内容を、別の変数に代入しているのは無駄に思えます。 ➁ jQueryを読み込んでいるようですが、最後の数行を除いて、ほとんど利用していませんよね？ ➂ イベントのバインドも、「addEventListener」であったり「onclick」であったりと一貫性が感じられません。 ご提示のスクリプトに限って言えば、「textContent」と「innerHTML」も同様です。 要素取得のためだけに、jQueryを読み込むのであれば、例えば自前で 　const QS = s => document.querySelector(s); のような関数を定義しておけば、読み込みは不要になりますし、「要素を取得する関数」とだけ覚えておけば、個別に要素の変数を多数作成するよりもわかりやすくなる可能性がありますし、スクリプトも全体的に短縮化が可能です。 （jQueryに慣れているなら、関数名を$にしておいてもよい）

Linuxをゼロから始めるにはプログラムの知識がないと駄目でしょうか？ 閲覧ありがとうございます。回答宜しくお願い致します。