-
Notifications
You must be signed in to change notification settings - Fork 10
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
[enhancement] Bind the this context to the methods on FormState and FieldState #27
Comments
@yinxulai 绝大部分情况下我们不需要手工去做绑定的事情,那样往往会要求我们直接接触 从这个角度考虑,这里提到的提前 bind |
|
@yinxulai 为啥不应该..
工具做得越少,使用者越自由;工具做得越多,使用者可能越便利;这边是在权衡:
JavaScript 语言就是不默认 bind 的,这个不能说是用 formstate-x 带来的“额外的检查或保护”;另外自己做
如上所说,有收益有代价;一般来说,增加 feature 这样的事情,都是默认不做(因为可以去做的 feature 太多了),有充分的理由才去做
formstate 预期 不过这点确实导致了不一致,我想一下(考虑到 |
仔细想想这个 其他几个,我的主要观点是认为:仅从 |
嗯所以是少量自由
主要的代价不是“无法选择不
因为我在项目里几乎没有直接 如果你的项目里确实存在不止一处这样的 case,你可以提个 PR 来加下
这事儿可能本身是有问题的 |
因为并不是所有的
FormState
使用场景中消费者都是标准的 Form 组件,所以不一定有、且不一定需要有对应的bindXX
方法,就会碰到下面的问题:当前的实现中,未对
FormState
andFieldState
上面的方法绑定this
上下文,从而当如上使用时,会发生错误.例如这些地方:
formstate-x/src/formState.ts
Line 139 in 0fb4603
formstate-x/src/formState.ts
Line 168 in 0fb4603
我们是否应该为这些方法绑定所在的
this
上下文?The text was updated successfully, but these errors were encountered: