Re: [请益] 如何沉住气读别人的 code

楼主: accessdenied (存取违规)   2018-01-05 06:54:50
※ 引述《godspeedlee (妳,我可以)》之铭言:
: ※ 引述《p52189 (鼠霸)》之铭言:
: : 如题……原始码不长
: : 但撰码习惯跟我完全不同
: : 到现在我也还没找到规则
: : 以至于想改一小部分都因为不知道规则而必须几乎全部爬一遍才改得动
: : 每次读都很难沉住气一下就很烦躁不想做了
: : 一直这样很像也不是办法
: : 请问大家有什么方法可以坐得住吗= =
: : 谢谢……
: :
作者: brianhsu (坟墓)   2018-01-05 08:18:00
但我更讨厌连好好缩排都不会的人……orz.不过同意 reformat 要独立一个 commit, 不然看 diff log会想骂干,两种都很干。
作者: alan23273850   2018-01-05 08:21:00
推!
作者: ku72 (ku72)   2018-01-05 08:27:00
只能推了 为了妈妈好XD!!
作者: godspeedlee (妳,我可以)   2018-01-05 08:34:00
我指的排版是自己看,请不要随便问候别人妈妈贵司的管理问题干我何事,我只是提出方案
作者: abccbaandy (敏)   2018-01-05 08:55:00
整个档案都变动是IDE搞得吧,谁会没事在那边排版所以应该要"问候"的是上一个作者为什么不排版(ry
作者: godspeedlee (妳,我可以)   2018-01-05 09:09:00
有没有排版是主管负责监督,别自己找事动了别人的code后面就要负责
作者: kaitokid1214 (羽)   2018-01-05 09:15:00
其实这些眉眉角角还蛮重要的
作者: Ommm5566 (56天團)   2018-01-05 09:25:00
排版是IDE送的 为什么不要用lint规定格式?马上验证上一篇 呛别人的自己也通常没照软工开发
作者: y3k (激流を制するは静水)   2018-01-05 09:29:00
commit的summary要写好 不要包裹一堆进去同一个 我认为是基本的基本耶 连这种事情都不办的 我会认为他都不知道自己在做什么 或是连用打字表达自己在做什么都有障碍..
作者: iamshiao (CircleHsiao)   2018-01-05 09:43:00
空白跟断行都是有选项可以去忽略的
作者: onlyeric23 (MiGG)   2018-01-05 09:50:00
不会忽略空白也很扯
作者: Deltaguita (伯利兹)   2018-01-05 09:53:00
你是不是没遇过IDE会帮你排版的
作者: t64141 (榕树)   2018-01-05 10:08:00
排版独立commit的确比较好,尤其是整页都变动的
作者: TitanEric (泰坦)   2018-01-05 10:51:00
呜呜 我要怒改我习惯了
作者: godspeedlee (妳,我可以)   2018-01-05 11:09:00
有没有见过把astyle塞在makefile里的?结果弄到不小心很多地方需要重新编译整个编译速度还变得超慢
作者: Raymond0710 (雷门)   2018-01-05 11:12:00
好7pupu 没有统合coding style吗?
作者: Ekmund (是一只小叔)   2018-01-05 11:25:00
哈哈 这个是真的我有次上版顺便用第三方排版工具修过 后来看变动纪录自己都觉得好笑
作者: steve1012 (steve)   2018-01-05 11:43:00
这种东西都马 linter 要规定好 没format都不准 check in就好了
作者: Luos (Soul)   2018-01-05 11:54:00
推 有时候想排版都会犹豫
作者: vi000246 (Vi)   2018-01-05 11:56:00
看到整块红的以为改了什么 结果只是多了空格
作者: NDark (溺于黑暗)   2018-01-05 12:12:00
----现在宣布 空白跟tab 大战开始-----
作者: chuegou (chuegou)   2018-01-05 12:36:00
tab应该是3个空格 4个的tab是怎样
作者: Darkword1987 (黑字)   2018-01-05 16:13:00
在路上乱拉屎 骂清的人刮坏草皮
作者: becca945 (频果芽子)   2018-01-05 17:47:00
推荐全角空白 我帮你加一个:)
作者: kreety (想再见到蓝天白云的天空)   2018-01-05 17:48:00
这篇让人莫名心有戚戚焉
作者: ah7675 (阿毛)   2018-01-05 20:36:00
git diff -w就好啦 这都不会用喔?
作者: sysrq (从零开始)   2018-01-05 23:33:00
修排板和 bug fix 要分开 commit.
作者: s0914714 (YA)   2018-01-06 01:40:00
排版的风格应该是要团队定好 用tool帮忙修改
作者: sorryla (Mr.东)   2018-01-06 07:18:00
tab要怎么显示是编辑器可以调的好吗...你想要一个tab一百个空格也可以
作者: y3k (激流を制するは静水)   2018-01-06 07:48:00
tab会有IDE显示的问题喔 不过space在极端状况也是有状况XD所以战tab跟space等于是战IDE的意思
作者: snaketsai (さいでんし)   2018-01-06 10:02:00
linter policy上好,lint failed = no checkin ++
作者: steve1012 (steve)   2018-01-06 11:49:00
完全不知道 tab有啥好吵的 统一就好 重来没在意过
作者: godspeedlee (妳,我可以)   2018-01-06 12:24:00
我推space,但碰到makefile就没辙
作者: askacis (ASKA)   2018-01-06 12:39:00
以下开放战括号位置~
作者: asleisureto (ASLE)   2018-01-06 14:16:00
我认为大括号要换行(战
作者: CodingMan (程式侠)   2018-01-06 14:47:00
作者: Sidney0503 (Sidney0503)   2018-01-06 14:58:00
不用linter在那边问候别人妈妈
作者: pttworld (批踢踢世界)   2018-01-06 16:03:00
这个情境是产品上线之后,在开发阶段调整是很正常的。
楼主: accessdenied (存取违规)   2018-01-06 17:37:00
一堆学生发言,以为人人都是新创,自以为自己进公司的第一天就是产品开发的第一天...无言。低薪真的刚好。
作者: THEWORLDS (天下)   2018-01-06 18:05:00
明显这串就是一堆菜逼巴在发专页文阿ide里面写个规则就好了,别人乱改在存一下就回来了
作者: godspeedlee (妳,我可以)   2018-01-06 19:41:00
薪水高到可以问候别人娘亲?台湾软件有高薪也有低薪,高薪也有一堆看不下去的code如何顺利看code跟不要笨到触犯规则是两码事
作者: pttworld (批踢踢世界)   2018-01-06 20:50:00
工作到最后领高薪还不能用嘴巴写程式蛮悲情的
作者: sunsamy   2018-01-06 23:02:00
这篇程度很差,从来没这种问题,你们工作流程出问题才会这样,也就是说让两个人以上写同一份code,到最后只会瞎忙,品质低落
作者: THEWORLDS (天下)   2018-01-06 23:35:00
人家是领400W的人 给这些推的学生们尊重好吗?
作者: Masakiad (Masaki)   2018-01-07 10:50:00
真的蛮菜的
楼主: accessdenied (存取违规)   2018-01-07 11:04:00
Sumsany, 你只参与过hello world专案吧?多人开发同一支程式本来就是常态,还你娘的工作流程问题咧?先不提团队开发,就算是单兵作战也有人员流动交接问题,学生菜鸟就安静,别秀下限
作者: dalconan (寒江雪)   2018-01-07 11:39:00
不一定是工作流程问题,有时候是旧code没规范,后来新的主管开始有规范,当改到旧code时就会有这种状况就会面临一个“需不需要顺便整理code style”的状况
作者: THEWORLDS (天下)   2018-01-07 13:55:00
accessdenied我在G+工作,上次讲篇论文被妳嘴砲我就想笑,老子在讲论文你在谈学生爱听的文组理论这篇就不是工作流程问题,单纯你根本不懂底层基本设定我是不知道ide基本设定年薪400+不会设定真的蛮狂的长官没有规划code style你7pupu欸CODE根本就超简单,除了算法要花一点时间看而已自己权限不够无法指挥共同工做的同事那很简单阿 2条路1.变长官2.自己设定一下ide就好了,学文组怪东怪西喔对了,上次我讲完那篇以后某企业就发表实作了^^软件业神人很多,下次不要一直拿薪水来提很丢人现眼可能我平行世界,认识领你两倍以上都不会像妳这样到处炫耀,不懂你这种行为怎么领400+的,真的很有专业程度
作者: vi000246 (Vi)   2018-01-07 15:53:00
G+是哪里 抱歉我手痒爱肉搜https://goo.gl/xwDhZd拍谢 我看错= =
作者: Sidney0503 (Sidney0503)   2018-01-07 16:53:00
呵呵呵 自己不会写程式先呛别人就对了连工具都不会用还可以说嘴年薪破百万不会使用git 敢问高就哪间公司?我绝对不会去面试的无聊看了一下这个ID 原来是一位年薪三百万的大大工作游刃有余 可以狂发优质文章
楼主: accessdenied (存取违规)   2018-01-07 18:07:00
楼上两位真是天才,第一,别人改排版干我的ide设定有什么关联?又不是我的ide改动的排版。我设定自己的ide是能解决什么问题?麻烦世界小子说说看!这种阅读理解力还敢写论文,真是丢脸到家Sidney兄也是没见过世面,以为改排版就只有空白的问题,随便举例,括号没断行变成有断行你的diff工具怎么设定怎么避开?没见识硬要秀下限?
作者: angusyu (〒△〒)   2018-01-07 18:10:00
科科,管不到你旁边的在这崩溃有用吗
作者: druid0214 (Assassin)   2018-01-07 19:04:00
就在那个commit前开个branch出去不就好了然后commit前后都用format工具整理不就好了这种解决问题能力可以领三百万? 哪间公司介绍一下我用脚打code应该就可以录取了原来括号是难题阿 只能推了连branch都不会开 没用过formater 还说别人没见过世面好险我没见过世面 我怕看到现实我会被自己的code帅醒就只会一天到晚问候别人
楼主: accessdenied (存取违规)   2018-01-07 20:26:00
看来楼上都是没有出过社会的学生,没有听过什么叫做production support, 不然就是穷途末路的低薪码农,公司连个在线营运的产品都没有,才会这么没见识。我谈production support, 你们却都在以悠悠哉哉的开发时期。楼上还好你没来我公司,我怕你要做production support就吓到尿出来喔
作者: Sidney0503 (Sidney0503)   2018-01-07 20:41:00
哪间公司产品在线的时候source不能让看code的人fork出来检查? 麻烦报个名号让大家知道我第一次知道不能fork的公司耶 是不是也不能merge回去 有bug通通不能改唷^.<我还真是没见识 谢谢你告诉我有这种公司
楼主: accessdenied (存取违规)   2018-01-07 20:48:00
楼上已经在胡言乱语了, 看 Diff 跟 fork 什么关联?
作者: Sidney0503 (Sidney0503)   2018-01-07 20:49:00
据我所知FLAGM都可以fork出来 看来你的公司比这些公司fork一只出来 然后在那个commit之前开一个branch然后改之前改之后用格式整理再做diff很困难?真是不好意思 会这个可能要年薪四百万对年薪三百万的人而言这个操作太困难了附带一提 把这个功能写成自动化可能要年薪五百万我记得你的年薪想要升到四百万 会了这个然后不会写自动化可以刚刚好
楼主: accessdenied (存取违规)   2018-01-07 21:04:00
楼上真是太闲了, production support 为了看 diff浪费时间这样大费周章, 这闲情逸致公司不知道多损多损失几百万只因为你慢慢搞这些这些损失扣你的薪资不知道白做几年工
作者: Sidney0503 (Sidney0503)   2018-01-07 21:10:00
这个应该是天天需要的功能吧 花一天写一个自动化手稿损失几百万? 那不知道会省下几千万啊我忘了你的年薪只有三百万不会写自动化手稿抱歉抱歉
作者: pttworld (批踢踢世界)   2018-01-07 22:09:00
只会维护技术容易停滞,跟不上时代
作者: THEWORLDS (天下)   2018-01-08 23:08:00
我真的觉得druid0214给的建议很好,不过她年薪很高不会用GIT很正常好吗,不关你ide的是没错,但你又爱在那边抱怨,改个ide机处设定就好有多难,笑死人还有我进公司第一天真的是自己开发一个专案没错损失几百万我笑了,你以为你是政府单位接10e以上案子?就算被改调自己ide存一下都会有基本规格是会差到哪?不过听accessdenied讲说公司不能fork又年领400万台币我猜各位可能是宇宙等级的公司上万兆台币资本额才会让年薪400万阶级的连开fork的权利都没有

Links booklink

Contact Us: admin [ a t ] ucptt.com