Jquery if textbox>；0

所以我对js很糟糕，我正在尝试获取一个jquery脚本，来onblur检查文本框是否有内容，如果有，然后继续执行脚本，这就是我所拥有的

<script type="text/javascript">
    $(document).ready(function() {
        $("#embed").blur(function (){
            $("#embed").val().length > 0){
                $("#fade").fadeIn(1000);olink = $("#embed").val(); 
                $("#oembed").oembed(olink, {
                    embedMethod: "append", 
                    maxWidth: 300, 
                    maxHeight: 150 
                });
            });
        }
    });
</script> 

$（文档）.ready（函数（）{
$（“#嵌入”）.blur（函数（）{
$（“#嵌入”）.val（）.length>0）{
$（“#淡入”）.fadeIn（1000）；olink=$（“#嵌入”）.val（）；
$（“#oembed”）。oembed（奥林克{
嵌入方法：“追加”，
最大宽度：300，
最大高度：150
});
});
}
});

但是我在读到的行中出现了一个语法错误（dreamweaver）

$（“#嵌入”）.val（）.length>0）{

---

任何建议都很好…谢谢

您似乎没有if语句，请尝试：

$(document).ready(function() {
    $("#embed").blur(function () {
        if($("#embed").val().length > 0) {
            $("#fade").fadeIn(1000);
            olink = $("#embed").val(); 
            $("#oembed").oembed(olink, {
                embedMethod: "append", 
                maxWidth: 300, 
                maxHeight: 150 
            });
         }
    });
});

编辑 您可以使代码更简洁，如下所示：

$(document).ready(function() {
    $("#embed").blur(function() {
        var $this = $(this);
        var embed_val = $this.val();
        if(embed_val.length) {
            $("#fade").fadeIn(1000);
            $("#oembed").oembed(embed_val, {
                embedMethod: "append", 
                maxWidth: 300, 
                maxHeight: 150 
            });
         }
    });
});

$（文档）.ready（函数（）{
$（“#嵌入”）.blur（函数（）{
如果（$（“#嵌入”）.val（）.length>0）{
美元("褪色")；
olink=$（“#嵌入”）.val（）；
$（“#oembed”）。oembed（奥林克{
嵌入方法：“追加”，
最大宽度：300，
最大高度：150
});
}
});
});

---

这就应该做到了。

这就是您的代码应该是什么样子（当然，在我看来），格式正确：

$(document).ready(function() {
    var $embed = $("#embed"),
        $fade = $('#fade'),
        $oembed = $('#oembed'),
        oembed = {
            embedMethod: "append",
            maxWidth: 300,
            maxHeight: 150
        },
        olink;

    $embed.blur(function(){
        if ($embed.val().length){
            $fade.fadeIn(1000);

            olink = $embed.val(); 
            $oembed.oembed(olink, oembed);
        }
    });
});

你发布的代码有什么问题，除了一堆混乱？你的

if（）

不在那里。你把

if（

out:

$(document).ready(function() {
    $("#embed").blur(function (){
        $("#embed").val().length > 0){

你能看到吗？很难区分这三行。现在看看上面的代码，它

正确使用选择器缓存（

var$embed=$（“#embed”）

）

使用

var

声明所有变量，用

，

分隔，因此它们都是本地变量；而且

使

olink

成为一个适当的非全局作用域变量（嘿，由于闭包的存在，仍然可以工作，多么漂亮）

将

oembed

对象（我从作为参数传递的

{}

创建）移动到适当的变量中，清理代码并缓存它

---

我认为，如果您在格式化代码时非常小心，并决定使用缓存和非全局变量等最佳实践，那么阅读和发现问题就更容易了（是最直接的问题，但整个代码需要一些关注和一点关爱。

Stellar，工作很好，谢谢……我错过了if，因为（就像我说的）我很糟糕……我看到了另一篇帖子（）这是lead小姐me@Luke，当您习惯于考虑周全的代码约定时，会有实质性的区别。在这种情况下，它会更容易发现您的问题。这就是为什么它是“最佳实践”，但不是强制要求。@Luke，无论您是否同意我的交付（这很好，是/是油嘴滑舌的），花一点时间读一读我写的东西。我不在乎我是否得到了一张赞成票或其他什么；我提供的答案实际上是为了给你提供一些真正有价值的东西，而不仅仅是包扎你所拥有的东西。我意识到你是个新手，我知道我做的比我想承认的要长（闭包曾经让我感到困惑）。所以，FWIW.the）在你的0不匹配之后，我很难过，到目前为止，没有一个答案试图展示一种“更好”的代码编写方式。上面的内容一团糟。我很难过，你的老师从来没有告诉过你“更好”是错误的……尽管如此，我还是说我“对js很糟糕”我正在学习…我不在乎它是不是一团糟…我理解我那里有什么，它是有效的…有人应该告诉你有很多不同的编码方式…这都取决于偏好和习惯，但感谢你的评论。我确实补充了一个答案；我很难过其他参与者认为没有必要做出更大的努力。这取决于你如何处理你的代码；这取决于我如何倡导我知道是有益的最佳实践。如果你是一个生活在美好时光中的角色，“更好”是正确的英语…）很抱歉，我说我在js方面很糟糕……我不知道这意味着什么……在发布之前，我确实试图弄乱括号和方括号，但没有成功，你的解决方案可能会奏效，但我把这个方案给了JakeRow123，因为它是一个像我这样的js新手能够理解的术语

$(document).ready(function() {
    $("#embed").blur(function (){
        $("#embed").val().length > 0){