首页   注册   登录
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX  ›  PHP

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

  •  
  •   Annual · 189 天前 · 8362 次点击
    这是一个创建于 189 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

    第 1 条附言  ·  188 天前
    先声明一下,程序并非我开发。我只是下载过来进行修改,应该可以称作二次开发吧??

    还有,LZ 本身也是刚入门 PHP,还有很多不了解的地方,目前还没有了解面向对象开发。所以各位推荐框架的我也是用不上。

    我自己定义学 PHP 开发的过程应该就是先去尝试修改别人的程序,弄懂原理再去自己单独开发一套程序。不然你让我怎么入门,甚至是先干什么?
    88 回复  |  直到 2018-01-23 19:10:29 +08:00
        1
    xfspace   189 天前 via Android
    能用就行 :doge:
        2
    IceBay   189 天前   ♥ 1
    看的我想用 phpstorm 的 ctrl+alt+L 格式化一下
        3
    WuwuGin   189 天前
    反正看上去挺乱的
        4
    akira   189 天前
    连续赋值那一段没对齐 看着好难受
        5
    zhizunzz   189 天前 via Android
    表示淡定,比我厂还是要整洁多了
        6
    des   189 天前 via Android
    好想吐槽…… 算了
    emmmm ……
    现在不都是用框架的么
        7
    Annual   189 天前 via Android
    @des 入门 php 都是从改程序入门的 233,框架还看不懂
        8
    yorTX9t   189 天前
    你们拍黄片的都会写这么长的一行代码么?我是指最后一行
        9
    Annual   189 天前 via Android
    @yorTX9t 怎么顺怎么来,再说,我只会 echo
        10
    iRiven   189 天前 via Android
    除了 if 后面没加大括号 其他很舒服
        11
    shiji   189 天前
    @yorTX9t 我 SQL 一般喜欢美化一下再往里面塞。 否则根本不可读啊
        12
    shiji   189 天前
    话说最后一行那个双引号。。还需要用 . 拼接么。。。。 变量直接放里面就信了吧,可能引起歧义的话套个花括号
        13
    weilongs   189 天前
    功能实现,谁看你代码。
        14
    defuck   189 天前 via Android
    那是因为你没看 discuz 的代码
        15
    meanmachine   189 天前 via Android
    哪天改代码时候 intval 被删了 emmmmm
        16
    pixes4world   189 天前
    至少逻辑结构还是比较清晰的,LGTM
        17
    king2014   189 天前
    楼主写的?
        18
    kran   189 天前
    出血者。
        19
    HuHui   189 天前
    这在我见过的代码中能算优秀了
        20
    free9fw   189 天前
    变量可以精简一下,用个字典就行了
        21
    oaix   189 天前
    不觉得太乱, 挺直白的.
    格式有点乱, 行内少了很多空格(比如`=`左右, if 后面 等等), 不是大问题, 用 IDE 的格式化功能格式化一下.
    另外 if 只有一个子语句时, 也建议加上大括号.
    sql 那段有点, 可能有注入的风险.
    另外行间不同功能逻辑其实用空行隔开了, 可以用 IDE 提供的重构功能, 提取成一个函数.
        22
    q409195961   189 天前   ♥ 2
    金额是不是可以传 0.0091
        23
    whx20202   189 天前
    突然觉得 python 真特么好,看代码就跟看文章一样
        24
    flyshu   189 天前
    挺好的 原生不都这样写吗
        25
    yichinzhu   189 天前 via Android
    一眼就看到了 SQL 注入,,,
        26
    DavidNineRoc   189 天前

    这样子优雅很多 (逃
    建议使用 StyleCI 格式化一下代码,有多种标准格式可以选
        27
    v2xe2v   189 天前
    没格式化,难受
        28
    xhinliang   189 天前
    前鹅肠员工路过。
    在鹅肠见过比这恶心一百倍的代码。
        29
    ZXCDFGTYU   189 天前
    这代码真的是能写成这样也是一种奇迹,哪怕用个 tp 也不至于写成这样啊。。
        30
    13036101641   189 天前
    一般都有代码规范的插件或者工具吧
        31
    carlclone   189 天前
    远古 PHPer 面向过程面条式代码
        32
    lbp0200   189 天前
    @yichinzhu +1 SQL 注入是原罪,初级水平。
        33
    yujieyu7   189 天前
    在 header 函数前加错误控制符是什么鬼?
        34
    killall   189 天前 via Android
    这就是传说中的面条代码…不分结构不分分层,所有的东西全都烩到一起,出现问题从头读到尾,数据结构变了得改一堆的文件…业务增长在文件里不停地打补丁,最后知道维护到整个人崩溃。
        35
    justfindu   189 天前
    原生也就只能这样写了 格式化一下也许好看一些
        36
    lepig   189 天前
    还行 没到吐槽的地步
        37
    b821025551b   189 天前
    挺好的了,能看懂,格式化一下逻辑很清晰。
        38
    bramblex   189 天前
    @killall 把整个网站写到一个 php 文件里面
        39
    3than   189 天前
    @yichinzhu +1 不管程序写的怎么样 但是 sql 注入..
        40
    hei1000   189 天前
    我不是稿 php 的不平论,但是最后那两行长度上真实毁了前面
        41
    mingzu   189 天前
    屏幕挺宽啊...
        42
    tongz   189 天前
    看倒是能看,这样写代码维护起来得崩溃,如果我接盘你的代码。。算了我不想爆粗口
        43
    polymerdg   189 天前
    @3than 哪有 SQL 注入 没看见呢
        44
    sbw   189 天前
    真羡慕你们不用做国际化的
        45
    HanSonJ   189 天前
    看着不爽,但还能抢救。

    嗯,我真的很严格
        46
    popu111   189 天前 via Android
    @polymerdg 最后一行,拼接 query
        47
    jea   189 天前
    这个 应该是早些年的老代码了, 当时这么写肯定是有原因的
    我这里也有这种代码, 现在的新入门搞 php 的基本都不知道这种缩写的方式
    而且目前都推荐 pdo 了, 预绑定参数和类型, 比这好看多了, 还安全
    不用吐槽这个, 直接改了就是了, 老代码中还没有类库的自动加载, 多看看当时的背景, 在当时, 这个可不 low

    另外如果是现在的新人写的...直接给个内部规定的规范到脸上去
        48
    NowTime   189 天前
    第 25 行,引入的文件好熟悉,第 61 行,这个 MySQL 操作类也好悉...
    消失的彩虹海?=.=
        49
    580a388da131   189 天前
    这是典型的初学者代码吧
        50
    gstqc   189 天前
    真羡慕你们这些拿到 args 就敢往 sql query 里塞的大佬
        51
    mht   189 天前
    面向过程的写法 不都这屌样吗....
        52
    catinsides   189 天前
    居然没有人吐槽这个主题的标题
        53
    Sypher   189 天前
    @gstqc
    真羡慕你们这些拿到 args 就敢往 sql query 里塞的大佬
    --------------------------------------------------------------------
    我 tm 被你这条回复笑哭了,哈哈哈哈哈哈嗝
        54
    litter123   189 天前
    @gstqc 唉,拿到 args 的一直瑟瑟发抖,又清又转才敢往 prepare bind 里面塞
        55
    deadEgg   189 天前
    看上去很难受。

    @jea 请问是什么原因让人不用三元表达式代替那段恶心的 if else 赋值?
        56
    cxh116   189 天前 via Android
    @whx20202 当你碰到写 py 的,空格和 tab 混用时,那酸爽 。。。
        57
    whx20202   189 天前
    @cxh116 不会吧,会报错的这样。。。。
        58
    tagtag   188 天前
    发一个认为自己代码很差很差的人 or 发一个自己认为很差很差的代码,他代码不行,你语文不行。
        59
    Annual   188 天前
    @q409195961 应该没有人可以这么傻。。
        60
    Annual   188 天前
    @NowTime 是的,我在修改这个程序。因为看着很乱,不好下手。应该属于二次开发或者叫其他的吧。
        61
    onion83   188 天前   ♥ 2
    说 SQL 注入的都没好好看代码,paraFilter、daddslashes 是什么?

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

    综上述,码主至少有 >= 1 年 PHP 实际工作经验,这年头不用框架还能裸写 PHPer 没几个了,能搞清逻辑不炫技,踏踏实实搞清业务流程更是难得,在帝都工资开 15k 没问题。
        62
    hcymk2   188 天前
    paraFilter、daddslashes 有多大用?
        63
    hcymk2   188 天前
    这个只能说是有些年代的代码了。
    还有直接把错误提示写死。
        64
    ioven   188 天前
    几年前的代码风格,和帝国 cms 类似
        65
    pein   188 天前
    还可以了,格式化一下会好很多,你是没见过乱的代码,敢不敢把 wordpress 源码打开看看?😄
        66
    chairuosen   188 天前
    @onion83 前面 echo 了后再写 header 会报 header already sent?
        67
    checgg   188 天前
    我曹。又来黑 PHP
        68
    nosay   188 天前
    这代码看着好熟悉,莫名的亲切感。并不感觉有什么问题。年代久远点的代码不都长这样?看到 require 和 @你们心里没点 B 数?
        69
    GooMS   188 天前 via Android
    这个订单号重复了怎么办
        70
    UnknownR   188 天前
    和我们新人写的脚本差不多。。
        71
    sensui7   188 天前
    32 行有函数式的影子, 这个不乱, 只是没经过重构而已.
        72
    Annual   188 天前
    @GooMS 不是大站点,所以不进行去重吧。
        73
    sodowe666   188 天前
    突然发现 Yii2 真美妙
        74
    bestkayle   188 天前 via Android
    没有过滤?
        75
    changwei   188 天前
    没有用框架都是这样,很乱很乱,很多重复冗余的代码,我之前在 QQ 群里面接过几个类似的改代码的活儿,吃力不讨好,有看懂这些代码的时间我都用框架给他重构一个了。

    但是我也看到了几个亮点,比如 SQL 语句关键词大写,这个可以让 IDE 和编辑器很方便的高亮以及自动查错,还有就是双引号拼接会用大括号将变量包含起来,可以有效防止变量粘连(用大括号强行分离,防止变量名和后面的字符串粘连导致出莫名其妙的 BUG ),这两点我看很多用框架写 php 写了很久的人都没有做到。
        76
    loveyu   188 天前 via Android
    经过楼上各位大神的分析,这段代码质量绝对过关。而且我遇到的大多数人在不用所谓的框架时也很难写得比这好!第一个 header 前面的 @符用来屏蔽 notice 错误,现在开 notice 错误写代码的人,我遇到的太少了。
        77
    mingyun   188 天前
    empty($) && return false;
        79
    suit   188 天前
    @mingyun && 运算符只能计算标量、函数和表达式
        80
    Senorsen   188 天前
    不光是乱的问题了,代码本身也有问题。用 addslashes 代替 sql escape 就是一种错误做法。
        81
    Annual   188 天前
    @suit 你好,回答一下图片。
    1,因为是一个 pay 接口,所以并不会预先知道用户开发上使用哪一种请求方式。
    2,yes
    3,用户状态属性,因为只有 1 和 0 两个值,所以就直接写入了。
    4,原作者为什么要这么写我也不清楚,为了不引起各种错误,就没有进行更改,因为这个程序经过大量的 pay 测试。
    5,因为是 pay 程序,需要传递商品名称进行保留。
        82
    580a388da131   188 天前
    @Annual $_SERVER['REQUEST_METHOD']
        83
    Seanfuck   188 天前
    挺清晰明了的,代码格式化一下就很好了
        84
    ctro15547   188 天前
    别吝啬空格和回车啊 你要是这样写 python 早就哭晕在大街上了
        85
    daniellu   188 天前
    @yichinzhu 正解,SQL 注入妥妥的。
        86
    Tink   188 天前 via iPhone
    需要格式化
        87
    wizardoz   188 天前
    这个标题我认为就挺乱的
        88
    wmwgijol28   177 天前
    你看过微擎的东西么
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   鸣谢   ·   实用小工具   ·   2864 人在线   最高记录 3541   ·  
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.1 · 111ms · UTC 08:42 · PVG 16:42 · LAX 01:42 · JFK 04:42
    ♥ Do have faith in what you're doing.
    沪ICP备16043287号-1