リファクタリングについて質問です

前提・実現したいこと

リファクタリングについて質問です

chromeで拡張機能を作っています。
ソースコードではifの条件をdateのみで判定しています。
このままでも動くのですがcontentがいい加減な感じで、気になったのでこれをcontentにも適用させたいです。
単に条件を追加で書けば良いのですが見ずらくなってしまい、
スマートに書くにはどうすれ良いのかご教示願います。

該当のソースコード

javascript
1chrome.storage.local.get(function (result) {
2                //日付の有無を条件として
3                if (typeof (result["date"]) !== 'undefined' && result["date"] instanceof Array) {
4                    result["date"].push(now);
5                    result["content"].push(select_text);
6                } else {
7                    result["date"] = [now];
8                    result["content"] = [select_text];
9                }
10                chrome.storage.local.set(result);
11            });

試したこと

ifを増やしてみました。

javascript
1chrome.storage.local.get(function (result) {
2                //日付の有無を条件として
3                if (typeof (result["date"]) !== 'undefined' && result["date"] instanceof Array) {
4                    result["date"].push(now);
5                } else {
6                    result["date"] = [now];
7                }
8                if (typeof (result["content"]) !== 'undefined' && result["content"] instanceof Array) 
9                {
10                    result["content"].push(select_text);
11                } else {
12                    result["content"] = [select_text];
13                }
14                chrome.storage.local.set(result);
15            });
16```**ボールドテキスト**

行動規範の内容に同意します

回答1件

ベストアンサー

javascript
1chrome.storage.local.get(function (result) {
2
3    const now = Date.now();
4    const select_text = "hoge";
5
6    // result["date"]が未定義(undefined)の場合に空リスト設定しといて、
7    result["date"] = result["date"] || []
8    result["content"] = result["content"] || []
9
10    // 常にpushすれば分岐不要。
11    result["date"].push(now)
12    result["content"].push(select_text)
13
14    chrome.storage.local.set(result);
15});