Angular reactive form validation with dynamically named controls
up vote
1
down vote
favorite
In my Angular 7 app using reactive forms I'm creating input
elements based on an *ngFor
loop, so I end up with an input dynamically named:
<nav class="level" *ngFor="let work of workLeft">
<input [formControlName]="work.abbrev">
which of course works fine, but now I'm trying to add the validation error messages to the form, but I'm not sure how to "address" the item. For example, the div would normally look like so:
<div *ngIf="name.errors.required">
but I don't have name
there as it's the dynamic work.abbrev
value. What's the right way to handle this?
You can see my attempt here: https://stackblitz.com/edit/angular-8zevc1
angular angular-reactive-forms angular2-form-validation
|
show 2 more comments
up vote
1
down vote
favorite
In my Angular 7 app using reactive forms I'm creating input
elements based on an *ngFor
loop, so I end up with an input dynamically named:
<nav class="level" *ngFor="let work of workLeft">
<input [formControlName]="work.abbrev">
which of course works fine, but now I'm trying to add the validation error messages to the form, but I'm not sure how to "address" the item. For example, the div would normally look like so:
<div *ngIf="name.errors.required">
but I don't have name
there as it's the dynamic work.abbrev
value. What's the right way to handle this?
You can see my attempt here: https://stackblitz.com/edit/angular-8zevc1
angular angular-reactive-forms angular2-form-validation
IsworkLeft
some sort of aFormArray
? If not, I recommend it to be one. That way you can create aget
ter on your Component Class and use theat
API on aFormArray
to get the relevantFormControl
/FormGroup
– SiddAjmera
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
1
That's becauseFormArray
is something that is generally used in such scenarios. You want to show validation errors for each item inworkLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.
– SiddAjmera
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago
|
show 2 more comments
up vote
1
down vote
favorite
up vote
1
down vote
favorite
In my Angular 7 app using reactive forms I'm creating input
elements based on an *ngFor
loop, so I end up with an input dynamically named:
<nav class="level" *ngFor="let work of workLeft">
<input [formControlName]="work.abbrev">
which of course works fine, but now I'm trying to add the validation error messages to the form, but I'm not sure how to "address" the item. For example, the div would normally look like so:
<div *ngIf="name.errors.required">
but I don't have name
there as it's the dynamic work.abbrev
value. What's the right way to handle this?
You can see my attempt here: https://stackblitz.com/edit/angular-8zevc1
angular angular-reactive-forms angular2-form-validation
In my Angular 7 app using reactive forms I'm creating input
elements based on an *ngFor
loop, so I end up with an input dynamically named:
<nav class="level" *ngFor="let work of workLeft">
<input [formControlName]="work.abbrev">
which of course works fine, but now I'm trying to add the validation error messages to the form, but I'm not sure how to "address" the item. For example, the div would normally look like so:
<div *ngIf="name.errors.required">
but I don't have name
there as it's the dynamic work.abbrev
value. What's the right way to handle this?
You can see my attempt here: https://stackblitz.com/edit/angular-8zevc1
angular angular-reactive-forms angular2-form-validation
angular angular-reactive-forms angular2-form-validation
edited 8 hours ago
asked 9 hours ago
Gargoyle
2,63842647
2,63842647
IsworkLeft
some sort of aFormArray
? If not, I recommend it to be one. That way you can create aget
ter on your Component Class and use theat
API on aFormArray
to get the relevantFormControl
/FormGroup
– SiddAjmera
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
1
That's becauseFormArray
is something that is generally used in such scenarios. You want to show validation errors for each item inworkLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.
– SiddAjmera
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago
|
show 2 more comments
IsworkLeft
some sort of aFormArray
? If not, I recommend it to be one. That way you can create aget
ter on your Component Class and use theat
API on aFormArray
to get the relevantFormControl
/FormGroup
– SiddAjmera
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
1
That's becauseFormArray
is something that is generally used in such scenarios. You want to show validation errors for each item inworkLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.
– SiddAjmera
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago
Is
workLeft
some sort of a FormArray
? If not, I recommend it to be one. That way you can create a get
ter on your Component Class and use the at
API on a FormArray
to get the relevant FormControl
/FormGroup
– SiddAjmera
9 hours ago
Is
workLeft
some sort of a FormArray
? If not, I recommend it to be one. That way you can create a get
ter on your Component Class and use the at
API on a FormArray
to get the relevant FormControl
/FormGroup
– SiddAjmera
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
1
1
That's because
FormArray
is something that is generally used in such scenarios. You want to show validation errors for each item in workLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.– SiddAjmera
9 hours ago
That's because
FormArray
is something that is generally used in such scenarios. You want to show validation errors for each item in workLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.– SiddAjmera
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago
|
show 2 more comments
1 Answer
1
active
oldest
votes
up vote
0
down vote
accepted
I suggest using FormArray
for this. With FormArray
, here's how your implementation is going to look like:
For the Component Class:
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators, FormArray } from '@angular/forms';
export interface Data {
abbrev: string;
max: number;
}
@Component({
selector: 'my-app',
templateUrl: './app.component.html',
styleUrls: [ './app.component.css' ]
})
export class AppComponent {
workForm: FormGroup;
workLeft: any;
constructor(private fb: FormBuilder) {}
ngOnInit () {
this.workForm = this.fb.group({
points: this.fb.array()
});
this.fillFormArray();
}
private fakeWebserviceCall(): Data {
return [
{ abbrev: 'foo', max: 12 },
{ abbrev: 'bar', max: 10 }
];
}
private fillFormArray() {
this.workLeft = this.fakeWebserviceCall();
const formControlsArray = this.workLeft.map(work => this.fb.control(work.abbrev, [Validators.min(0), Validators.max(work.max)]));
formControlsArray.forEach(control => this.points.push(control));
console.log(this.workForm.value);
}
get points(): FormArray {
return <FormArray>this.workForm.get('points');
}
pointAt(index) {
return (<FormArray>this.workForm.get('points')).at(index);
}
}
And in the template:
<form [formGroup]="workForm">
<div formArrayName="points">
<div *ngFor="let point of points.controls; let i = index">
{{ workLeft[i].abbrev }}: <input type="number" [formControlName]="i">
<div *ngIf="pointAt(i).invalid && (pointAt(i).dirty || pointAt(i).touched)">
The field is invalid
</div>
</div>
</div>
</form>
Here's a Sample StackBlitz for your ref.
PS: I've made a few updates to the StackBlitz that you've shared including things that Angular Style Guide recommends along with the actual solution. Hope that helps.
Thanks, appreciate the help!
– Gargoyle
7 hours ago
add a comment |
1 Answer
1
active
oldest
votes
1 Answer
1
active
oldest
votes
active
oldest
votes
active
oldest
votes
up vote
0
down vote
accepted
I suggest using FormArray
for this. With FormArray
, here's how your implementation is going to look like:
For the Component Class:
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators, FormArray } from '@angular/forms';
export interface Data {
abbrev: string;
max: number;
}
@Component({
selector: 'my-app',
templateUrl: './app.component.html',
styleUrls: [ './app.component.css' ]
})
export class AppComponent {
workForm: FormGroup;
workLeft: any;
constructor(private fb: FormBuilder) {}
ngOnInit () {
this.workForm = this.fb.group({
points: this.fb.array()
});
this.fillFormArray();
}
private fakeWebserviceCall(): Data {
return [
{ abbrev: 'foo', max: 12 },
{ abbrev: 'bar', max: 10 }
];
}
private fillFormArray() {
this.workLeft = this.fakeWebserviceCall();
const formControlsArray = this.workLeft.map(work => this.fb.control(work.abbrev, [Validators.min(0), Validators.max(work.max)]));
formControlsArray.forEach(control => this.points.push(control));
console.log(this.workForm.value);
}
get points(): FormArray {
return <FormArray>this.workForm.get('points');
}
pointAt(index) {
return (<FormArray>this.workForm.get('points')).at(index);
}
}
And in the template:
<form [formGroup]="workForm">
<div formArrayName="points">
<div *ngFor="let point of points.controls; let i = index">
{{ workLeft[i].abbrev }}: <input type="number" [formControlName]="i">
<div *ngIf="pointAt(i).invalid && (pointAt(i).dirty || pointAt(i).touched)">
The field is invalid
</div>
</div>
</div>
</form>
Here's a Sample StackBlitz for your ref.
PS: I've made a few updates to the StackBlitz that you've shared including things that Angular Style Guide recommends along with the actual solution. Hope that helps.
Thanks, appreciate the help!
– Gargoyle
7 hours ago
add a comment |
up vote
0
down vote
accepted
I suggest using FormArray
for this. With FormArray
, here's how your implementation is going to look like:
For the Component Class:
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators, FormArray } from '@angular/forms';
export interface Data {
abbrev: string;
max: number;
}
@Component({
selector: 'my-app',
templateUrl: './app.component.html',
styleUrls: [ './app.component.css' ]
})
export class AppComponent {
workForm: FormGroup;
workLeft: any;
constructor(private fb: FormBuilder) {}
ngOnInit () {
this.workForm = this.fb.group({
points: this.fb.array()
});
this.fillFormArray();
}
private fakeWebserviceCall(): Data {
return [
{ abbrev: 'foo', max: 12 },
{ abbrev: 'bar', max: 10 }
];
}
private fillFormArray() {
this.workLeft = this.fakeWebserviceCall();
const formControlsArray = this.workLeft.map(work => this.fb.control(work.abbrev, [Validators.min(0), Validators.max(work.max)]));
formControlsArray.forEach(control => this.points.push(control));
console.log(this.workForm.value);
}
get points(): FormArray {
return <FormArray>this.workForm.get('points');
}
pointAt(index) {
return (<FormArray>this.workForm.get('points')).at(index);
}
}
And in the template:
<form [formGroup]="workForm">
<div formArrayName="points">
<div *ngFor="let point of points.controls; let i = index">
{{ workLeft[i].abbrev }}: <input type="number" [formControlName]="i">
<div *ngIf="pointAt(i).invalid && (pointAt(i).dirty || pointAt(i).touched)">
The field is invalid
</div>
</div>
</div>
</form>
Here's a Sample StackBlitz for your ref.
PS: I've made a few updates to the StackBlitz that you've shared including things that Angular Style Guide recommends along with the actual solution. Hope that helps.
Thanks, appreciate the help!
– Gargoyle
7 hours ago
add a comment |
up vote
0
down vote
accepted
up vote
0
down vote
accepted
I suggest using FormArray
for this. With FormArray
, here's how your implementation is going to look like:
For the Component Class:
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators, FormArray } from '@angular/forms';
export interface Data {
abbrev: string;
max: number;
}
@Component({
selector: 'my-app',
templateUrl: './app.component.html',
styleUrls: [ './app.component.css' ]
})
export class AppComponent {
workForm: FormGroup;
workLeft: any;
constructor(private fb: FormBuilder) {}
ngOnInit () {
this.workForm = this.fb.group({
points: this.fb.array()
});
this.fillFormArray();
}
private fakeWebserviceCall(): Data {
return [
{ abbrev: 'foo', max: 12 },
{ abbrev: 'bar', max: 10 }
];
}
private fillFormArray() {
this.workLeft = this.fakeWebserviceCall();
const formControlsArray = this.workLeft.map(work => this.fb.control(work.abbrev, [Validators.min(0), Validators.max(work.max)]));
formControlsArray.forEach(control => this.points.push(control));
console.log(this.workForm.value);
}
get points(): FormArray {
return <FormArray>this.workForm.get('points');
}
pointAt(index) {
return (<FormArray>this.workForm.get('points')).at(index);
}
}
And in the template:
<form [formGroup]="workForm">
<div formArrayName="points">
<div *ngFor="let point of points.controls; let i = index">
{{ workLeft[i].abbrev }}: <input type="number" [formControlName]="i">
<div *ngIf="pointAt(i).invalid && (pointAt(i).dirty || pointAt(i).touched)">
The field is invalid
</div>
</div>
</div>
</form>
Here's a Sample StackBlitz for your ref.
PS: I've made a few updates to the StackBlitz that you've shared including things that Angular Style Guide recommends along with the actual solution. Hope that helps.
I suggest using FormArray
for this. With FormArray
, here's how your implementation is going to look like:
For the Component Class:
import { Component } from '@angular/core';
import { FormBuilder, FormGroup, Validators, FormArray } from '@angular/forms';
export interface Data {
abbrev: string;
max: number;
}
@Component({
selector: 'my-app',
templateUrl: './app.component.html',
styleUrls: [ './app.component.css' ]
})
export class AppComponent {
workForm: FormGroup;
workLeft: any;
constructor(private fb: FormBuilder) {}
ngOnInit () {
this.workForm = this.fb.group({
points: this.fb.array()
});
this.fillFormArray();
}
private fakeWebserviceCall(): Data {
return [
{ abbrev: 'foo', max: 12 },
{ abbrev: 'bar', max: 10 }
];
}
private fillFormArray() {
this.workLeft = this.fakeWebserviceCall();
const formControlsArray = this.workLeft.map(work => this.fb.control(work.abbrev, [Validators.min(0), Validators.max(work.max)]));
formControlsArray.forEach(control => this.points.push(control));
console.log(this.workForm.value);
}
get points(): FormArray {
return <FormArray>this.workForm.get('points');
}
pointAt(index) {
return (<FormArray>this.workForm.get('points')).at(index);
}
}
And in the template:
<form [formGroup]="workForm">
<div formArrayName="points">
<div *ngFor="let point of points.controls; let i = index">
{{ workLeft[i].abbrev }}: <input type="number" [formControlName]="i">
<div *ngIf="pointAt(i).invalid && (pointAt(i).dirty || pointAt(i).touched)">
The field is invalid
</div>
</div>
</div>
</form>
Here's a Sample StackBlitz for your ref.
PS: I've made a few updates to the StackBlitz that you've shared including things that Angular Style Guide recommends along with the actual solution. Hope that helps.
answered 8 hours ago
SiddAjmera
9,98721137
9,98721137
Thanks, appreciate the help!
– Gargoyle
7 hours ago
add a comment |
Thanks, appreciate the help!
– Gargoyle
7 hours ago
Thanks, appreciate the help!
– Gargoyle
7 hours ago
Thanks, appreciate the help!
– Gargoyle
7 hours ago
add a comment |
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53400565%2fangular-reactive-form-validation-with-dynamically-named-controls%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Is
workLeft
some sort of aFormArray
? If not, I recommend it to be one. That way you can create aget
ter on your Component Class and use theat
API on aFormArray
to get the relevantFormControl
/FormGroup
– SiddAjmera
9 hours ago
No, it's just an array of objects that was returned from an http webservice. But it's not a class variable, it's just created in the call that generates the form data.
– Gargoyle
9 hours ago
I'm not sure how your comment helps though as I'm asking about how to deal with it in the HTML itself, and I specifically mentioned not wanting to use a FormArray.
– Gargoyle
9 hours ago
1
That's because
FormArray
is something that is generally used in such scenarios. You want to show validation errors for each item inworkLeft
and it again is a form control. Also, I don't think you will need to keep any mapping to track the index and the form control anywhere. That's not how it works.– SiddAjmera
9 hours ago
OK, if it's the right way so be it. Can you show me what the div's *ngIf should look like please?
– Gargoyle
9 hours ago