From fa59f4526a01d5bc702fb804a25d4e6ff75b82af Mon Sep 17 00:00:00 2001 From: syuilo Date: Sun, 21 Oct 2018 14:44:37 +0900 Subject: [PATCH] Improve readability --- src/services/stats.ts | 76 +++++++++++++++++++++---------------------- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/src/services/stats.ts b/src/services/stats.ts index 82cb6becf..d5c6916a5 100644 --- a/src/services/stats.ts +++ b/src/services/stats.ts @@ -108,50 +108,50 @@ abstract class Chart { if (currentStats) { return currentStats; - } else { - // 集計期間が変わってから、初めてのチャート更新なら - // 最も最近の統計を持ってくる - // * 例えば集計期間が「日」である場合で考えると、 - // * 昨日何もチャートを更新するような出来事がなかった場合は、 - // * 統計がそもそも作られずドキュメントが存在しないということがあり得るため、 - // * 「昨日の」と決め打ちせずに「もっとも最近の」とします - const mostRecentStats = await this.collection.findOne({ + } + + // 集計期間が変わってから、初めてのチャート更新なら + // 最も最近の統計を持ってくる + // * 例えば集計期間が「日」である場合で考えると、 + // * 昨日何もチャートを更新するような出来事がなかった場合は、 + // * 統計がそもそも作られずドキュメントが存在しないということがあり得るため、 + // * 「昨日の」と決め打ちせずに「もっとも最近の」とします + const mostRecentStats = await this.collection.findOne({ + group: group, + span: span + }, { + sort: { + date: -1 + } + }); + + if (mostRecentStats) { + // 現在の統計を初期挿入 + const data = await this.generateTemplate(false, mostRecentStats.data); + + const stats = await this.collection.insert({ group: group, - span: span - }, { - sort: { - date: -1 - } + span: span, + date: current, + data: data }); - if (mostRecentStats) { - // 現在の統計を初期挿入 - const data = await this.generateTemplate(false, mostRecentStats.data); + return stats; + } else { + // 統計が存在しなかったら + // * Misskeyインスタンスを建てて初めてのチャート更新時など - const stats = await this.collection.insert({ - group: group, - span: span, - date: current, - data: data - }); + // 空の統計を作成 + const data = await this.generateTemplate(true); - return stats; - } else { - // 統計が存在しなかったら - // * Misskeyインスタンスを建てて初めてのチャート更新時など + const stats = await this.collection.insert({ + group: group, + span: span, + date: current, + data: data + }); - // 空の統計を作成 - const data = await this.generateTemplate(true); - - const stats = await this.collection.insert({ - group: group, - span: span, - date: current, - data: data - }); - - return stats; - } + return stats; } }