在 jQuery 中垂直排列代码是否被认为是有害的?
Is vertically lining up code in jQuery considered harmful?
此代码:
if (this.config.imageTarget) {
this.caption = $('<div />')
.addClass('finaff-lightbox-caption')
.html(content.html())
.appendTo(this.wrapper)
.hide(); // starts out hidden
this.captionSpacer = $('<div> </div>')
.appendTo(this.contentContainer)
.hide();
...导致 JSLint 崩溃。它说:
if (this.config.imageTarget) {
58 this.caption = $('<div />')
59 .addClass('finaff-lightbox-caption')
==========================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
60 .html(content.html())
===================================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
61 .appendTo(this.wrapper)
===================================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
62 .hide();
是否真的有必要像这样将这类代码折叠在一起:
. . .
this.caption = $('<div />').addClass('finaff-lightbox-caption').html(content.html()).appendTo(this.wrapper).hide();
. . .
?
通过排列点将其分开对我来说很有意义;使 read/grok 更容易。希望这只是JSLint扮演的白手套婆婆
您所要做的就是将 white
选项(对于空格)设置为 true。
此代码在 JSLint.com:
处检查
/*jslint white:true */
/*global $, content */
if (this.config.imageTarget) {
this.caption = $('<div />')
.addClass('finaff-lightbox-caption')
.html(content.html())
.appendTo(this.wrapper)
.hide(); // starts out hidden
this.captionSpacer = $('<div> </div>')
.appendTo(this.contentContainer)
.hide();
}
我所做的唯一更改是:
- 将 jslint 指令
white
设置为 true
(容忍非严苛的空白规则)
- 为 jQuery 和
content
添加了 global
行
- 不确定
content
是什么;
- 注意:使用
global
几乎可以肯定 不是 处理 content
的最佳方式
- 为您的
if
添加了右括号 }
。
此代码:
if (this.config.imageTarget) {
this.caption = $('<div />')
.addClass('finaff-lightbox-caption')
.html(content.html())
.appendTo(this.wrapper)
.hide(); // starts out hidden
this.captionSpacer = $('<div> </div>')
.appendTo(this.contentContainer)
.hide();
...导致 JSLint 崩溃。它说:
if (this.config.imageTarget) {
58 this.caption = $('<div />')
59 .addClass('finaff-lightbox-caption')
==========================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
60 .html(content.html())
===================================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
61 .appendTo(this.wrapper)
===================================^
lint warning: unexpected end of line; it is ambiguous whether these lines are part of the same statement
62 .hide();
是否真的有必要像这样将这类代码折叠在一起:
. . .
this.caption = $('<div />').addClass('finaff-lightbox-caption').html(content.html()).appendTo(this.wrapper).hide();
. . .
?
通过排列点将其分开对我来说很有意义;使 read/grok 更容易。希望这只是JSLint扮演的白手套婆婆
您所要做的就是将 white
选项(对于空格)设置为 true。
此代码在 JSLint.com:
处检查/*jslint white:true */
/*global $, content */
if (this.config.imageTarget) {
this.caption = $('<div />')
.addClass('finaff-lightbox-caption')
.html(content.html())
.appendTo(this.wrapper)
.hide(); // starts out hidden
this.captionSpacer = $('<div> </div>')
.appendTo(this.contentContainer)
.hide();
}
我所做的唯一更改是:
- 将 jslint 指令
white
设置为true
(容忍非严苛的空白规则) - 为 jQuery 和
content
添加了global
行- 不确定
content
是什么; - 注意:使用
global
几乎可以肯定 不是 处理content
的最佳方式
- 不确定
- 为您的
if
添加了右括号}
。