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

Fix data mapper #19

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

cocoa-maemae
Copy link
Contributor

変更点は以下の3点です。

  • to use these mechanisms to better organize the data and the behavior that goes with itの不定詞の「〜するために」という訳を前から順番に訳す流れに変更しました。
  • no knowledgeにneedが掛かっているように見えるので、need no knowledgeの文脈にしました。
  • Data Mapper itself is even unknownのevenの訳が抜けているように見えたので追加しました。

@kdmsnr
Copy link
Member

kdmsnr commented Sep 25, 2020

基本的に良いと思いますが、コンフリクトしているみたいなので修正してもらえればと。

ついでに、ちょっと表現が気になったので、以下も修正してもらえると非常に助かります。

  1. これらのメカニズムを使って、データおよびデータに付随する振る舞いをうまくまとめることが重要である。
  2. DataMapperを使うと、メモリ内のオブジェクトはデータベースの存在すら知る必要がない。
  3. ままでよいです

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants