typescript オブジェクトの中身に値を入れたい

Question

addOffersの処理が呼ばれた時にオブジェクト値を追加しているのですが、
その時に追加するオブジェクトの値のisShowをtrueにしてあげたいです。
どうやったらいいでしょうか？
わかる人ご教授お願いしたいです。

userEntities: UserEntity[] = []

class UserEntity {
  userCode: string = ''
  userName: string = ''
  title: string = ''
  body: string = ''
  isShow: boolean = false

  constructor(user: User) {
    this.userCode =user.code
    this.userName = user.user.name
    this.title = user.title
    this.body = user.body
  }

  addOffers(users: user[]) {
    users.forEach((user) => {
      this.usersEntities = [...this.usersEntities, new UserEntity(user)]
    })
  }

Comments

[gentaro]

コードとして不完全すぎるので、まずは構文エラーが出ないコードに修正しましょう。

[cuku]

一部だけ書いているのですが全部載せないといけないのでしょうか？
今回のは
addOffers(users: user[]) {
users.forEach((user) => {
this.usersEntities = [...this.usersEntities, new UserEntity(user)]
})

[cuku]

上記のオブジェクトの処理が呼ばれるときに聞きたいことがあるので、必要のない箇所は書いていません

[gentaro]

コピペでエラーが出ない程度（出るとしても質問文の問題になっている箇所ぐらい）のコードにしましょう、という意味です。
例えばclassの{を閉じてないとか、型名のUserとuserが混在して現れているとか、そのUserという型がどういうメンバーを持っているのかを提示していないとか。
基本的な構文チェックすら通らないコードを載せているようだと、そもそも提示されたコード以外にも問題があるんじゃないかという話になるので、ちゃんと最小限のエラーが出ないコードを書きましょう。

[cuku]

いや、今回のはバグで動かないから教えてくださいというエラーではないのですがww
オブジェクトを展開して追加するやり方を教えて欲しいだけなので。
いちいちコメントしないで欲しいです。

[gentaro]

コメントは自由でしょ。
ガイドライン読んで出直してください。
https://teratail.com/help/question-tips

Answer 1

こんにちは
以下の修正でいかがでしょう？

(1) class UserEntity の constructor に省略可能な引数 isShow を追加して、this.isShow に代入

- constructor(user: User) {
+ constructor(user: User, isShow: boolean = false) {
    this.userCode = user.code
    this.userName = user.name
    this.title = user.title
    this.body = user.body
+   this.isShow = isShow
  }

(2) addOffers で、追加するUserEntityをnewで作るところで、引数に true を追加

- this.usersEntities = [...this.usersEntities, new UserEntity(user)]
+ this.usersEntities = [...this.usersEntities, new UserEntity(user, true)]

以下は上記のコードを試すサンプルです。

• 動作確認用サンプル: https://codepen.io/jun68ykt/pen/NWGOdOj?editors=0012

上記のサンプルでは、オブジェクト obj にusersEntitiesプロパティとaddOffersメソッドを持たせて、obj.addOffers で2つのUserを含む配列 [ userB, userC ]を渡しています。その結果、obj.usersEntities には2つの要素が追加されますが、それらの isShow は true になっていることを確認できます。

以上、参考になれば幸いです。

追記

別案を挙げます。以下のようにすれば、UserEntity のconstructorを修正せずに、addOffersのみの修正で済ませることもできます。

- this.usersEntities = [...this.usersEntities, new UserEntity(user)]
+ this.usersEntities = [...this.usersEntities, { ...new UserEntity(user), isShow: true }]

• 動作確認用サンプル: https://codepen.io/jun68ykt/pen/GRpYWqr?editors=0012

なお、追記した別案のほうのコードだと、型エラーにはならない(と思います。)ですが、追加されるオブジェクトが、UserEntityクラスのインスタンスではなく普通のオブジェクトになってしまうので、usersEntities の要素を UserEntityクラスのインスタンスに揃えたいのであれば、回答のはじめに書いたほうのコードのほうがよいかと思います。

Answer 2

以下でできるかと思います。

addOffers(users: user[]) {
  users.forEach((user) => {
    let userEntity new UserEntitiy(user)
    userEntity.isShow = true
    this.usersEntities = [...this.usersEntities, userEntity]
  })
}

ただし、全体像がわからないので何とも言い難いですが、このメソッドがUserEntityのクラスに含まれている設計は違和感を覚えます。(省略されているだけで実際はそうではない？)
クラス内のメソッドで自身のインスタンスを生成しているので今回のように、どのインスタンスの変数を修正すればいいのか分からなくなってしまっているのではないですか？
また、thisで指定するのも、スコープが分かりづらく、バグを埋め込む元なので、設計をやり直した方がいいかもしれません。