TypeError：无法读取未定义的属性“父级”

Question

url = "www.w3schools.com/html/html5_video.asp"

VIDEO_LINKS = [];
VIDEO_LIST = [];

function fillVideo(callback) {
  request(url, function(err, res, body) {
    if (body) {
      $ = cheerio.load(body);
    }
    links = $('source');
    $(links).each(function(i, link) {
      var value = $(link).attr('src');
      if (value.slice(-3) == "mp4" ||
        value.slice(-4) == "webm" ||
        value.slice(-3) == "ogv") {
        VIDEO_LINKS.push(value);
        VIDEO_LIST.push($(link).text());

      }

    })
    callback();
  });
}

function writeVideo() {

  for (j = 0; j < VIDEO_LIST.length; j++) {
    request(VIDEO_LINKS[j]).pipe(fs.createWriteStream(VIDEO_LIST[j]));
  }
}

fillVideo(writeVideo);

//www.electronicinfo.ca/printable-pdfs

PDF_LINKS = [];
PDF_LIST = [];

function fillPDF(callback) {
  request(url, function(err, res, body) {
    $ = cheerio.load(body);
    links = $('a');
    $(links).each(function(i, link) {
      var value = $(link).attr('href');
      if (value.slice(-3) == "pdf") {
        PDF_LINKS.push(value);
        PDF_LIST.push($(link).text());
      }
    })
    callback();
  });
}

function writePDF() {
  for (j = 0; j < PDF_LIST.length; j++) {
    request(PDF_LINKS[j]).pipe(fs.createWriteStream(PDF_LIST[j]));
  }
}

fillPDF(writePDF);

嗨，这是以前可以工作的代码，从大约 5 分钟前开始，我几乎没有改变任何东西，我唯一改变的就是复制它，并更改变量名。我的问题是如何修复此代码？我知道错误是 body 是空的，但我不知道如何解决它，我会寻求任何帮助...

Answer 1

错误：

1. 由于未指定协议，URL 无效。我刚刚测试了你的代码，request 抛出了这个错误：

错误： URI“www.w3schools.com/html/html5_video.asp”无效。

1. 从<source> 元素中提取的链接是相对的。此link 将帮助您了解和解决问题。
2. <source> 元素本身没有文本。您必须使用网址中的Url.parse(src).pathname。
3. 错误处理和一些与变量相关的错误：未声明的变量、未初始化的变量和全局变量会污染全局范围并可能导致其他问题。

修复：

您应该为 url 指定协议（http、https、...）。

此外，您的代码需要进行一些清理（声明变量、删除全局变量……）：

var Url = require("url");
// ...

function fillVideo(url, callback) {
    request(url, function(err, res, body) {
        if(err) {
            return callback(err, null);                         // or throw an error if you like
        }

        var $ = cheerio.load(body),
            result = [];
        $('source').each(function() {
            var $this = $(this),
                src = $this.attr("src");
            if(/(?:mp4|webm|ogv)$/i.test(src)) {
                result.push({
                    url: Url.resolve(url, src),                 // make the url absolute
                    text: Url.parse(src).pathname
                });
            }
        });

        callback(null, result);
    });
}

然后像这样使用它：

fillVideo("http://www.w3schools.com/html/html5_video.asp", function(err, videos) {
    if(err) {
        console.log("Error: ", err);
        return;
    }

    videos.forEach(function(video) {
        request(video.url).pipe(fs.createWriteStream(video.text));
    });
});

注意：

始终检查/记录回调的 err 对象。他们准确地告诉您的代码有什么问题。在这种情况下，URI 是无效的，这实际上是正确的。