dell_OKのプロフィール
- ベストアンサー数
- 776
- ベストアンサー率
- 13%
- お礼率
- 100%
プロフィールの画像は、 「フリーペンシル」さまの素材を使わせていただいてます。 https://iconbu.com/
- 登録日2005/09/28
- WordPressの質問(q10315144)の続
共有ボタンをクリックすると 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に慣れているなら、関数名を$にしておいてもよい)
- WordPressの質問(q10315144)の続
共有ボタンをクリックすると 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月で契約満了になりました。理由を聞きましたが、クビみたいなものです(経費節減とかではなく、私の問題です。 そのことは上が判断したことなので良いのですが(良くはないですが苦笑 仕方ないです)、問題は周りの同僚に聞かれた時にどう言うかです。正直に「きられたんだ」と言えばいい反面、私にもプライド?があり、きられたということを言い出せません。恥ずかしいですし、やはり少なからずショックも受けているので、言うと悲しくなってしまう気持ちもあります。 ただ「自分の都合で3月で辞めることにした」と言ったとしても、どこかから漏れたらウソをついたことになるし(直属の上司は口はとても固いですが、その上司の上司はちょっと軽そう。しかも私の課の勤続が長いパートさんと仲が良い)、「プライドあったんだな。実はきられたのに」と思われるのも何か嫌です。 あまり良い質問ではないですが、「きられた」と正直に言うか、「自主退職します」と言うか、皆さんならどうしますか?理由も教えていただけると嬉しいです。
- Linuxをゼロから始めるには
Linuxをゼロから始めるにはプログラムの知識がないと駄目でしょうか? 閲覧ありがとうございます。回答宜しくお願い致します。
- ベストアンサー
- Nardis_666
- Linux系OS
- 回答数8