Skip to content

【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78

Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom
chanchan-co:master
Open

【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom
chanchan-co:master

Conversation

@chanchan-co

@chanchan-cochanchan-co commented Oct 28, 2025

Copy link
Copy Markdown

概要

お世話になっております。杉山です。

チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。
課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md
詳細情報: README

実施内容

Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。

  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築

本番環境

使い方

  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること

環境構築手順

cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed

動作確認手順

備考

ご確認よろしくお願いいたします。

seichan23and others added 30 commits October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others added 14 commits October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view

@chanchan-co

chanchan-co commented Oct 29, 2025

Copy link
Copy Markdown
Author

追記(10月30日)

ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。

ER図

erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"
Loading

アーキテクチャ構成図

スクリーンショット 2025-10-29 23 51 20

@ksugiura-ene

ksugiura-ene commented Oct 30, 2025

Copy link
Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。
バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。
また段階的な計算のロジックも正しく実装されていることを確認しました。
いくつか実装に関するコメントをさせていただきます。

ksugiura-ene
ksugiura-ene reviewed Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb Outdated
# Uncomment if you wish to allow Action Cable access from any origin.
# config.action_cable.disable_request_forgery_protection = true

config.middleware.insert_before 0, Rack::Cors do

@ksugiura-eneksugiura-eneOct 29, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q

本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。

@chanchan-cochanchan-coOct 30, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、
段階的にdevelopment/ productionで設定を分けて行いましたが
最終的に共通化するのを失念していました。

内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。

Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml Outdated
from_kwh: 使用量(kwh)の下限
to_kwh: 使用量(kwh)の上限
unit_price: 単価
date:

@ksugiura-eneksugiura-eneOct 29, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q

今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。

@chanchan-cochanchan-coOct 30, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、
今回の実装では使用していませんが、そのまま残しております。

理由:

  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる

今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。

ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。

Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv Outdated
@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB

@ksugiura-eneksugiura-eneOct 29, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS

いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。

具体的には
plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。
DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。

@chanchan-cochanchan-coOct 30, 2025

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。
こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので
関連する要素のidを指定して登録するようにしておりました。

ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。

@ksugiura-ene

ksugiura-ene commented Oct 30, 2025•
edited
Loading

Copy link
Copy Markdown

Q

CloudSqlの方がCloudRunからの接続が容易だと思いますが
Supabaseを選択された理由があれば教えていただけますでしょうか。

@chanchan-co

chanchan-co commented Oct 30, 2025

Copy link
Copy Markdown
Author

CloudSqlの方がCloudRunからの接続が容易だと思いますが
Supabaseを選択された理由があれば教えていただけますでしょうか。

仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。
その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。

Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。
また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。

@chanchan-co

chanchan-co commented Oct 30, 2025

Copy link
Copy Markdown
Author

@ksugiura-ene
お忙しい中、
レビュー及びコメントをしていただき、ありがとうございます。

頂いたコメントに対してご回答させていただいておりますので
お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️

@ksugiura-ene

ksugiura-ene commented Oct 30, 2025•
edited
Loading

Copy link
Copy Markdown

お忙しい中ご回答ありがとうございます。
私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。

@chanchan-co

chanchan-co commented Oct 30, 2025

Copy link
Copy Markdown
Author

@ksugiura-ene
ご確認ありがとうございました。
コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。
よろしくお願いいたします。

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments
+1 more reviewer
@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 by chanchan-co · Pull Request #78 · enechange/coding-challenge · GitHub
Skip to content
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装 - #78
Open
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Open
【杉山】serverside_challenge_2: 電気料金のシミュレーションを実装#78
chanchan-co wants to merge 116 commits into
enechange:masterfrom 
chanchan-co:master
Conversation
@chanchan-co
@chanchan-cochanchan-co
commented
Oct 28, 2025

Copy link


Copy Markdown

概要
お世話になっております。杉山です。
チャレンジ課題「serverside_challenge_2」電気料金のシミュレーションの実装をさせていただきました。

課題内容: https://github.com/enechange/coding-challenge/blob/master/serverside_challenge_2/README.md

詳細情報: README
実施内容
Ruby on RailsとNext.jsを用いて、電気料金シミュレーションAPIおよびフロントエンドを実装しました。
  • アンペア数(A)、使用量(kWh)を受け取りプランごとの電気料金を返すAPIを実装
  • UIからシミュレーションを利用できるようにフロントエンドを実装
  • 料金データをDBで管理できるようにしてCSVからデータ登録を可能に
  • フロントエンド、バックエンドの本番環境構築
本番環境
使い方
  • 上記アプリケーションURLにアクセス
  • 「契約アンペア数(A)」と「使用量(kWh)」を入力後「計算する」ボタンを押下
  • 各プランの「プロバイダ名」「プラン名」「電気料金」情報が返ってくること
環境構築手順
cd coding-challenge/serverside_challenge_2/challenge
# 環境変数ファイルの作成
cp api/dot.env api/.env
cp front/dot.env front/.env
# ビルド
make build
# 起動
make up
# apiコンテナに入る
make api-bash
# migrate&初期データ投入
rails db:migrate:reset db:seed
動作確認手順
備考
ご確認よろしくお願いいたします。
seichan23and others
added 30 commits
October 17, 2025 21:33
seichan23seichan23
rename docker-compose.yml to compose.yml
c5f306d
seichan23seichan23
version指定は削除
948f963
seichan23seichan23
.gitignoreに.envを追加
0c3a211
seichan23seichan23
webコンテナの環境変数は.envから読み込むように
be98946
seichan23seichan23
dot.envを作成
3a115a8
seichan23seichan23
database.ymlの設定値は環境変数から読み込むように
61986c4
seichan23seichan23
dot.test.envを追加
52ddc70
seichan23seichan23
test用のcomposeファイルを作成
cdbf2c7
seichan23seichan23
Makefileを作成
bc156fd
seichan23seichan23
gem install rubocop rubocop-rails rubocop-shopify & bundle install
dc368aa
seichan23seichan23
rubocopの設定追加
15917b1
seichan23seichan23
rubocop -a
4e97f8f
seichan23seichan23
gem install annotate & bundle install
40d89f7
seichan23seichan23
rails g annotate:install
674475c
seichan23seichan23
testで使うgemを追加
101069f
seichan23seichan23
test_helperに設定を追加
79088cb
seichan23seichan23
time_zoneの設定を追加
d0dda1f
seichan23seichan23
デフォルトの言語設定追加とja.ymlを作成
488d75c
seichan23seichan23
.rubocop.ymlのrequireをpluginsに修正
4db2ac4
seichan23seichan23
Makefileを修正
3d60eab
seichan23seichan23
DATABASE_HOSTを環境変数に追加
cac81fd
@chanchan-co
Merge pull request#1from seichan23/chore/setup
8d15bb8
開発環境の最低限のセットアップ
seichan23seichan23
gem install pry-rails & bundle install
ce7ed45
seichan23seichan23
rails g model Provider & rails db:migrate
6ed84ca
seichan23seichan23
Providerモデルにバリデーションを追加
db673e5
seichan23seichan23
rails g factory_bot:model Provider
cf27da3
seichan23seichan23
Providerモデルにバリデーションのテストを追加
da84ec5
seichan23seichan23
ja.ymlにrails-i18nのja.ymlを追記
300d619
seichan23seichan23
Providerモデルの属性の日本語訳を追加
fa5636b
seichan23seichan23
csvをインポートするconcernを作成
c54a23b
seichan23and others
added 14 commits
October 28, 2025 22:51
seichan23seichan23
電気料金計算ページを作成
fb22076
seichan23seichan23
環境変数共有用ファイルに環境変数を追加
b6037a5
@chanchan-co
Merge pull request#9from seichan23/feature/add_electricity_charges_…
8b81fbd
…page
電気料金計算ページのUIとAPI連携処理を実装
seichan23seichan23
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
a6e4fe4
@chanchan-co
Merge pull request#10from seichan23/fix/front_validator_setting
85f4ce1
本番環境のビルドで失敗するので、設定ファイルのファイル読み込みの箇所を修正
seichan23seichan23
apiの本番環境にcorsの設定を追加
505e3d8
@chanchan-co
Merge pull request#11from seichan23/feature/add_cors_setting_produc…
538b126
…tion
apiの本番環境にcorsの設定を追加
seichan23seichan23
metadataを修正
a5eb926
seichan23seichan23
フォームと計算結果を横並びにしてみやすいようにデザインを変更
13fbfd3
@chanchan-co
Merge pull request#12from seichan23/feature/update_design
d54194b
デザイン周りを少し更新
seichan23seichan23
READMEを作成
4b20b84
@chanchan-co
Merge pull request#13from seichan23/docs/generate_readme
ed5a051
READMEを作成
seichan23seichan23
テキスト修正
2e8d2b9
@chanchan-co
Merge pull request#14from seichan23/fix
c438eda
テキスト修正
cursor[bot]

This comment was marked as outdated.

Sign in to view
@chanchan-co
chanchan-co
commented
Oct 29, 2025

Copy link


Copy Markdown

Author
追記(10月30日)
ER図とアーキテクチャ構成図を作成させていただきましたので追記いたします。
ER図
erDiagram
Provider {
int id PK
string name
string code UK
}
Plan {
int id PK
int provider_id FK
string name
string code UK "※ provider_id + code の複合ユニーク制約"
}
BasicCharge {
int id PK
int plan_id FK
int ampere "※ plan_id + ampere の複合ユニーク制約"
decimal amount
}
UsageCharge {
int id PK
int plan_id FK
int from_kwh "※ plan_id + from_kwh の複合ユニーク制約"
int to_kwh
decimal unit_price
}
Provider ||--o{ Plan : "has many"
Plan ||--o{ BasicCharge : "has many"
Plan ||--o{ UsageCharge : "has many"

Loading
アーキテクチャ構成図
スクリーンショット 2025-10-29 23 51 20
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025

Copy link


Copy Markdown

お忙しい中チャレンジの提出ありがとうございました。

バックエンドだけでなくフロントエンド、クラウドへのデプロイまでご対応いただきありがとうござます。

また段階的な計算のロジックも正しく実装されていることを確認しました。

いくつか実装に関するコメントをさせていただきます。
ksugiura-ene
ksugiura-ene
reviewed
Oct 30, 2025
View reviewed changes
Comment threadserverside_challenge_2/challenge/api/config/environments/development.rb
Outdated

 # Uncomment if you wish to allow Action Cable access from any origin.
 # config.action_cable.disable_request_forgery_protection = true

 config.middleware.insert_before 0, Rack::Cors do
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
本番環境と共通化できそうですが環境ごとに指定している意図はありますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

開発環境で設定した内容がそのまま本番環境に反映されることを避けるため、

段階的にdevelopment/ productionで設定を分けて行いましたが

最終的に共通化するのを失念していました。
内容的に共通化可能なため、initializers/cors.rbにまとめる形で修正したいと考えております。
Comment threadserverside_challenge_2/challenge/api/config/locales/ja.yml
Outdated

 from_kwh: 使用量(kwh)の下限
 to_kwh: 使用量(kwh)の上限
 unit_price: 単価
 date:
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q
今回の実装で日付関係の要素はなかったかと思いますが、local定義をしている意図があれば教えていただけますでしょうか。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

date等のlocale定義につきましては、rails-i18nのja.ymlをベースファイルとして使用したため、

今回の実装では使用していませんが、そのまま残しております。
理由:
  • 将来的に日付フォーマットが必要になった際にすぐ使える
  • Railsの標準的なlocaleファイル構成を維持できる
今回の実装で必要だったわけではありませんが、ベースファイルの構成を崩さない方が良いと判断しました。
ただ、使用していない定義を残していると複雑性が増すとも考えられますので、削除させていただきたく考えております。
Comment threadserverside_challenge_2/challenge/api/db/csv/plans.csv
Outdated

@@ -0,0 +1,5 @@
provider_id,name,code
1,従量電灯B,juuryouB
@ksugiura-eneksugiura-eneOct 29, 2025

Copy link


Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NITS
いくつかのcsvでprovider_idやplan_idが参照されていますが運用上の管理を考えますと、idでの管理は難しいと感じます。
具体的には

plan.csvでまだ登録されていないproviderのidを予想して指定している点が難しく感じました。

DBの自動採番はDBの再起動などで現在値が飛ぶ場合もあるので、マスターが壊れてしまう原因になりうると思いました。
@chanchan-cochanchan-coOct 30, 2025

Copy link


Copy Markdown

Author
Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご指摘ありがとうございます。

こちらのCSV登録機能に関しましてはrails db:migrate:reset db:seed前提で考えてしまっておりましたので

関連する要素のidを指定して登録するようにしておりました。
ただ、運用上の管理を考えると仰られるとおり、マスターが壊れてしまう原因になりうると考えられるため
idを予想して指定するのではなく一意のcodeを指定するように修正させていただきます。
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

Q
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
CloudSqlの方がCloudRunからの接続が容易だと思いますが

Supabaseを選択された理由があれば教えていただけますでしょうか。
仰られる通り、Cloud SQLの方がCloud Runからの接続が容易だと認識しております。

その上でSupabaseを選択した理由といたしましてはコスト面を考慮させていただきました。
Supabaseは無料枠で利用でき、PostgreSQL互換かつ構築も容易なため、今回の要件を満たすのには十分と考えました。

また、要件や運用方針によってはCloud SQLへのデータ移行も可能なため、Supabaseを選択いたしました。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

お忙しい中、

レビュー及びコメントをしていただき、ありがとうございます。
頂いたコメントに対してご回答させていただいておりますので

お手数をお掛けいたしますが、ご確認よろしくお願いいたします🙇‍♂️
@ksugiura-ene
ksugiura-ene
commented
Oct 30, 2025•

edited

Loading

Copy link


Copy Markdown

お忙しい中ご回答ありがとうございます。

私としては納得が行くご回答いただいていますので、対応はできる範囲でよろしくお願いいたします。
@chanchan-co
chanchan-co
commented
Oct 30, 2025

Copy link


Copy Markdown

Author
@ksugiura-ene

ご確認ありがとうございました。

コメントを頂いた箇所の修正をさせていただきプッシュさせていただきました。

よろしくお願いいたします。
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@cursorcursor[bot]cursor[bot] left review comments

+1 more reviewer

@ksugiura-eneksugiura-eneksugiura-ene left review comments
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

2 participants

@chanchan-co@ksugiura-ene