アプリ自体のロジックには関わらないものをまとめるフォルダが無い

su-its / ams-backend

:briefcase: (This repository is no longer maintained) The backend server of our Access-management-system.

MIT License

0 stars 0 forks source link

Closed h-takeyeah closed 2 years ago

h-takeyeah commented 3 years ago

自分はルートフォルダにはエントリポイント以外のtsはない方がいいと思います。今後こういう「アプリ自体のロジックには関わらないもの」が追加される時のことを考えると、app以下にutilsとかmiscみたいなフォルダを掘ってそこに入れておくと良いかもしれません。

h-takeyeah commented 3 years ago

流れとしてはこんな感じ

今のところそのフォルダにはconfig.tsを入れたいという需要しかないので，こんな感じをイメージしています．

app/
  controllers/
  database/
  models/
  routes/
  <新フォルダ>/
    config.ts

h-takeyeah commented 3 years ago

名称の候補はutil，misc，configあたりでしょうか

h-takeyeah commented 3 years ago

新フォルダをapp/に配置したのはアプリ自体のロジックには関わらないとはいえ，app/以下の既存のファイルからしか参照されないフォルダだと認識しているためです．もし本当に全く関係のないファイルを置きたい，というニーズがあったらちょっと考えます

ghost commented 3 years ago

@h-takeyeah 割と冗談抜きでローンチ直前にリファクタリングするのやめてください、責任取れるなら構いせんけど...

気持ちは分からんくもないのですが、今まで動いてたのは確証があるのです。ディレクトリとは言えそれなりに改修加えて可読性上げましたって言われても、喜ぶのは開発者であって、エンドユーザではないんです。

教授がめちゃｸｿできるPJならその方が良いんですが、教授が払った1万円というコストを無視あるいは責任取れるなら、ご自由にやればよいと思います。

まあ、所詮学生のやった戯言、で済ませてもらえるならそもそもこれ以上私関わりたくないんですけどね... (双極性障害がかなり悪化してて手が震えてる)