招了一个人，技术不达标怎么办

[gggyyybbb]

招了一个人，面试各方面都不错，就让他进来了。 结果写的代码一言难尽，还总是喜欢自己造轮子，有现成的框架不知道用，也不会问，关键是写的还很烂，代码可维护性根本没有。举个例子，组里都是用 mybatis mapper 的写法，他倒好，一来就上 springdata ，跟现有的写法完全不搭，写之前也不说一声，闷头就写了。现在组里的其他成员已经非常反感了，请问有什么办法请走他

[barrywey]

试用期不合格啊，还需要多说什么？

[falcon05]

hr 劝退

[Tiller]

试用期的话就约谈，不改就劝退

擅自引入新的技术栈跟拉屎没什么区别

[lambdaq]

code review

reject

不匹配，再见。

[shadowyue]

谈话要求改善，没效果就劝退。

[gggyyybbb]

@Tiller 形容的很对，就感觉项目里被强制拉了一坨大便

[hackyuan]

当然是关闭他的提交，让他重写啊，就算合并进来的也可以 cherry-pick 出来扔了

[microscopec]

不合适就和人事说

[hackyuan]

刚开始也有憋个三五天提交上来一份不能用的，遇到后直接打回。
后来强制要求提高提交频率，等他代码风格质量提高后基本上不怎么管了，前半个月一定是折磨的，工作量倍增。

[superliy]

@hackyuan cherry-pick 不是把其他分支的 commit 同步拉到当前分支吗？ 怎么还能扔掉？

[spritecn]

组里都是用 mybatis mapper 的写法，他倒好，一来就上 springdata
------------
这跨度其实蛮大的

[potatowish]

新手吧，老员工在屎山上闪转腾挪，他在旁边悄悄拉了坨新的

[iOCZS]

有入职辅导吗？我感觉不管不顾，风险还是比较大的。至少初期分配任务的时候，需要进行一定辅导和代码审查。新人不知道自己代码的自由度有多大。应该尽量帮助他捋顺开发流程。

[oldcwj]

@superliy 应该是 revert

[COW]

拉个会统一下开发规范不就完了，怎么别人还能强行反抗不成？

[hackyuan]

@superliy 抱歉说错了，revert 😂

[cwliang]

自己招的这不是打自己脸吗

[gggyyybbb]

@iOCZS 关于他的代码质量问题，我是从一开始就在跟的，但怎么说呢，他年纪比我还大，非常有自己的想法，有时候我说不能这么写得按规范来，他口头上答应了好，但就是不改。。领导不是很关注代码质量，项目也比较紧张，有时候工期紧了也就没细看，好家伙，现在直接搞了个大的，给大家都整无语了

[gggyyybbb]

@cwliang 确实，丢脸

[iOCZS]

@gggyyybbb 写代码就要抛弃个性，融入项目，让代码看起来像一个人写的。基本的模仿学习能力还是要有的。风格问题早就要强制了，要不然债务越积越多，改不过来就早点分手，拖拖拉拉，对你们组员、项目，对他都不好，造成了巨大的时间浪费，应该尽早换人。这时候决不能心软，一定要杀伐果断了。