团队开发中如何看待或者与成员沟通代码质量等问题 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
The Go Programming Language
http://golang.org/
Go Playground
Go Projects
Revel Web Framework
ryan961
V2EX    Go 编程语言

团队开发中如何看待或者与成员沟通代码质量等问题

  •  
  •   ryan961 2021-12-22 16:03:14 +08:00 2754 次点击
    这是一个创建于 1458 天前的主题,其中的信息可能已经有所发展或是发生改变。

    最近公司新起了个项目,主管让我带新人做做,新人之前做 php ,没有接触过 go 。 我将之前的项目代码以及文档都分享给了他,并给他简单将讲解了项目的架构以及流程。 我给他起好了模板给他开发,结果第二天发现他直接把代码结构直接咔嚓一顿给改了(我起的 代码架构我们已经沿用的两个项目了,其他人也都这么搞的)。然后我就给他说了一下,然后他就给我一顿争论, 还提了他的这个思想是上家主管说的...我那叫一个气的。不过还好本人脾气还算好,我直接又给他理了一下我们的代码架构, 让他重新照着开发一遍,1 是为了方便我们团队合作开发,2 是等你有了一定的基础和经验你再去构思自己的结构布局,哪有没 学会走就要去跑的呀。 今天又去看了下他复写的代码,我是真不知道该怎么跟他说了

    func getParam(c *gin.Context) *models.BaseParams { param := models.BaseParams{} _ = c.ShouldBindBodyWith(&param, binding.JSON) param.ClientIp = c.ClientIP() param.RegisterTime = c.GetInt64(keys.RegisterTime) return &param } func UserInitialize(c *gin.Context) { Info := new(struct { models.User models.UserInfo }) _ = c.ShouldBindBodyWith(&Info, binding.JSON) ctx := c.Request.Context() param := getParam(c) _ = c.ShouldBindBodyWith(&param, binding.JSON) if err := srv.UserInitialize(ctx, Info, param); err != nil { utils.FailJsonTest(c, -1, err.Error()) return } utils.SuccessJsonTest(c, 0, ecode.OK.Message(), Info) return } 
    func (s *Service) UserCreate(ctx context.Context, Info *struct { models.User models.UserInfo }, params *models.BaseParams) error { err := s.dao.UserCreate(ctx, Info) ... } func (d *Dao) UserCreate(ctx context.Context, Info *struct { models.User models.UserInfo }) error { //开启事务 tx := d.dbForMatch.Begin() //回滚事务 defer tx.Rollback() //创建用户表 err := d.dbForMatch.Create(&Info.User).Error if err != nil { return err } //创建用户信息表 Info.UserInfo.CustomerId = Info.User.CustomerId err = d.CreateUserInfo(&Info.UserInfo) if err != nil { return err } //提交事务 tx.Commit() return nil } 

    真不知道跟怎么跟他说了...是不是我太钻牛角尖了?

    11 条回复    2021-12-29 14:56:23 +08:00
    LoNeFong
        1
    LoNeFong  
       2021-12-22 16:22:23 +08:00   1
    尊重, 祝福
    LoNeFong
        2
    LoNeFong  
       2021-12-22 16:31:00 +08:00
    告诉他不用分目录, 分结构了, 直接一个*gin.Context 传到数据库 岂不美滋滋(逃
    14v45mJPBYJW8dT7
        3
    14v45mJPBYJW8dT7  
       2021-12-22 16:47:37 +08:00
    server service dao 好像没啥问题。。。这是你们的模板还是他改成这样的,不过方便维护肯定是按照模板来。
    nicebird
        4
    nicebird  
       2021-12-22 17:41:49 +08:00
    和上司沟通下,这种不遵守规范的,死三次可以开了。
    wyhooo
        5
    wyhooo  
       2021-12-22 17:47:55 +08:00
    错在招了个写 PHP 的(逃
    walkerliu
        6
    walkerliu  
       2021-12-22 17:52:39 +08:00
    最后的 code segment , 事务控制有问题吧
    missdeer
        7
    missdeer  
       2021-12-22 17:52:48 +08:00
    新人你还客气啥,就怕是老油条还顽固不化
    qq1340691923
        8
    qq1340691923  
       2021-12-22 18:05:24 +08:00
    这个代码不是挺好的吗...
    siguretto
        9
    siguretto  
       2021-12-22 18:13:02 +08:00
    挺好的,只是可能都塞在一个文件里了
    iQXQZX
        10
    iQXQZX  
       2021-12-23 10:14:11 +08:00
    这个参数 struct 就堆在这,各个地方都重复一遍?哪有这么写的,不应该统一定义在外边吗。上面还有人说没啥问题。。。
    liaohongxing
        11
    liaohongxing  
       2021-12-29 14:56:23 +08:00
    语言都不对 ,能开始写算不错了 。大部分像看天书一样坐着不动。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     5060 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 48ms UTC 08:25 PVG 16:25 LAX 00:25 JFK 03:25
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86