b***i
发帖数: 3043 | 1 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
用大写。帮助我的那个人动态生成了一个类的变量：
XXXX* x = new XXXX(...);
我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
其他的函数里面都是用智能指针，没有用栈上的变量。
我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
情况造成了复制却不知道。这个人是不是很难对付啊？要是这样的话，我就装孙子吧。
听说我们公司喜欢听话的，不喜欢自己有主意的，不喜欢犯上的。 |
d***a
发帖数: 13752 | 2 如果可以用栈上的变量，为什么要用堆呢？我有同样的问题。:) |
b***i
发帖数: 3043 | 3 同一个单元测试文件的其他函数都是用智能指针。我就照着写了。我觉得一个文件至少
要风格一致吧。
后来帮我的那个人把大写的变量改成小写了。可是这个文件本来的变量全是大写，是不
是要全部改掉呢？他又拿不定主义。
照着写居然也有错！
【在 d***a 的大作中提到】
: 如果可以用栈上的变量，为什么要用堆呢？我有同样的问题。:)
|
m*****n
发帖数: 3575 | 4 可能走了的那个人就是因为写的代码被你的boss烦 |
b***i
发帖数: 3043 | 5 有可能。那个审查者不是我老板。
走的那个人听说以前都哭过，男的
【在 m*****n 的大作中提到】
: 可能走了的那个人就是因为写的代码被你的boss烦
|
w********m
发帖数: 1137 | |
w***g
发帖数: 5958 | 7 一路C++上来的不喜欢auto很正常. 我只有两个情况用auto:
1. very_long_::iterator it = ... 这个用auto
2. for loop循环变量不确定是int还是unsigned时用auto
【在 b***i 的大作中提到】
: 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: 我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
: 用大写。帮助我的那个人动态生成了一个类的变量：
: XXXX* x = new XXXX(...);
: 我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
: 然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
: 过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
: 堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
: 其他的函数里面都是用智能指针，没有用栈上的变量。
: 我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
|
z******n
发帖数: 1183 | 8
老板要怎么改就怎么改。
老板的话我觉得还是很有用的。应该很开心的跟着学
尽量多做，老板的要求满足，然后再加自己的东西
接手的project都肯定不完美才给你的，不一定要按人家的来
我接手的project那人还在，不过不是我老板
然后我就把别人的改的面目全非，只留了一点需要的，其他的全删了
老板很满意：）
【在 b***i 的大作中提到】
: 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: 我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
: 用大写。帮助我的那个人动态生成了一个类的变量：
: XXXX* x = new XXXX(...);
: 我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
: 然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
: 过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
: 堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
: 其他的函数里面都是用智能指针，没有用栈上的变量。
: 我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
|
z******n
发帖数: 1183 | 9
另外你的态度不太对。别人也是为了整个project好的。不一定是故意说你
你肯定要虚心学习。毕竟别人肯定更有经验的。就跟着人家学，把别人手里
的都学到手，然后再加自己的想法。可能有时候人家忙不一定会每天都要跟你
笑脸相待。完成任务把project做好然后大家都开心是最重要的
另外人家给你提了意见应该表示感谢的
【在 b***i 的大作中提到】
: 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: 我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
: 用大写。帮助我的那个人动态生成了一个类的变量：
: XXXX* x = new XXXX(...);
: 我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
: 然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
: 过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
: 堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
: 其他的函数里面都是用智能指针，没有用栈上的变量。
: 我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
|
m*****n
发帖数: 3575 | 10 代码的规范性很重要
这个规范性一般是老板定义的
你拍拍屁股走人后
他是维护你的代码的第一责任人
在接盘侠还没找到的时候 |
|
|
m*****n
发帖数: 3575 | 11 我当实习生的老板的时候
最后他们走了，我撅着屁股改了两周的bug |
o****p
发帖数: 9785 | 12 你这个老年究竟是多老的资历啊？碰到像我这样的仅仅能看懂c++的reviewer，你在堆
上申请这些变量我也不让你过，这是基本工程师的素养问题。别人可以做得不对，你为
啥要跟着错下去？变量用大写是谁家的规范？你们公司没有code style的吗？
【在 b***i 的大作中提到】
: 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: 我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
: 用大写。帮助我的那个人动态生成了一个类的变量：
: XXXX* x = new XXXX(...);
: 我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
: 然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
: 过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
: 堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
: 其他的函数里面都是用智能指针，没有用栈上的变量。
: 我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
|
d********f
发帖数: 43471 | 13 你们公司也真节约,intern的垃圾不都是前脚走后脚就进垃圾桶么，你还真敢用？
【在 m*****n 的大作中提到】
: 我当实习生的老板的时候
: 最后他们走了，我撅着屁股改了两周的bug
|
c******n
发帖数: 16666 | 14 问问他们有没有style guide或者他们用哪家的style guide
只有const我才会全大写
【在 b***i 的大作中提到】
: 同一个单元测试文件的其他函数都是用智能指针。我就照着写了。我觉得一个文件至少
: 要风格一致吧。
: 后来帮我的那个人把大写的变量改成小写了。可是这个文件本来的变量全是大写，是不
: 是要全部改掉呢？他又拿不定主义。
: 照着写居然也有错！
|
b***i
发帖数: 3043 | 15 为什么？这是一个test case，就是说是测试用的。为什么不能用？ Java的对象可都是
堆上的。
我是老年转行，刚转。
【在 o****p 的大作中提到】
: 你这个老年究竟是多老的资历啊？碰到像我这样的仅仅能看懂c++的reviewer，你在堆
: 上申请这些变量我也不让你过，这是基本工程师的素养问题。别人可以做得不对，你为
: 啥要跟着错下去？变量用大写是谁家的规范？你们公司没有code style的吗？
|
L****8
发帖数: 3938 | 16 auto 复制
auto& 引用
多简单
这个人是榆木脑袋
另外 你new这个占多大内存？
【在 b***i 的大作中提到】
: 第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: 我就写了。照着代码的其他部分写的。比如，代码里面变量都是大写，我就同样的变量
: 用大写。帮助我的那个人动态生成了一个类的变量：
: XXXX* x = new XXXX(...);
: 我一看，喝，没释放。于是看其他部分用了智能指针就也用了智能指针。
: 然后不会测试，叫帮我的那个人来看看。他看了一下，配置了测试的文件，于是可以通
: 过了。他就在我机器上提交了。立刻收到了审查者的两封邮件，第一封说，为什么要用
: 堆，不用栈上的变量。第二封嫌测试的内容太少。问题是其他的测试函数都是这么少，
: 其他的函数里面都是用智能指针，没有用栈上的变量。
: 我以前跟他说C++11有很多好处，我提到可以用auto。他立马说不喜欢auto，因为很多
|
L****8
发帖数: 3938 | 17 C++如果类设计得好 不需要new
【在 b***i 的大作中提到】
: 为什么？这是一个test case，就是说是测试用的。为什么不能用？ Java的对象可都是
: 堆上的。
: 我是老年转行，刚转。
|
d***a
发帖数: 13752 | 18 用heap，相对于用stack，程序的复杂度稍高一点，性能稍差一点。也就是说，本来可
以用stack的情况下用heap，只有坏处，没有好处。当然，差别并不大。
【在 b***i 的大作中提到】
: 为什么？这是一个test case，就是说是测试用的。为什么不能用？ Java的对象可都是
: 堆上的。
: 我是老年转行，刚转。
|
w***g
发帖数: 5958 | 19 同意.
【在 L****8 的大作中提到】
: C++如果类设计得好 不需要new
|
w********m
发帖数: 1137 | |
|
|
a9
发帖数: 21638 | 21 那不就成C了吗，还要类干嘛
都是
【在 L****8 的大作中提到】
: C++如果类设计得好 不需要new
|
L****8
发帖数: 3938 | 22 比如 std::vector 不需要new
【在 a9 的大作中提到】
: 那不就成C了吗，还要类干嘛
:
: 都是
|
S*******s
发帖数: 13043 | 23 他应该问还要堆干嘛
【在 L****8 的大作中提到】
: 比如 std::vector 不需要new
|
i*****9
发帖数: 3157 | 24 auto 会造成复制不知道？反过来说该复制没复制还差不多吧？
：第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
： |
S****8
发帖数: 401 | 25 是因为程序自己的结构都用基于stl和boost那堆library封装好了的原因吧， 不过智能
指针reset的啥的用一下new无可厚非吧:)
【在 w***g 的大作中提到】
: 同意.
|
b***i
发帖数: 3043 | 26 我没用new啊，是帮我的那个人new, 然后我看他没有delete，就改成智能指针了。
到现在还没看到coding standard。反正就是多大点事就是不建议用堆上的对象，哈哈
。最后改了了事。就是一个测试函数，用来做单元测试的，不是正式的发行的代码。
【在 L****8 的大作中提到】
: 比如 std::vector 不需要new
|
c*********e
发帖数: 16335 | 27 你就等着哭吧。你就是你老板眼里的一棋子。你老板是个虐待狂。这种人一般技术不精
，但是又要显得技术比你精，特别难对付。
【在 b***i 的大作中提到】
: 有可能。那个审查者不是我老板。
: 走的那个人听说以前都哭过，男的
|
b***i
发帖数: 3043 | 28 这人不是我老板！
【在 c*********e 的大作中提到】
: 你就等着哭吧。你就是你老板眼里的一棋子。你老板是个虐待狂。这种人一般技术不精
: ，但是又要显得技术比你精，特别难对付。
|
S********t
发帖数: 3431 | 29 这个不是code style的问题，这个是engineering culture的问题。
一种culture是，差不多的代码，省哪怕几个cpu cycle，why not。另外一种culture是
，做优化省的那点钱微乎其微没人care，懒得瞎折腾。
楼主还是要看团队和公司的文化是啥样的。
：我没用new啊，是帮我的那个人new, 然后我看他没有delete，就改成智能指针了。
： |
c*********e
发帖数: 16335 | 30 什么公司啊，还在用Ｃ＋＋?
【在 b***i 的大作中提到】
: 这人不是我老板！
|
|
|
b***i
发帖数: 3043 | 31 科学计算的领域，还是C++吧。
当然，我希望用Java，不知道是否可行
【在 c*********e 的大作中提到】
: 什么公司啊，还在用Ｃ＋＋?
|
b***i
发帖数: 3043 | 32 果然不好对付
今天做一个bug，发现如果用户使用小写就不会有问题，问他这个是否会改变这个bug的
级别和紧迫性，因为原定要周五搞定。我觉得，如果用户有办法继续用兴许就不那么紧
了，可以放下个月。结果他回文说，这个和解决这个有什么关系？问题是我才来两个星
期，我怎么知道谁决定bug的紧迫性和严重性？这个难道不是architect决定吗？不需要
讨论吗？难道就焖头做吗？
【在 c*********e 的大作中提到】
: 你就等着哭吧。你就是你老板眼里的一棋子。你老板是个虐待狂。这种人一般技术不精
: ，但是又要显得技术比你精，特别难对付。
|
a****o
发帖数: 6612 | 33 如果函数返回的是const reference，这个时候用auto会造成复制，需要用auto&返回
reference。
【在 i*****9 的大作中提到】
: auto 会造成复制不知道？反过来说该复制没复制还差不多吧？
:
: ：第一次提交就被骂。要求是增加一个getter/setter，并加到单元测试里头。
: ：
|
a****o
发帖数: 6612 | 34 能用auto就用auto多好呀。这样要是class type变了，程序不用修改。
【在 w***g 的大作中提到】
: 一路C++上来的不喜欢auto很正常. 我只有两个情况用auto:
: 1. very_long_::iterator it = ... 这个用auto
: 2. for loop循环变量不确定是int还是unsigned时用auto
|
c*********e
发帖数: 16335 | 35 下次碰到个bug,别问他任何问题。他关心的只是bug解决了没有。至于怎么解决的，是
你的事。
【在 b***i 的大作中提到】
: 果然不好对付
: 今天做一个bug，发现如果用户使用小写就不会有问题，问他这个是否会改变这个bug的
: 级别和紧迫性，因为原定要周五搞定。我觉得，如果用户有办法继续用兴许就不那么紧
: 了，可以放下个月。结果他回文说，这个和解决这个有什么关系？问题是我才来两个星
: 期，我怎么知道谁决定bug的紧迫性和严重性？这个难道不是architect决定吗？不需要
: 讨论吗？难道就焖头做吗？
|
b***i
发帖数: 3043 | 36 说的就是，auto如果造成复制用auto&即可。他非得完全避免auto.
【在 a****o 的大作中提到】
: 如果函数返回的是const reference，这个时候用auto会造成复制，需要用auto&返回
: reference。
|
b***i
发帖数: 3043 | 37 听说他要转到别的项目了，好像更有意思。暂时不会跟他有交集了。
【在 c*********e 的大作中提到】
: 下次碰到个bug,别问他任何问题。他关心的只是bug解决了没有。至于怎么解决的，是
: 你的事。
|
i*****9
发帖数: 3157 | 38 我就是说你用 auto, 不用 auto & 就应该是默认会复制呀。这和你直接写 TypeA 不
加引用有啥区别？
：如果函数返回的是const reference，这个时候用auto会造成复制，需要用auto&返回
：reference。 |
s*******t
发帖数: 44 | 39 diff review有这样的问题应该立即问manager或lead 这是team或者公司的coding
standard还是个人preference. 如果确实是standard就要求正式文档化这个standard.
甚至你可以写这个standard. 把任何个人之间的矛盾转化为team的process或者culture
是解决这类问题的方法。当然，前提是你有个靠谱的manager。
修bug的优先级啥的应该问你的manager这个team谁决定优先级。一般应该是product
manager或者project manager。当然你得给出为啥修不修是个问题的原因。这样决定优
先级的那个人才能做决定。 |
c*********e
发帖数: 16335 | 40 和和。你以为个个公司都那么正规啊？
.
culture
【在 s*******t 的大作中提到】
: diff review有这样的问题应该立即问manager或lead 这是team或者公司的coding
: standard还是个人preference. 如果确实是standard就要求正式文档化这个standard.
: 甚至你可以写这个standard. 把任何个人之间的矛盾转化为team的process或者culture
: 是解决这类问题的方法。当然，前提是你有个靠谱的manager。
: 修bug的优先级啥的应该问你的manager这个team谁决定优先级。一般应该是product
: manager或者project manager。当然你得给出为啥修不修是个问题的原因。这样决定优
: 先级的那个人才能做决定。
|
|
|
s*******t
发帖数: 44 | 41 和公司正不正规没关系。越是不正规越是要交流清楚公司culture和team对某件具体事
情的一般期望是什么。发现特别不靠谱就得早打算了。
最糟糕的是以为搞定了一些tough case就好了结果manager给你写评语说完全没达到要
求。其实最重要的不是怎么处理这一个人的下马威。而是搞清楚怎么做让你的manager
觉得是对的。
【在 c*********e 的大作中提到】
: 和和。你以为个个公司都那么正规啊？
:
: .
: culture
|
