我在为一个项目设计一个文件信息管理模块
现在有两个文件,如下
const fs = require('fs');
const Manga = require('./manga');
let cache = []; // 存放漫画的数组
// 读取配置文件
let init = () => {
// 伪代码,别在意……
fs.readFile('./config.json').then(fileContent => {
fileContent.forEach(opt => {
cache.push(new Manga(opt));
});
});
}
// 保存配置文件
let save = () => {
fs.writeFile('./config.json', cache);
}
module.exports = {
init,
save
}
const save = require('./index').save;
class Manga {
constructor(opt) {
this.data = opt;
}
set(key, val) {
this.data[key] = val;
save(); // 问题出现在这
return this;
}
}
module.exports = Manga;
可以看到,这两个文件出现了依赖循环。
而我想,每调用Manga.set
的时候,就可以自动保存,而不用再在外部手动调用save
。
除了把这两个文件合并成一个,该怎么设计,才能避免这种情况的出现呢?
save 和 cache 单独提出来
基本思路有问题。 Manga作为一个Bean,为什么有save操作? 职责之外。 像1L说的提出来cache 和 save才是解决的问题