ERP/MIS开发 ERP开发人员常犯的十种错误

avatar 2020年3月10日10:47:48 评论 329

经过数个ERP/MIS项目的开发,列举常见的问题与错误,共勉励。

1 界面代码与逻辑代码混淆

在项目的开始,一般都可以做到界面与逻辑的分离,界面只做数据绑定,业务逻辑则实现客户要求的计算。

WinForms中的BindingSource和Web中的ObjectDataSource,可以实现对object绑定到界面控件属性的功能。

这会带来极大的方便,降低程序的维护代价,做到低耦合。举例说明
采购单Sales Order的价格公式

Sales Order Price Amout=Item Price + Tax + Additional charge

采购单物料总金额=物料金额+ 税金+ 附加费用(运费,保险费…)

Tax 税金的计算,一般会根据物料的总金额,乘以百分比即可,是由系统自动运算的。
现在改需求了,客户不需要考虑税金,免税,这样公式即变化为
Sales Order Price Amout=Item Price +  Additional charge
公式中少了一项内容。如果界面和逻辑严格分离,这里只需要改一下逻辑即可,不需要改动界面。当项目中发生这样的需求改动越来越多时,业务与界面分离的好处会大大降低维护的代价。

2 不能适应需求变化

做ERP/MIS软件,需求不可能没有变化。即使是实施之后,修改起来比较麻烦,客户愿意出钱,也需要改动原来需求。

常见的需求更改:增加数据库字段,添加更多的明细表记录。增加数据库字段是经常遇到的,如果你的数据绑定代码是手写的,或是拼凑SQL语句,这回你可真要痛苦一下,加一个字段会令你重新检查所有的数据读写代码。另外,界面也要添加控件,绑定实体的属性。我的经验是,设计代码生成可靠,生成数据读写代码,当增加数据库字段后,只需要重新生成数据读写代码,非常轻松的维护代码。如果应用到ORM框架和数据绑定,增加字段只需要改一下界面即可,会更轻松。

3 缺乏足够的逻辑验证Validation代码

用户的输入总是有各种问题的,要常常记得验证客户的输入。设计自定义的控件,NumbericInputBox,CurrencyBox专有来输入数值数据,货币数据,可以减少很多验证代码。其次,在保存数据时,要验证数据的主键是否重复,检查外键的字段是否已经存在,这些检查是必要,尽管SQL会帮忙你找出错误,但是,终究是你没有写验证的原因。
ERP/MIS开发,验证也是体现逻辑的一个地方。工作单过帐时要检查物料清单是否有效,工序是否已经确认,下销售单时,要判断客户的信用额是否超过预定的额度,大量的验证代码存在于ERP/MIS系统中,请添加足够的验证代码,而不是假设用户的输入或系统的判断是正确的。

4 没有始终贯彻接口与实现分离

在项目开发前中期,我们预定接口与实现一定要分离,像这样的
public interface IAccountManager {  }
public interface AccountManager  {  }
调用方法
IAccountManager  accountManager = ClientProxyFactory.CreateProxyInstance<IAccountManager >();

可是,当项目处于不停的维护和修改的状态时,我们常常会偷懒,写出这样的代码
IAccountManager accountManager =new AccountManager ();

这种偷懒也是要付出代价的,前一种标准的写法,可以实现WCF,.NET Remoting通信框架的互换,而不影响任何代码,CreateProxyInstance这个方法里面,可以做的事情太多了。前一种写法,可以实现cache缓存接口实现类,而后一种方法则无法做到,前一种方法,可以减少编译时的依赖,完全在运行时才解析出需要的类型。

5 对部署/应用环境没有做周全的考虑

我们开发的代码,没有对代码的执行环境做很全面的检测。比如,客户的电脑没有安装.NET 3.0/3.5,我们写的Linq代码就会出错,如果没有安装.NET 4.0,用到并行类的方法也会出错。如果客户安装的Crystal Report的版本与我们开发时引用的版本不一致,代码执行时,会发生FileNotFoundedException异常,Report Viewer控件也是如此。客户的机器上没有安装Reporting Services,而我们的代码也没有检查Reporting Services服务是否存在,就开始读取它的报表项来呈现报表。如果竞争对手想试用你的ERP/MIS系统,把你的软件装在虚拟机中,对于依靠检测系统日期来判断是否过期的验证方式,则在虚拟机中改日期后,你的对手可以有足够的时间来消化学习你的系统。
对此,要充分考虑软件的运行环境,做出判断以减少低级错误。

 

6 对系统的更改和维护缺少设计规范(Design Specifications)

你的系统维护是否是这样的,客户说要改,你就马上开始改代码。老板说要改一个地方,你也毫不怨言的打开VS.NET,开始修改代码。这样是可以的,但不可取。客户要改,尽管他和老板的关系好,但你也要先记下来,当面沟通后,事后写个邮件给你的上司,说明有哪些任务要做,有哪些修改。这样做,一来是因为你的上司给你发工资,你的工作只对上司负责,不是对客户负责,二来是,记下来后,再书面确认一下,以防止客户抵赖或是沟通不充分。在不需要交修改维护费用的情况下,客户要改的东西实在是太多了,有时候可能客户因为心情不好,把你叫过去说半天,改这里改那里,等下班了回去之后,客户他自己都忘记了要改什么了。沟通不充分的问题,主要是怕你误解了客户的意思,或是说客户没有说明白,既然客户是上帝,上帝是没有错的,那就是你的理解错了。所以,事后用书面形式再确认一下,会是增加沟通效果有好办法。这个维护修改的书面表达形式,就是要说的Design Specifications。把客户要改的东西,放到标准的Design Specifications模板文件中去,打印出来,拿给客户签个名,他会很重视自己的说过的修改。我有曾见过厉害的客户,他明明知道这个修改有些不合理,但是工作完成后,等验收时,完全是按照他的意思做的,他也表示满意。这就叫做信誉,下次再找我改需求,我肯定会很乐意。

 

7 没有隔离第三方软件/控件供应商的组件变化

在我们的项目中,常常会引用一些第三方的控件,因为他们强大,好用。但是,我们还需要自定义一套控件,用来隔离这种第三方控件的变化。第三方的控件供应商,会常常更新它的API,属性和方法,如果项目中缺少这种隔离,则控件一变,我们的所有代码都需要改变。举例说明,第三方控件WebGrid,有一个属性DisplayLayout,用于显示界面元素,我们在ERP/MIS中大量应用到这个属性。如果有一天,WebGrid把这个属性DisplayLayout改名为Layout,这下惨了,我们要改的代码太多了。推荐一个好办法是,重新这个第三方控件,加入我们的自定义属性
public class Grid:WebGrid
{

pubic DisplayLayoutDisplayLayout;
}
当第三方的组件WebGrid把属性DisplayLayout改名为Layout,我们只需要这样做

public class Grid:WebGrid
{

pubic Layout  DisplayLayout;
}
引用此控件的ERP/MIS代码则不需要任何变化,即可适应第三方软件的API的变化。

8 报表开发与定制缺少必要的文档,公式和字段来源说明

这是我们公司ERP维护的一个困境,有大量的客户会要求实施人员帮忙定制一些报表。随着时间的推移,ERP维护人员走了一批又一批,当初定制报表的存储过程,修改的目的,文档也丢失了,很难从几百上千行的代码中,找到合适的理由,为什么要这样修改。工作多年后,虽然习惯了看没有文档的代码,习惯看没有说明和注释的代码,只好硬着头皮帮忙客户找SQL,找报表的存储过程,一点点的修复bug。ERP/MIS中,与数据有关的bug会占有很大的比例,就因为数据的不同,程序无法作出适当的处理,出错。虽然现在已经用建立Sharepoint建立了以客户为目录的共享文件夹,但要维护好客户的需求记录,还真不容易做到。

 

9 没有考虑数据库的可移植性

我一直都很佩服Oracle的市场宣传做的很好,从开发人员到工程师,从老板到销售助理,一提到数据库,都会提到Oracle。有时候销售助理,会大肆宣扬Oracle的好处,甚至比开发人员还懂得多。也有的客户,指定只用Oracle,拒绝用其它的数据库,没有任何理由。所以,如果是做一个长期的产品,还是要考虑将来数据库移植的问题。数据访问,要用通用的DbConnection,DbCommand,DbDataAdapter,或是用ORM;少用存储过程,多用程序代码来实现逻辑。在系统开发时,开发人员常常只是理论上实现了多种数据库的支持,真要到真实的多种数据库平台中跑一下的时候,问题会接二连三的来了。以SQL语句,选择前十行数据为例子
MySQL 和PostgreSQL : SELECT * FROM emp limit 10
Oracle :  SELECT * FROM emp WHERE rownum<=10
SQL Server :  SELECT TOP 10  *  FROM  emp
我的建议是,如果要支持多种数据库,尽早做规划,尽早建立真实的数据库环境测试。

10 对命名规范(文件,控件,变量)不能始终坚持

以举例为证,窗体类一般会在后面加个frm来表是是窗体类,如SalesOrderfrm,所有显示报表的窗体,后面加rpt,如SalesOrderrpt。 这里有个小技巧,为什么不是rptSalesOrder,而是SalesOrderrpt。这是为了在IDE的Solution Explorer视图中,以方便的看清同一组的窗体而以此命名的。如果我再加一个报表SalesOrderQuotationrpt,这样,在Solution Explorer视图中,这两个报表窗体是显示在一起的,方便查看。

对于控件,请看下面的命名
private EPN.WinUI.Editors.TextEditor txtRevisedBy;
private EPN.WinUI.Misc.Label lblLastRevised;
private EPN.WinUI.Editors.DateTimeEditor dteRevisedDate;
private EPN.WinUI.Misc.Button btnPrint;

txt表示TextEditor,btn表示Button,lbl表示Label.

日期格式,有的地方是用yyyy-MM-dd,但是有的地方是yyyy/MM/dd,没有统一。

价格Price字段的长度,是decimal(12,4), Amout合计字段的长度是decimal(16,4)
对于这些规范,要始终的坚持。如果可能,可以开发一个Code Analysis规则,来自动分析程序集。

avatar

发表评论

:?: :razz: :sad: :evil: :!: :smile: :oops: :grin: :eek: :shock: :???: :cool: :lol: :mad: :twisted: :roll: :wink: :idea: :arrow: :neutral: :cry: :mrgreen: