模板驱动表单未显示正确的值

Question

当我删除一个地址并添加一个新地址时，我遇到了一个错误。旧地址会被新地址覆盖。

我创建了一个小型演示应用程序来展示我的问题：

github：https://github.com/kolomu/HeroDemo
堆栈闪电战：https://stackblitz.com/edit/angular-gitter-5o6ydi

不是模型错了，不知怎么角度渲染的视图没有正确更新。

查看：

  <form #heroForm="ngForm">

    <div class="form-group">
      <div class="addresses" *ngFor="let address of model.addresses; let i=index">
        <label for="address-{{i}}">Address {{i}}</label>
        <input type="text" class="form-control" [(ngModel)]="address.location" name="location-{{i}}">
        <button class="btn btn-danger" (click)="remove(address)">Remove Address</button>
      </div>
    </div>

    <button class="btn btn-primary" (click)="add()">Add Address</button>

  </form>

组件类：

export class HeroFormComponent {

  model = new Hero(18, 'Dr IQ', [new Address(1,'main','Earth'), new Address(9001,'sub','Moon')]);

  remove(address) {
    const index = this.model.addresses.findIndex(
      address => address.id === address.id
    );

    this.model.addresses.splice(index, 1);
  }

  add(){
    this.model.addresses.push(new Address());
  }

}

数据模型：

export class Address {
    constructor(public id?: number, public type?:string, public location?: string) { }
}


export class Hero {

    constructor(
      public id: number,
      public name: string,
      public addresses: Address[]
    ) {  }

}

展示问题的 GIF 图片： https://gfycat.com/DelightfulSpiffyBetafish

Answer 1

一个问题是您对remove 方法的参数和findIndex 中的参数使用相同的名称address。因此，比较 address.id === address.id 会为第一个测试项目返回 true，因为它将项目与自身进行比较。

remove(address) {
  const index = this.model.addresses.findIndex(
    address => address.id === address.id          <-- Always true!
  );
  ...
}

您应该为findIndex中的参数使用不同的名称，例如_address（如this stackblitz所示）：

remove(address) {
  const index = this.model.addresses.findIndex(
    _address => _address.id === address.id
  );
  ...
}

或者您可以将索引作为参数传递给remove、as suggested by Embrioka。

---

在您的代码示例中，当您在删除第一个地址后添加新地址时，两个字段显示相同的数据（新地址的值）。这很奇怪，因为模板和代码中的一切看起来都很好。根据我的测试，问题在于新输入元素的名称与之前用于其他元素的名称相同。在您的模板中，如果您替换：

name="location-{{i}}"

与

name="location-{{address.id}}"

只要每个address.id 是唯一的，问题就会消失。显然，为不同的项目重复使用相同的名称会阻止 Angular 正确更新输入内容。

Answer 2

不要传递整个地址，只传递索引：

<div class="form-group">
      <div class="addresses" *ngFor="let address of model.addresses; let i=index">
        <label for="address-{{i}}">Address {{i}}</label>
        <input type="text" class="form-control" [(ngModel)]="address.location" name="location-{{i}}">
        <button class="btn btn-danger" (click)="remove(i)">Remove Address</button>
      </div>
</div>

在 ts 中：

remove(index) {
    this.model.addresses.splice(index, 1);
}

使用此解决方案，您甚至不必查找索引。

前面的评论者提到的解决方案的问题以及您的 id 将是未定义的，因此在 add 方法中只需定义一个 ID 或在查找索引时选择另一个属性！