5

本质上，我需要做的是获取一个本地的Grader.js文件，然后在命令行中使用它来输入 HTML，然后将 JSON 数据输出到控制台以验证多个 HTML 元素的存在。用法如下所示：

./grader.js --checks checks.json --file index.html
./grader.js --checks checks.json --url http://google.com

使用的 Node 模块是 Commander（用于在命令行工作）、Cheerio（用于 HTML）和 Restler（用于从 URL 获取 HTML）。

checks.json文件很简单，因为它只是要求检查一些简单的 HTML 元素是否存在，以确定它们是否存在于页面上：

["h1",
 ".navigation",
 ".logo",
 ".blank",
 ".about",
 ".heading",
 ".subheading",
 ".pitch",
 ".video",
 ".thermometer",
 ".order",
 ".social",
 ".section1",
 ".section2",
 ".faq",
 ".footer"]

Grader.js文件是让事情变得更复杂的地方。下面的代码实际上可以工作，因为它接受命令行参数，并且确实指示了 HTML 元素是否存在的真值或假值。但是在底部添加了 URL 检查后就不能正常工作了。我的checkURL 函数有问题以及我使用底部的指挥官代码实现它的方式。尽管 true 和 false 值是否正确取决于我使用的 HTML 文件/URL，但即使我只想检查文件或 URL，而不是两者，我最终也会向控制台发出两个检查。我对此很陌生，所以我很惊讶它完全有效。它可能与默认值有关，但是当我尝试进行这些更改时，checkURL 函数似乎崩溃了。提前感谢您的帮助，我真的很感激。

#!/usr/bin/env node
var fs = require('fs');
var program = require('commander');
var cheerio = require('cheerio');
var rest = require('restler');
var HTMLFILE_DEFAULT = "index.html";
var CHECKSFILE_DEFAULT = "checks.json";
var URL_DEFAULT = "http://cryptic-spire-7925.herokuapp.com/index.html";

var assertFileExists = function(infile) {
    var instr = infile.toString();
    if(!fs.existsSync(instr)) {
        console.log("%s does not exist. Exiting.", instr);
        process.exit(1); // http://nodejs.org/api/process.html#process_process_exit_code
    }
    return instr;
};

var cheerioHtmlFile = function(htmlfile) {
    return cheerio.load(fs.readFileSync(htmlfile));
};

var loadChecks = function(checksfile) {
    return JSON.parse(fs.readFileSync(checksfile));
};

var checkHtmlFile = function(htmlfile, checksfile) {
    $ = cheerioHtmlFile(htmlfile);
    var checks = loadChecks(checksfile).sort();
    var out = {};
    for(var ii in checks) {
        var present = $(checks[ii]).length > 0;
        out[checks[ii]] = present;
    }
    return out;
};

var checkUrl = function(url, checksfile) {
    rest.get(url).on('complete', function(data) {
                $ = cheerio.load(data);
            var checks = loadChecks(checksfile).sort();
            var out = {};
            for(var ii in checks) {
                var present = $(checks[ii]).length > 0;
                out[checks[ii]] = present;
            }
            console.log(out);
        });
}

var clone = function(fn) {
    // Workaround for commander.js issue.
    // http://stackoverflow.com/a/6772648
    return fn.bind({});
};

if(require.main == module) {
    program
        .option('-f, --file <html_file>', 'Path to index.html', clone(assertFileExists), HTMLFILE_DEFAULT)
        .option('-u, --url <url>', 'URL to index.html', URL_DEFAULT)
        .option('-c, --checks <check_file>', 'Path to checks.json', clone(assertFileExists), CHECKSFILE_DEFAULT)
        .parse(process.argv);

    var checkJson = checkHtmlFile(program.file, program.checks);
    var outJson = JSON.stringify(checkJson, null, 4);
    console.log(outJson);

    var checkJson2 = checkUrl(program.url, program.checks);
    var outJson2 = JSON.stringify(checkJson2, null, 4);
    console.log(outJson2);

} 
else {
    exports.checkHtmlFile = checkHtmlFile;
}
4

3 回答 3

3

这是在@David 和 @ankitsabharwal 的有用提示之后更新的checkUrl函数。

var checkUrl = function(url, checksfile) {
    rest.get(url).on('complete', function(data) {
        $ = cheerio.load(data);
        var checks = loadChecks(checksfile).sort();
        var out = {};
        for(var ii in checks) {
            var present = $(checks[ii]).length > 0;
            out[checks[ii]] = present;
        }
        var outJson = JSON.stringify(out, null, 4);
        console.log(outJson);
    });
}

下面是更新的指挥官代码：

if(require.main == module) {
    program
        .option('-f, --file <html_file>', 'Path to index.html')
        .option('-u, --url <url>', 'URL to index.html')
        .option('-c, --checks <check_file>', 'Path to checks.json')
        .parse(process.argv);

    if (program.url) {
        checkUrl(program.url, program.checks);
    } else {
        checkHtmlFile (program.file, program.checks);

        var checkJson = checkHtmlFile(program.file, program.checks);
        var outJson = JSON.stringify(checkJson, null, 4);
        console.log(outJson);
    }
}
于 2013-07-11T16:06:49.073 回答
3

那些指挥官 .option 行在我看来是错误的。

删除克隆功能并修改您的选项行如下：

    .option('-f, --file <html_file>', 'Path to index.html', HTMLFILE_DEFAULT)
    .option('-u, --url <url>', 'URL to index.html', URL_DEFAULT)
    .option('-c, --checks <check_file>', 'Path to checks.json', CHECKSFILE_DEFAULT)

这应该可以解决您的指挥官问题。

于 2013-07-11T08:35:32.540 回答
3

根据参数调用checkHtmlFile()或checkUrl()

就像是： 

if (program.url)
   checkUrl(program.url, program.checks);
else checkHtmlFile(program.file, program.checks);

阅读本文以获取更多参考：commander.js 选项解析

此外，不checkJson2返回任何东西。undefinedcheckUrl()

于 2013-07-10T22:12:27.643 回答