建议把 merge_config.php 挪到config目录中

Fecshop系统 · ezsky · 于 3年前 发布 · 1448 次阅读

希望目录能整洁一些

共收到 8 条回复
Fecmall#13年前 0 个赞

1.问题需要写清楚,到底是那个目录的文件,移动到那个地方?不然看不懂的。

2.整洁一点,这个理由没有说服力。

ezsky#23年前 0 个赞

@Fecmall

比如:fecmall/appfront/merge_config.php是不是放到fecmall/appfront/config更合理一些

下面各个入口文件都包含了类似的文件

  • appserver
  • apphtml5
  • appfront
  • appbdmin
  • appapi
  • appadmin
ezsky#33年前 0 个赞

本想着发起个讨论帖,看来应该拿出代码来讨论更有针对性。git上提交pull request吧。

Fecmall#43年前 0 个赞

首先,你知道 merge_config.php 这个文件是干嘛的吗?

fecmall的配置文件非常多,当初始化的时候,通过array_merge函数合并,高优先级的覆盖低优先级的配置,进行重写 ,但是这个会带来性能损耗,merge数组还是有计算开销的

如果对性能有要求,那么可以将N个配置文件合并成一个配置文件,配置将保存到merge_config.php,因此这个文件不是初始配置文件,写在config文件夹也不合理

参考文档:http://www.fecmall.com/doc/fecshop-guide/instructions/cn-2.0/guide-fecmall_config_speed.html

Fecmall#53年前 0 个赞

参看这个文档把:https://www.yiichina.com/tutorial/1493

这个是再yiichina写的,更详细一些。

ezsky#63年前 0 个赞

这个就是相当于缓存,但是又有别于缓存文件。 缓存文件是可以随时删除的,系统在运行时会重建。

这个文件生成是通过另外一个入口手动生成,所以还不能像缓存一样可以随时清除。 所以放到config下面,就像main-local.php一样是个本地化性质的配置文件

如果从根上解决这个问题,那就得重新设计这个机制了

  1. index.php 中判断是否启用merge_config
  2. 如果启用则去找这个文件,文件不存在就去创建

这样的话,这个文件就可以放到runtime中也是比较自然

Fecmall#73年前 0 个赞

1.文档已经发你了,是否仔细看,是你个人的问题了,看你的回复,应该也没有仔细看。

2.这个问题不做讨论了, 你自己理解吧

ezsky#83年前 0 个赞

@Fecmall #7楼 是的,个人洁癖而已

添加回复 (需要登录)
需要 登录 后方可回复, 如果你还没有账号请点击这里 注册
Your Site Analytics