仅一次调用一次,函数执行两次

时间:2019-10-04 09:15:15

标签: javascript node.js

我正在制作一个程序,该程序可以读取嵌套文件夹中文件的内容。现在,我只是尝试在控制台中记录文件的内容。但是我得到的是两本日志,而不是一本。这是我到目前为止所做的事情

const fs = require('fs');
const path = require('path');
const { promisify } = require('util');
const getStats = promisify(fs.stat);
const readdir = promisify(fs.readdir);
const http = require('http');

handle_files = async (req, res) => {
    let files = await scanDir("logs_of_109");
    let result = await read_content(files)
    check_file_content(result)
    res.writeHead(200, { 'Content-Type': 'text/html' });
    // console.log(result)
    res.write("Hello");
    res.end();
};

check_file_content = (file_data) => {
    console.log(file_data[1])
}

async function read_content(files) {
    let file_data = []
    files.map(file => {
        let start_index = file.toString().lastIndexOf('.') + 1
        let ext = file.substring(start_index, file.length)
        if (ext == 'data') {
            file_data.push(fs.readFileSync(file, { encoding: 'utf-8' }))
        }
    })
    return file_data
}

http.createServer(handle_files).listen(8080)

async function scanDir(dir, fileList = []) {
    // fetch list of files from the giver directory
    let files = await readdir(dir);
    // loop through all the files
    for (let file of files) {
        // join new folder name after the parent folder
        // logs_of_109/24
        let filePath = path.join(dir, file);
        try {
            // 
            let stats = await getStats(filePath);
            if (!stats.isDirectory()) {
                // add the filepath to the array
                fileList.push(filePath);
            }
            if (stats.isDirectory()) {
                await scanDir(filePath, fileList);
            }
        } catch (err) {
            // Drop on the floor.. 
        }
    }

    return fileList;
}

我希望文件内容仅记录一次,但在控制台上记录两次。为什么会发生这种情况,我该如何阻止呢?

2 个答案:

答案 0 :(得分:5)

您的浏览器正在向您的服务器发出两个请求,最有可能是您在地址栏中输入的URL的请求,另一个是favicon.ico的请求。 (您可以通过在浏览器中打开开发工具并转到“网络”标签来快速分辨。)

handleFiles应该查看req(特别是它的url property),并根据要求进行操作。 (这是代码无论如何都要执行的操作。)


旁注1:您正在将async函数传递到某个东西(createServer)中,该函数对其返回的承诺没有任何作用。如果这样做,在函数内本地捕获函数中的任何错误非常重要,因为(再次)没有其他东西可以处理它们。例如:

handle_files = async (req, res) => {
    try {
        let files = await scanDir("logs_of_109");
        let result = await read_content(files)
        check_file_content(result)
        res.writeHead(200, { 'Content-Type': 'text/html' });
        // console.log(result)
        res.write("Hello");
        res.end();
    } catch (e) {
        // ...handle error here...
    }
};

旁注2:该代码已成为The Horror of Implicit Globals¹的牺牲品。在适当的范围内声明变量。在松散模式下不声明它们会使它们成为全局变量。 (还建议使用严格模式,因此会出现错误。)


¹(这是我贫乏的小博客上的帖子)

答案 1 :(得分:1)

以上答案正确。 我的方法是通过任何形式的“路由”来解决。 这是如何完成的小基本示例

const fs = require('fs');
const path = require('path');
const { promisify } = require('util');
const getStats = promisify(fs.stat);
const readdir = promisify(fs.readdir);
const http = require('http');

handle_routes = async (req, res) => {
    switch(req.url) {
        case '/files':
            handle_files(req, res);
        default:
            console.log('for default page');
    }
}

handle_files = async (req, res) => {
    let files = await scanDir("logs_of_109");
    let result = await read_content(files)
    check_file_content(result)
    res.writeHead(200, { 'Content-Type': 'text/html' });
    res.write("Hello");
    res.end();
};

check_file_content = (file_data) => {
    console.log(file_data[1])
}

async function read_content(files) {
    let file_data = []
    files.map(file => {
        let start_index = file.toString().lastIndexOf('.') + 1
        let ext = file.substring(start_index, file.length)
        if (ext == 'data') {
            file_data.push(fs.readFileSync(file, { encoding: 'utf-8' }))
        }
    })
    return file_data
}

http.createServer(handle_routes).listen(8080)

async function scanDir(dir, fileList = []) {
    // fetch list of files from the giver directory
    let files = await readdir(dir);
    // loop through all the files
    for (let file of files) {
        // join new folder name after the parent folder
        // logs_of_109/24
        let filePath = path.join(dir, file);
        try {
            // 
            let stats = await getStats(filePath);
            if (!stats.isDirectory()) {
                // add the filepath to the array
                fileList.push(filePath);
            }
            if (stats.isDirectory()) {
                await scanDir(filePath, fileList);
            }
        } catch (err) {
            // Drop on the floor.. 
        }
    }

    return fileList;
}

这使您可以通过转到handle_files网址来调用localhost:8080/files函数