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

共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい）

（いつもコードをまるごと提示いただきありがとうございます。） ファイル整理、例えばフィル名からフォルダを作り、そこに集約するとか。写真撮影日でファイルを整理するとか、ファイル名をまとめて変更するとか。 このような、シンプルなプログラムを書く場合、いろいろな言語を操れる方は、どの言語が適していると考えているのでしょうか？ （VBA、バッチ、VC＃など） 私の場合はエクセルを持っていれば、VBAを使います。

共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい）

共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい）

超初心者です。 以下の指定日より指定日数を書き出すコードを以下のように作成し運用していましたが、PHPのヴァージョンを上げる必要があり書き換えないとならないようです。 もし出来ましたら新しいコードをご教授頂けませんでしょうか。 $today=time(); $limit=90; echo "<select name=月日>"; $menu=$today; for($i=3; $i<$limit; $i++){ $menu=date("Y/m/d",$today+86400 * $i); echo "<option>$menu</option>"; } echo "</select>";

昔いた「太寿山」って相撲取りをしってますか？ ムーミンに似てるので愛称が「ムーミン」って言われてました。 参考動画↓ https://www.youtube.com/watch?v=sMqkzPoc59Y ＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝ 1、覚えてる 2、知らない

皆さんはビットコインの積立はしてます？ 私は3年前くらい前から積み立ててます。0.1btcも持ってませんが。。 ＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝＝ １、積み立ててる。 2、一括で買って放置。 3、やってない。 4、別の仮想通貨を積み立てている。 5、その他。

男性でフリーター、パートの非正規雇用、プータロー、ニート、でも結婚できている男性ですよね。 なぜか20代で結婚できて子供いたりしてね。 どうしてですか？　なぜ結婚できているか不思議です。 真面目に働いている女性でも非正規だとなぜ正社員にならないかいろんな人に聞かれたりしますよ。 私は、真面目なタイプですが普段の生活に出会いがなくて 結婚相談所に入会しました。 お見合いでは、女性の雇用形態、職歴をこだわる人が本当に多いです。面接のように過去の職歴を聞かれたり、今の仕事どれくらい勤めているか前の仕事の勤務期間、退職理由など聞かれました。 不快でしたがお見合いの場なので我慢していました。 数年活動したのち相談所で結婚が決まりました。 親戚にいるんです、夢追いフリーターなのに20代で結婚して子供がいるんです。年齢は30歳前後だそうですが10歳近く年上の医療関係の正規雇用の女性と結婚しているそうです。 男性は、そこそこイケメン風で奥さんは不美人。 奥さんの収入で生活しているみたいな夫婦です。 それ聞いてムカつきました。 なんでこんな夢追いフリーターが結婚できて子供を持てていて 定職持っていて真面目な男性が結婚できないの？ きちんと費用かけて真面目に婚活して真面目に働いているのに 正規雇用じゃないとなぜ正社員にならないかいろいろ言われたり、 過去の職歴を根掘り葉掘り聞かれて不愉快な思いをしたりしました。婚活で嫌な思いや辛いを思いをたくさんしました。 親戚のその夫婦がどうも良い印象を持てません。 何、男でフリーターのくせに20代で結婚して子供持っているんだよ、美味しい思いしてるね笑　い 今後会うことが会った場合どう接したらいいですか。

共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい）

大掃除はしましたか？ 1.した 2.まだしていない 3.その他 僕は、2のまだしていないです。 皆さんは、大掃除はしましたか？

共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい）