我是否需要重构此 javascript 以实现条件形式逻辑?
我在 Laravel 中有一个表单,它遍历了多个模板(刀片)。对于每个模板,我都实现简单的条件逻辑来显示和隐藏问题组。
对于模板,我使用 HTML5 进行高效的客户端验证。我编写了下面的 javascript 来使用 HTML 属性显示和隐藏字段。
下面的示例通过暴露隐藏的单选按钮组来回答是/否问题。一组用于回答“是”,一组用于回答“否”。
我的问题是,我需要重构它吗?有没有更有效的方法我应该编写这个javascript,或者我做得还好吗?
我正在使用事件侦听器,以便点击可以冒泡,我知道这是最有效的。由于我有多个页面,因此我有多个事件监听器,每个模板一个。有更好的方法吗?
感谢您的意见。
// "Purpose of Notification" Blade
document.addEventListener('click', function (event) {
// When yes or no is choosen
if (event.target.matches('#waiver30100byes')) {
// show the patient level group container
document.getElementById("patient-level").removeAttribute("hidden");
// and show 100 level for yes and make it required
document.getElementById("patient-level-100").removeAttribute("hidden");
document.getElementById("level1").required = true;
// hide and un-require the 30 level radio group
document.getElementById("patient-level-30").hidden = true;
document.getElementById("level3").required = false;
}
else if (event.target.matches('#waiver30100bno')) {
// show the patient level group
document.getElementById("patient-level").removeAttribute("hidden");
// and show 30 level for no and make it required
document.getElementById("patient-level-30").removeAttribute("hidden");
document.getElementById("level3").required = true;
//hide and un-required the 100 level radio group
document.getElementById("patient-level-100").hidden = true;
document.getElementById("level1").required = false;
//
}
}, false);
I have a form in Laravel that goes over several template (blades). For each template I am implementing simple conditional logic to show and hide groups of questions.
For the templates, I'm using HTML5 for efficient client-side validation. I wrote the javascript below to show and hide fields using HTML attributes.
The example below responds to a yes/no question by exposing hidden sets of radio buttons. One set is for answering Yes, one is for No.
My question is, do I need to refactor this? Is there a more efficient way I should be writing this javascript, or am I doing ok?
I'm using an event listener so that clicks can bubble up, understanding that this is most efficient. Since I have more than one page, I have more than one event listener, one for each template. Is there a better way to do that?
Thanks for your input.
// "Purpose of Notification" Blade
document.addEventListener('click', function (event) {
// When yes or no is choosen
if (event.target.matches('#waiver30100byes')) {
// show the patient level group container
document.getElementById("patient-level").removeAttribute("hidden");
// and show 100 level for yes and make it required
document.getElementById("patient-level-100").removeAttribute("hidden");
document.getElementById("level1").required = true;
// hide and un-require the 30 level radio group
document.getElementById("patient-level-30").hidden = true;
document.getElementById("level3").required = false;
}
else if (event.target.matches('#waiver30100bno')) {
// show the patient level group
document.getElementById("patient-level").removeAttribute("hidden");
// and show 30 level for no and make it required
document.getElementById("patient-level-30").removeAttribute("hidden");
document.getElementById("level3").required = true;
//hide and un-required the 100 level radio group
document.getElementById("patient-level-100").hidden = true;
document.getElementById("level1").required = false;
//
}
}, false);
如果你对这篇内容有疑问,欢迎到本站社区发帖提问 参与讨论,获取更多帮助,或者扫码二维码加入 Web 技术交流群。

绑定邮箱获取回复消息
由于您还没有绑定你的真实邮箱,如果其他用户或者作者回复了您的评论,将不能在第一时间通知您!
发布评论