看到有讨论 PHP 框架的，我最近是被新公司的 PHP 代码弄哭了

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 3841 天前的主题，其中的信息可能已经有所发展或是发生改变。

之前的一年都在用Yii写网站,越写越喜欢这个框架,代码真的相当优雅。

不过，自从我到新公司上班这1个月,这代码改的我整个人都不好了。。。。

启动的index.php代码那叫一个乱，各种require_once，各种 if(defined(XX)) 各种if else 连环套。

业务逻辑那个叫不忍直视。。。同一个PHP文件，每个方法都有一段一样的代码：验证用户，然后今天看到某个函数什么的800行代码也是大段大段的if else。

在这里吐下槽是因为，今天实在不爽，改人家写的东西BUG，就问了下这个他写的这个方法怎么老是不对。。同事有点不耐烦就回我：“你怎么不会看代码，看代码啊”。。。当时心中顿时千只草泥马奔腾。

反思了下，但还是没客观的认识到自己，以前写OOP的时候，不会常量变量什么的横空出世，至少会包个类抽象下，如这样调： Class::CONTANT ，这样也方便找啊！！尼玛现在没事就required 一个文件我还要到处ctrl+F 翻。。。

现在在新公司我效率极低，每天都是哭晕在厕所的节奏。。。是框架用多了我成煞笔了吗。

第 1 条附言 · 2014-06-21 13:36:51 +08:00

我们目前系统对性能要求挺高的，我想吐槽的不是用不用框架，至少要写点OOP的东西吧。或者，谁有PHP好的开源东东用OO方式写的。

代码

框架

PHP

44 条回复 • 2014-06-23 17:52:42 +08:00

blacktulip

2014-06-21 03:44:38 +08:00

跳槽吧，不然你很快就降格到你同事的水平了

zyx89513

2014-06-21 03:50:11 +08:00

我进来现在这个公司也是, 没用任何framework, 所谓的纯PHP 写的, 没有MVC架构, 大量的php html在一个文件混编, 大量的require_once看着就不爽.

上个月开始我说服老板开始做手机版的, 我重写的后台, 用的codeigniter, 代码看起来舒服多了, 以后直接把主站的后台也用这个.

之前来这公司之前是印度人写的代码, 可见他们水平也不咋地, 就会吹牛逼.

johnsneakers

2014-06-21 04:00:48 +08:00

@blacktulip
@zyx89513

暂时不会啊。初步观察我的同事信仰函数式编程喜欢简单暴力的做法。。不过这里服务器上面的架构还不错。至少，他们服务器玩的比我牛逼，还有些对底层性能什么的也很有研究，我就是搞不懂为什么代码写那么烂

blacktulip

2014-06-21 04:14:39 +08:00

@johnsneakers 额，其实我不懂 PHP ，PHP 到底是不是 OO 的？

panlilu

2014-06-21 04:26:28 +08:00 via iPhone

php不用框架也是能写出好代码的，不同的代码风格而已，各有利弊。

em70

2014-06-21 05:58:09 +08:00 via Android

你觉得不好，就花时间重构吧，没必要抱怨。产品最重要的是能跑起来，实现需要的功能。

代码优雅又不是必须的，至少老板不在乎，用户不在乎。

rannnn

2014-06-21 06:31:51 +08:00 via iPhone

@em70 重构是吃力不讨好的事情，别人只在乎能不能用。我们一个文件打开7万多行，20年的shit都汇集在一块了谁敢重构。。。

msg7086

2014-06-21 06:39:50 +08:00

@panlilu 不用框架的结果多数就是自己把底层代码写成框架了。

em70

2014-06-21 07:07:59 +08:00 via Android

@rannnn 看你用什么态度对待了，如果这是你的事业，加上尽可能详细的注释，能封装一点是一点，逐渐局部优化。如果只是一份工作，那就凑合着看吧。

zencoding

2014-06-21 08:08:29 +08:00

超轻量的 https://github.com/willper/JunePHP

mivec

2014-06-21 08:27:24 +08:00

同感.写一堆require.找个方法要按目录来搜.真累人

thinkif

2014-06-21 08:45:12 +08:00

可能你的同事也觉得这些代码跟屎一样, 可是都不敢(或者是懒得)去重构

imn1

2014-06-21 08:47:15 +08:00

怎么越听越像是面向过程？如果是这样，也不能完全说他们错啊，PHP面向过程也有其优势的～

hitsmaxft

2014-06-21 08:58:21 +08:00

老的代码别动，新的代码模块化处理

其实每一代人都是这么想的，不过时代变了。。

Mutoo

2014-06-21 09:03:06 +08:00

AlanZhang

2014-06-21 09:05:08 +08:00 via iPhone

这年代，不用框架等同于耍流氓。

mahone3297

2014-06-21 09:05:21 +08:00

可怜。。。确实如ls一位所说，说不定过段时间，lz的能力也成这样了。。

pango

2014-06-21 09:31:00 +08:00

@Mutoo 每当看到这样的代码我真的想一波动拳打过去。

dongbeta

2014-06-21 09:47:05 +08:00

这种情况确实令人郁闷。

我们在平时非常注意代码的干净整洁，我们每一个人都有代码洁癖，也肯分配时间来整理代码，要不楼主你来我们这里吧。

raincious

2014-06-21 10:50:30 +08:00 via Android

@Mutoo 我艹，干得漂亮。

虽然我以前的代码跟这个类似。

另外一个isset都没有，不会notice么？

shiny

2014-06-21 11:27:48 +08:00

require 太多说明 autoload 没用好或者用不起来；这同时说明了代码结构混乱以至于没法 autoload。
严格执行代码规范就能解决大半问题了，一般的项目引入框架还是太重了。

kenzi

2014-06-21 11:27:55 +08:00 via Android

@raincious 可以把 notice warning 提示关掉。。。

feuvan

2014-06-21 11:34:32 +08:00

没框架的代码找变量用 ack，很方便。

ddzz

2014-06-21 11:43:37 +08:00

自从用了phpstorm（还有jetbrains的其他IDE），感觉框架语言都无所谓了，一个好的IDE带来的震撼比好语言好框架都要大

hitsmaxft

2014-06-21 11:59:05 +08:00

@ddzz ide能帮你解决多人协作开发的问题？你把手段当成目的了

huoxiaochai

2014-06-21 12:07:59 +08:00

根本就不是一个水平上的人啊，你如果想进步，尽早扭转局面吧，要么改变提高，要么你走。时间宝贵啊

ddzz

2014-06-21 12:09:31 +08:00

@hitsmaxft 框架也解决不了，在IDE下用require_once，ctrl点击直达变量、常量、方法定义的地方，比什么框架的import方便多了。

多人协作开发，每人脑子里装着数据库结构，表间的联系，弄得滚瓜烂熟，面向对象还是面向过程都不重要了。

我们上一个项目每个action一个php文件，比如：
article_view.php
article_list.php
article_add.php
article_edit.php
article_delete.php
每个文件基本不到100行代码，用phpstorm模糊搜索切换文件，谁用谁知道。

raincious

2014-06-21 12:34:07 +08:00

@ddzz 这很明显是框架或者AutoLoad有问题，要不就是IDE不强。

你这个例子可以用Namespace来管理，现在PHPStorm这样的IDE对Namespace的支持很好。这样你就可以用

Project\Article\View 之类来拆分控制器的类。

框架个人觉得就是为了避免多人协作的时候需要记忆的东西太多，避免出错。

yakczh

2014-06-21 13:31:34 +08:00

@ddzz 如果一个模块一个目录呢?

article
\
add.php
edit.php
list.php
这种呢

raincious

2014-06-21 13:31:57 +08:00

刚拿自己的框架试了下，用Namespace + IDE加载没问题。PHPStorm还真挺智能的。

(图片8MB)

hitsmaxft

2014-06-21 13:37:57 +08:00

@ddzz 说明应用太简单。

)> ~ cloc .
1268 text files.
1222 unique files.
4221 files ignored.

http://cloc.sourceforge.net v 1.60 T=14.02 s (74.5 files/s, 10291.6 lines/s)
-------------------------------------------------------------------------------
Language files blank comment code
-------------------------------------------------------------------------------
PHP 997 13658 22488 87119
Javascript 15 2541 2159 12423
CSS 10 287 121 1523
XML 12 12 0 791
HTML 3 231 0 741
Python 1 17 9 80
Bourne Shell 3 22 4 44
YAML 3 0 0 17
DOS Batch 1 6 4 6
-------------------------------------------------------------------------------
SUM: 1045 16774 24785 102744
-------------------------------------------------------------------------------

AlanZhang

2014-06-21 13:43:06 +08:00 via iPhone

@ddzz 如果web开发不用框架，肯定会再一次发明轮子。