项目框架已经搭好用 springboot3 和 JPA ,然后他居然在 repository 里写了个 insert 方法,而且还是用 nativeSQL ,当然可能人家没用过 JPA ,那也算了。
然后一个发邮件的功能,他说调试了一天才搞定,然后我一看他的代码,在发邮件的函数里用 java 自带的 http client ,每发一封就 new 一个,就算不用 sprigbooot 的 rest template ,好歹也用一下 okhttpclient 吧,而且也不用 bean 管理一下。然后发邮件有个验证码机制,有有效期之类的,我本来的想法是用单独的表管理所有验证方式,比如注册验证邮件,改密码验证邮件,还有大概率手机验证,2FA 之类的,然后他直接把 token 和过期时间存 user 表里了。
还有很多小细节,比如全局捕获异常也不设置 http 状态码,直接默认都 200 (项目定的 rest 风格),有 loback 也不用,异常直接 printstack ,也从不管已经存在的代码,比如已经有 util 包和 Util 类了,还要新建 utils 包和 Util 类,异常码也是,明明有个 user_not_found 了,还要在一个他写的重复判断用户是否存在的逻辑里加一个 no_email 的错误码,反正我也不想管了,把我负责的 api 弄稳妥就行。
101
zhang77555 228 天前
建议慎重
公司规模不大,这事儿没必要管,你有技术追求早晚会离开。 公司规模大,这事儿没必要管,你也不清楚他跟领导关系如何,说了也没啥好处,真要你接锅时跑路也无所谓,代码烂总有优化空间还有利于团队生存,项目运行太好团队存在感太低这个环境下容易被砍[doge] 至于想做个高质量项目啥的,公司项目你又不能开源,而且任何商业项目随着开发周期增长很难说不变成一坨,只是快和慢的差别,能吹的项目也从来不是啥代码质量,而是项目依托于平台所做的事情 |
102
iintothewind 228 天前
@857681664 #79 多经历一些老系统就知道了, 离谱的事情真的非常多.
不是说你, 如果我如果拿我自己对代码的标准要求别人, 那估计基本其他人写的那些东西都得回炉重造. 但是看多了, 麻木了. |
103
iyaozhen 228 天前
和昨天老哥一样 /t/1039891
我认为还是环境问题,你不换个公司,只能让这些劣币驱逐良币了 |
104
rxswift 228 天前
我也有个这样的前同事
|
105
morrieati 228 天前
@857681664 #97 大部分场景下可以相信数据库的性能,supabase user 表的定义我贴在下面:
```SQL create table auth.users ( instance_id uuid null, id uuid not null, aud character varying(255) null, role character varying(255) null, email character varying(255) null, encrypted_password character varying(255) null, email_confirmed_at timestamp with time zone null, invited_at timestamp with time zone null, confirmation_token character varying(255) null, confirmation_sent_at timestamp with time zone null, recovery_token character varying(255) null, recovery_sent_at timestamp with time zone null, email_change_token_new character varying(255) null, email_change character varying(255) null, email_change_sent_at timestamp with time zone null, last_sign_in_at timestamp with time zone null, raw_app_meta_data jsonb null, raw_user_meta_data jsonb null, is_super_admin boolean null, created_at timestamp with time zone null, updated_at timestamp with time zone null, phone text null default null::character varying, phone_confirmed_at timestamp with time zone null, phone_change text null default ''::character varying, phone_change_token character varying(255) null default ''::character varying, phone_change_sent_at timestamp with time zone null, confirmed_at timestamp with time zone null, email_change_token_current character varying(255) null default ''::character varying, email_change_confirm_status smallint null default 0, banned_until timestamp with time zone null, reauthentication_token character varying(255) null default ''::character varying, reauthentication_sent_at timestamp with time zone null, is_sso_user boolean not null default false, deleted_at timestamp with time zone null, constraint users_pkey primary key (id), constraint users_phone_key unique (phone), constraint users_email_change_confirm_status_check check ( ( (email_change_confirm_status >= 0) and (email_change_confirm_status <= 2) ) ) ) tablespace pg_default; create index if not exists users_instance_id_idx on auth.users using btree (instance_id) tablespace pg_default; create index if not exists users_instance_id_email_idx on auth.users using btree (instance_id, lower((email)::text)) tablespace pg_default; create unique index confirmation_token_idx on auth.users using btree (confirmation_token) where ((confirmation_token)::text !~ '^[0-9 ]*$'::text) tablespace pg_default; create unique index recovery_token_idx on auth.users using btree (recovery_token) where ((recovery_token)::text !~ '^[0-9 ]*$'::text) tablespace pg_default; create unique index email_change_token_current_idx on auth.users using btree (email_change_token_current) where ( (email_change_token_current)::text !~ '^[0-9 ]*$'::text ) tablespace pg_default; create unique index email_change_token_new_idx on auth.users using btree (email_change_token_new) where ( (email_change_token_new)::text !~ '^[0-9 ]*$'::text ) tablespace pg_default; create unique index reauthentication_token_idx on auth.users using btree (reauthentication_token) where ( (reauthentication_token)::text !~ '^[0-9 ]*$'::text ) tablespace pg_default; create unique index users_email_partial_key on auth.users using btree (email) where (is_sso_user = false) tablespace pg_default; create trigger on_auth_user_created after insert on auth.users for each row execute function handle_new_user (); ``` |
106
destiny0114 228 天前
@857681664 这种情况我觉得不用待太久,可以找别家公司了。
|
108
dif 228 天前
大概率是不会吧,JPA 有 JPA 的写法,MyBatis 有 MyBatis 的写法。
|
109
GeekGao 228 天前
@wzdsfl 首先要知道自己能掌控的事情范围,超出自己责任和能力范围的事儿不爽也仅限于不爽(心理病、内分泌失调、心脏病等同事、公司不负责啊)。
btw:没准这系统代码 1 年后就没任何商业价值被遗弃了,或者下个季度就裁员了… |
110
Mantext1989 228 天前
你教教他
|
112
morgan1freeman 228 天前
说一下我的新路历程吧,
刚毕业的时候,我也是跟楼主一样的想法,觉得要注重代码质量,多关注阅读代码人的感受,后来真的想明白了,大部分人就混口饭吃罢了,何况 绩效 收入 升职 跟你技术能力 真的有那么大的关系么?你有追求,可以去个人项目里面去追求代码可读性,团队里面 现有代码什么风格,就跟着继续玩就好了,反正升职加薪 跟 代码 技术大概率没有半毛钱关系,关键在于你在团队里面的位置,以及你在哪个圈子里面 后面工作几年后发现,就这样吧,反正领导也不在乎,也没人为良好可阅读的代码付费,何必自我纠结,完成任务就好。 其实烂代码的团队,比较吃亏的是团队里面的边缘人,因为很多业务代码 ,第一版不是边缘人写的,因为很多重点项目都是先让领导亲信去写,只要完成任务即可,没有人在乎架构跟代码质量,完成任务就能拿各种奖项 KPI 绩效刷一波。 但是第二三版,改动的时候,这个时候产品要做改动,边缘人就会被派上场,因为项目已经完成,再投入资源去修修改改,通常这种活不会有什么功绩可说,边缘人这个时候就会被派去 填坑吃屎,而且阅读狗屎代码,吃屎真的就容易踩坑,有的时候吃自己的屎,还能闻出那股屎位,别人写的屎,真的就是摸不清脉络,可能一个方法上几百行,然后各种复制粘贴,漏了一个改动还容易吃 bug 背锅 |
114
james122333 228 天前 via Android
如果约定好使用什么那就使用什么 但使用框架等同质量好或有保证我不认同 一个框架要怎么使用也没有定论 毕竟虽然 java 封装很糟心 但你研究久了还是可以依照喜好定义
我就曾经用 mybatis 但我不想用智障 xml 也想保持灵活性和直观 直接用 mybatis 底层方法建立 statement 过 这样是不是在你眼裏是劣币? 如何把项目搞的傻瓜化才是真的 不限任何方法 我看过太多屎一般的 spring 项目 内部用的工具也是一堆 |
115
james122333 228 天前 via Android
至于代码质量的审美 不可能统一的 因为每个人历程不一样 以我的经验 能够单步除错就能够找到原因的多半不会太差 而 springboot 用註解本身就会降低可除错性
|
116
LookUpAndSmile 227 天前
@sagaxu 哈哈···,都是过来人啊
|