Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【システム】単純に西暦年月日→ 和暦年月日の変換をしたい場合のヘルパ、メソッドが見当たらない #1190

Open
kaburk opened this issue May 10, 2019 · 1 comment
Labels
Review レビュー要 Ver4 バージョン4

Comments

@kaburk
Copy link
Collaborator

kaburk commented May 10, 2019

概要

BcTextHelper、BcTimeHelper で和暦の新元号に対応していますが、単純に西暦年月日→ 和暦年月日の変換するためは複数のメソッドを呼び出す必要があるようでしたので、BlogのViewなどで簡単に変換できるヘルパーがあると良いと思われます。

例えば、BcTextHelper::dateTimeWareki が近い実装なのですが、
年月日前後に余分なスペースが入ってしまったり、1年→元年の対応が無いようでした。

TODO

  • BcTextHelper::dateTimeWareki を拡張するべきかどうか。
  • 新たにヘルパー、メソッドを作る場合は、BcTextHelper、BcTimeHelper どちらにどのようなメソッド名で作ったらわかりやすいのか。
  • BlogHelperのgetPostDateの引数で和暦変換できるともっとよいのかも。

備考

  • 既に同様の実装がありましたらすみません。
@ryuring
Copy link
Collaborator

ryuring commented May 12, 2019

@kaburk コア処理は、BcTimeHelper::convertToWarekiArray() にまとめてしまった方が良さそうですね。そして、BcTextHelper::dateTimeWareki() でラッピングする。

  • BcTimeHelper::convertToWarekiArray():年号の処理や、1年→元年対応もここにまとめる。配列から配列への変換を行う。
  • BcTextHelper::dateTimeWareki():文字列から文字列への変換(配列の引数も後方互換のため残しておく)内部的に、BcTimeHelper::convertToWarekiArray() を、オプションに応じて呼び出し文字列結合する
  • BcTimeHelper::convertToWareki()BcTextHelper::dateTimeWareki()に処理が近いが、削除はせず後方互換のために残しておく
  • BlogHelperのgetPostDateの引数で和暦変換:オプションで変換できるようにする

@ryuring ryuring added the Review レビュー要 label May 12, 2019
@ryuring ryuring changed the title 単純に西暦年月日→ 和暦年月日の変換をしたい場合のヘルパ、メソッドが見当たらない 【システム】単純に西暦年月日→ 和暦年月日の変換をしたい場合のヘルパ、メソッドが見当たらない May 5, 2020
@ryuring ryuring added the Ver4 バージョン4 label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Review レビュー要 Ver4 バージョン4
Projects
None yet
Development

No branches or pull requests

2 participants