发一个认为自己代码很差很差的

[Annual]

RT，主题 https://mi90.com/20180112012844.png
是不是乱的无可救药？

[onion83]

说 SQL 注入的都没好好看代码，paraFilter、daddslashes 是什么？

从代码的风格和逻辑上看没啥大毛病，除了啰嗦点，改做的非空判断也有，能看懂为什么要在 header 前加一个 @ 的人我敢说楼上没几个。

综上述，码主至少有 >= 1 年 PHP 实际工作经验，这年头不用框架还能裸写 PHPer 没几个了，能搞清逻辑不炫技，踏踏实实搞清业务流程更是难得，在帝都工资开 15k 没问题。

[hcymk2]

paraFilter、daddslashes 有多大用？

[hcymk2]

这个只能说是有些年代的代码了。
还有直接把错误提示写死。

[ioven]

几年前的代码风格，和帝国 cms 类似

[pein]

还可以了，格式化一下会好很多，你是没见过乱的代码，敢不敢把 wordpress 源码打开看看？😄

[chairuosen]

@onion83 前面 echo 了后再写 header 会报 header already sent?

[checgg]

我曹。又来黑 PHP

[nosay]

这代码看着好熟悉，莫名的亲切感。并不感觉有什么问题。年代久远点的代码不都长这样？看到 require 和 @你们心里没点 B 数？

[3IOhG7M0knRu5UlC]

这个订单号重复了怎么办

[UnknownR]

和我们新人写的脚本差不多。。

[sensui7]

32 行有函数式的影子, 这个不乱, 只是没经过重构而已.

[Annual]

@GooMS 不是大站点，所以不进行去重吧。

[sodowe666]

突然发现 Yii2 真美妙

[KgM4gLtF0shViDH3]

没有过滤？

[changwei]

没有用框架都是这样，很乱很乱，很多重复冗余的代码，我之前在 QQ 群里面接过几个类似的改代码的活儿，吃力不讨好，有看懂这些代码的时间我都用框架给他重构一个了。

但是我也看到了几个亮点，比如 SQL 语句关键词大写，这个可以让 IDE 和编辑器很方便的高亮以及自动查错，还有就是双引号拼接会用大括号将变量包含起来，可以有效防止变量粘连（用大括号强行分离，防止变量名和后面的字符串粘连导致出莫名其妙的 BUG ），这两点我看很多用框架写 php 写了很久的人都没有做到。

[loveyu]

经过楼上各位大神的分析，这段代码质量绝对过关。而且我遇到的大多数人在不用所谓的框架时也很难写得比这好！第一个 header 前面的 @符用来屏蔽 notice 错误，现在开 notice 错误写代码的人，我遇到的太少了。

[mingyun]

empty($) && return false;

[suit]


[suit]

@mingyun && 运算符只能计算标量、函数和表达式

[Senorsen]

不光是乱的问题了，代码本身也有问题。用 addslashes 代替 sql escape 就是一种错误做法。