huyomi
V2EX  ›  C++

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

  •  1
     
  •   huyomi · Jan 25, 2024 · 7404 views
    This topic created in 1021 days ago, the information mentioned may be changed or developed.
    难受,但是又懒得改。
    44 replies  •  2024-01-26 17:30:10 +08:00
    jazzg62
    jazzg62
    Jan 25, 2024
    1
    能跑就行
    jazzg62
    jazzg62
    Jan 25, 2024❤️5
    2
    代码和人有一个能跑就行
    i8086
    i8086
    Jan 25, 2024
    3
    世上没有不存在完美。
    要么就欺骗自己,屏蔽编辑警告,有句话:眼不见心不烦。
    Tenlearn
    Tenlearn
    Jan 25, 2024
    4
    编译警告又不是运行 bug
    qingshui33
    qingshui33
    Jan 25, 2024
    5
    我现在这项目也是这样的
    Noicdi
    Noicdi
    Jan 25, 2024 via iPhone❤️2
    6
    反正不是 error ,哪怕做到自己没 warn ,指不定引入的第三方有 warn 嘞
    Yuanlaoer
    Yuanlaoer
    Jan 25, 2024
    7
    Alert 还好吧。不同公司里待过,见过的项目里数千个 Alert 的比比皆是。
    hhjuteman
    hhjuteman
    Jan 25, 2024
    8
    吃力不讨好,我也不管
    beyondstars
    beyondstars
    Jan 25, 2024
    9
    不是所有的 warning 都需要严肃对待吧,有的只是 linter 太过敏感了,例如 ([[nodiscard]]),改为 static member function 的建议等,可以配置一下 linter ,屏蔽一些规则, 让它少输出一些 warning 。
    spadger
    spadger
    Jan 25, 2024❤️1
    10
    我自己的项目,一个警告也不允许有
    sleepyfevniv
    sleepyfevniv
    Jan 25, 2024
    11
    我还专门打开浏览器控制台看了一下,是不是我图片没显示出来,结果是没有图。

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

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

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

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

    能跑=没问题
    magicianlib
    magicianlib
    Jan 26, 2024
    42
    @jazzg62 就跟我凯爹五无尽一个道理,不是你走就是我走
    yanyiming
    yanyiming
    Jan 26, 2024
    43
    有些 bug 可能是同源的, 解决一个其它的也就好了.
    whenov
    whenov
    Jan 26, 2024
    44
    自己从头写的代码开-Wall -Wextra -Werror ,原来的代码开-w 。如果包含的别人的头文件中有 warning ,就用#pragma GCC diagnostic 处理一下
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   1273 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 98ms · UTC 23:51 · PVG 07:51 · LAX 15:51 · JFK 18:51
    ♥ Do have faith in what you're doing.