工程的艺术(3):代码健壮性之函数之间不要互相依赖

我的代码有三个文件,一个负责产生SVG字符串,一个负责存储该字符串,一个负责绘制
代码大体如下所示,并且三个分处不同的文件

class Producer {
	void createSvgString();
	...
};
class Storage {
	void setSvgStying();
	...
};
class Draw {
	void drawSvgFigure();
	...
};

现在有一个需求,我需要在判断这个SVG图案是不是一个封闭的图案。
一开始为了避免修改太多的逻辑,我在构造SVG字符串的时候,对于封闭的容器,在字符串的尾部添加了一个Z结尾。然后我只需要在绘制类中,对其进行判断就可以解决问题了。
这个方法存在着一个相当大的问题,很容易给后续的开发者埋坑。判断是否为容器的方法严重的依赖于产生SVG字符串的函数,后续若忽视了这点,针对于容器进行其他功能开发时,一旦出现问题,需要在另外的文件中,一个不相干的类中去解决。
所以此处的做法应该是专门创建一个函数,用于判断是否为容器,再将其结果传递给绘制类。
以后再干活的时候,要注意一个函数就专心干一个活,不要在其中穿插其他函数应该干的活。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值