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

哎,我这心态对吗,接受一个项目,现在看着这 1000 多个编译警告不想管了。

  •  1
     
  •   huyomi · 2024-01-25 15:27:19 +08:00 · 5849 次点击
    这是一个创建于 374 天前的主题,其中的信息可能已经有所发展或是发生改变。
    难受,但是又懒得改。
    44 条回复    2024-01-26 17:30:10 +08:00
    jazzg62
        1
    jazzg62  
       2024-01-25 15:45:01 +08:00
    能跑就行
    jazzg62
        2
    jazzg62  
       2024-01-25 15:45:21 +08:00   ❤️ 5
    代码和人有一个能跑就行
    i8086
        3
    i8086  
       2024-01-25 15:51:39 +08:00
    世上没有不存在完美。
    要么就欺骗自己,屏蔽编辑警告,有句话:眼不见心不烦。
    Tenlearn
        4
    Tenlearn  
       2024-01-25 15:52:20 +08:00
    编译警告又不是运行 bug
    qingshui33
        5
    qingshui33  
       2024-01-25 15:54:08 +08:00
    我现在这项目也是这样的
    Noicdi
        6
    Noicdi  
       2024-01-25 15:54:34 +08:00 via iPhone   ❤️ 2
    反正不是 error ,哪怕做到自己没 warn ,指不定引入的第三方有 warn 嘞
    Yuanlaoer
        7
    Yuanlaoer  
       2024-01-25 15:55:50 +08:00
    Alert 还好吧。不同公司里待过,见过的项目里数千个 Alert 的比比皆是。
    hhjuteman
        8
    hhjuteman  
       2024-01-25 16:00:38 +08:00
    吃力不讨好,我也不管
    beyondstars
        9
    beyondstars  
       2024-01-25 16:08:36 +08:00
    不是所有的 warning 都需要严肃对待吧,有的只是 linter 太过敏感了,例如 ([[nodiscard]]),改为 static member function 的建议等,可以配置一下 linter ,屏蔽一些规则, 让它少输出一些 warning 。
    spadger
        10
    spadger  
       2024-01-25 16:09:46 +08:00   ❤️ 1
    我自己的项目,一个警告也不允许有
    sleepybear1113
        11
    sleepybear1113  
       2024-01-25 16:11:54 +08:00
    我还专门打开浏览器控制台看了一下,是不是我图片没显示出来,结果是没有图。

    我自己写代码都是尽量避免 warning 的,基本没有。然后公司的项目,一言难尽。只能说,能用就行。很多逻辑是建立在 bug 之上的。因为我看到项目的代码,比如,一个 if 中 Java Enum 对象直接 equals 字符串,IDEA 都 warn 了说不可能相等的,但是我又不敢改,改了说不定逻辑变了就得背锅。

    但是我也会顺手改那些不影响逻辑的 warning ,优化代码,添加注释(不加下次改这里还得费劲理解代码)。然后将改的那部分格式化,缩进这种,至少看起来我动过的能过赏心悦目就行。其他的,交给上帝吧。
    Daniel17
        12
    Daniel17  
       2024-01-25 16:12:14 +08:00
    确实有点多,不过不是 error ,问题不大,你要是强迫症就改
    mxT52CRuqR6o5
        13
    mxT52CRuqR6o5  
       2024-01-25 16:15:17 +08:00
    不管就不管吧,毕竟整出 error 和 warning 两个不同的概念,就是为了然你区别去对待,如果你全把 warning 当做 error 去处理,不就代表 warning 的设计没有意义了吗
    erictang66
        14
    erictang66  
       2024-01-25 16:17:48 +08:00   ❤️ 1
    哈哈哈哈,看起来楼主有代码洁癖
    1423
        15
    1423  
       2024-01-25 16:27:50 +08:00
    让领导定,领导让处理,那就是工作内容
    领导不关心,那就不管
    hervey0424
        16
    hervey0424  
       2024-01-25 16:44:19 +08:00
    把警告关了
    ugpu
        17
    ugpu  
       2024-01-25 16:48:16 +08:00
    C++ 没 warning 只有一种可能 屏蔽了.
    KOMA1NIUJUNSHENG
        18
    KOMA1NIUJUNSHENG  
       2024-01-25 16:48:40 +08:00   ❤️ 1
    改不完的,自己会慢慢妥协的。一开始都会不习惯,满满的就习惯了。
    janus77
        19
    janus77  
       2024-01-25 16:50:28 +08:00
    又不是不能用.jpg
    ziwen1943
        20
    ziwen1943  
       2024-01-25 16:55:17 +08:00
    编译警告不影响功能就行,修代码需要花费大量的时间和精力,要么让你公司加代码静态扫描工具提高代码整体质量,要么就问领导修告警算不算工作量,能不能写周报。如果不行,放弃,错不在你,责不在你。
    pkokp8
        21
    pkokp8  
       2024-01-25 16:55:31 +08:00   ❤️ 7
    1000 wanings 0 errors
    修改一处代码
    999 warnings 999 errors
    ctrl-z
    1000 wanings 0 errors
    ydpro
        22
    ydpro  
       2024-01-25 16:58:06 +08:00
    正常,人类都有趋利避害的天性
    weidaizi
        23
    weidaizi  
       2024-01-25 17:01:43 +08:00
    set(CMAKE_COMPILE_WARNING_AS_ERROR ON)
    weidaizi
        24
    weidaizi  
       2024-01-25 17:05:58 +08:00
    上面不小心手滑发出去了......
    ---

    在公司 c++ 应用项目里,建议强制加下面的东西,在 ci 编译 release 的时候才把 sanitizer 关了
    ```
    option(CMAKE_COMPILE_WARNING_AS_ERROR ON)
    option(BUILD_SANITIZER ON)

    if (BUILD_SANITIZER)
    # 开启各种 sanitize
    endif()
    ```
    BeiChuanAlex
        25
    BeiChuanAlex  
       2024-01-25 17:06:18 +08:00
    行业经典名言:《又不是不能跑》
    nevermoreluo
        26
    nevermoreluo  
       2024-01-25 17:12:22 +08:00
    你要是不说我都忘了手里几个古老的项目 warning 有多少个
    wangtian2020
        27
    wangtian2020  
       2024-01-25 17:38:02 +08:00
    不影响下班时间就改改,又不是不能跑
    thorneLiu
        28
    thorneLiu  
       2024-01-25 17:51:20 +08:00 via Android
    我会去改 不然它影响我的心情 我的睡眠
    xuelang
        29
    xuelang  
       2024-01-25 18:02:56 +08:00
    开始有强迫症,想改。后来发现改不完,只挑重要的改了
    mipawn
        30
    mipawn  
       2024-01-25 18:28:24 +08:00
    我这个公司的项目,刚开始我拿着自己的苹果本,打开控制台,日常网页直接崩溃,我以为是我的电脑的问题,8G 内存太小了,然后换了一个 16G M2 的 pro ,结果还是卡成狗,后来才发现是 warning 太多了,问领导,领导说关了 waning 就行。。。
    zypy333
        31
    zypy333  
       2024-01-25 18:30:58 +08:00
    见过一个运营商老项目,都没法正常编译,无数 error ,部署项目需要在本地用 eclipse 的强制编译,把.class 文件拿出来,丢到服务器的 webshpere 里去
    bybyte
        32
    bybyte  
       2024-01-25 18:37:31 +08:00
    自己的项目追求完美,别人的项目能跑就行
    dearmymy
        33
    dearmymy  
       2024-01-25 19:33:13 +08:00
    主要很多警告是第三方的
    samhjn
        34
    samhjn  
       2024-01-25 20:41:52 +08:00 via iPhone
    得看看项目处在啥阶段。
    如果没啥用户,挂了也没关系(或者腹黑一点,不关你自己啥事),告警再多一个数量级又何妨?
    但是如果你们和我们现在一样,一个几百毫秒甚至 50 毫秒的业务抖动都要被用户找过来问一番为什么,老板对稳定性的关注空前,那还是把前人技术债还了或者找个冤大头帮你还吧。
    stillyu
        35
    stillyu  
       2024-01-25 23:21:57 +08:00   ❤️ 1
    @spadger +1
    拼写错误都不会有
    bfjm
        36
    bfjm  
       2024-01-25 23:28:29 +08:00 via iPhone
    除了 UB 其他的不用太仔细看就好了
    churchmice
        37
    churchmice  
       2024-01-26 00:12:48 +08:00 via Android
    有啥,我之前一天改过 2w 个警告
    lovelylain
        38
    lovelylain  
       2024-01-26 08:21:21 +08:00 via Android
    不引入新的 warning ,存量的除非算工作量,否则不管。
    daimiaopeng
        39
    daimiaopeng  
       2024-01-26 09:10:00 +08:00
    你就说能不能用吧
    layxy
        40
    layxy  
       2024-01-26 09:20:08 +08:00
    接手的项目没问题就不要动,1000 个 warning 你就是再仔细修很可能会修出问题,后面自己做的时候别引入新 warning 就可以了,历史的东西问题多了去了
    m1klos
        41
    m1klos  
       2024-01-26 09:23:19 +08:00
    警告!=错误

    能跑=没问题
    magicianlib
        42
    magicianlib  
       2024-01-26 10:51:40 +08:00
    @jazzg62 就跟我凯爹五无尽一个道理,不是你走就是我走
    yanyiming
        43
    yanyiming  
       2024-01-26 15:25:00 +08:00
    有些 bug 可能是同源的, 解决一个其它的也就好了.
    whenov
        44
    whenov  
       2024-01-26 17:30:10 +08:00
    自己从头写的代码开-Wall -Wextra -Werror ,原来的代码开-w 。如果包含的别人的头文件中有 warning ,就用#pragma GCC diagnostic 处理一下
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2038 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 06:14 · PVG 14:14 · LAX 22:14 · JFK 01:14
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.