dodgepudding / wechat-php-sdk

微信公众平台php开发包, weixin developer SDK.
4.44k stars 2.32k forks source link

add more comments on getQRCode functions; re-implement some code #277

Closed nfer closed 8 years ago

nfer commented 8 years ago

修改getQRCode注释 调整代码,并增加更详细的条件判断 增加微信小店的接口

dodgepudding commented 8 years ago

 微信小店的接口本不应放这里,不过之后再独立出来吧,卡券现在的改动也是太大了,然后也要独立出来

nfer commented 8 years ago

我觉得,“wechat-php-sdk”这个库最大的优势就是一个文件全部搞定,我不建议把一些功能独立出来,这样使用者还需要有选择性的include多个文件。 而对于开发人员来讲,则可以自己根据需要对文件进行裁剪,或者可以提供一个脚本或者页面让最终用户自己选择需要的功能。

dodgepudding commented 8 years ago

嗯,不过如果加入autoload的方式不知道大家会不会接受。

nfer commented 8 years ago

说实话,我对于autoload了解的不多,如果说还是需要多个文件(虽然不一定是多个include),我觉得在使用上仍然不是很舒服。

比如,在我目前的一个项目中,需要在多处使用这个库,那么我就可以直接拷贝这一个文件,或者在每个地方给这个文件创建软链接就可以使用。而如果是多个文件或者完全是一个文件夹的形式。在个人感情上,我不是很倾向使用(虽然并没有什么)。

其实,我在想,能不能提供多个版本,比如:

或者,再智能一点,提供一个页面或脚本,让用户自己选择需要哪些功能,然后build出来一个定制化的版本自己使用。当然这个对于项目的维护人员就复杂多了。

个人意见,仅供参考。

yunfong commented 8 years ago

我倾向于:开发的时候,分开功能文件,打包的时候,可以包装到一起。这样方便两种需求用户。目前很多Javascript项目都是这样做的。

将需要的模块,打包到一起,不需要的就不打包。 只需要增加一个打包工具。这个工具的工作就是模块代码合并。

nfer commented 8 years ago

是的,这正是我想要的结果,但是上面没有表达清楚的,:(。