昨天跟同事聊天，说入了籍去当个IO也挺好# Immigration - 落地生根
d*y
1 楼
新手卖了个东西，突然发现买家有点儿suspicious。
还没寄出去，应该cancel这个order吗？cancel了买家会不会给留不好的feed back？
ebay还会不会charge钱？
多谢各位大侠！
还没寄出去，应该cancel这个order吗？cancel了买家会不会给留不好的feed back？
ebay还会不会charge钱？
多谢各位大侠！
u*8
2 楼
要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
throw new Exception("ERROR: startDate " + startDate + " is
invalid.");
}
reviewer说，Please compile the pattern + matcher, and use the matcher to
match the dates.
我问，值得么？
他说。In terms of performance, yes.
这个真的很大的performance 的关系么？
还有，return new X>(new Y (Y)
非要分段写， 3行。
说是Readability.
这算鸡蛋里面挑骨头么？
更新： 3楼之后。
http://stackoverflow.com/questions/2149680/regex-date-format-va
这个检测我是看这个的。
第2条， 我看code里面很多都是如此写的。 不用第二次的local variable就省在new里
面不可以么？
if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
throw new Exception("ERROR: startDate " + startDate + " is
invalid.");
}
reviewer说，Please compile the pattern + matcher, and use the matcher to
match the dates.
我问，值得么？
他说。In terms of performance, yes.
这个真的很大的performance 的关系么？
还有，return new X
非要分段写， 3行。
说是Readability.
这算鸡蛋里面挑骨头么？
更新： 3楼之后。
http://stackoverflow.com/questions/2149680/regex-date-format-va
这个检测我是看这个的。
第2条， 我看code里面很多都是如此写的。 不用第二次的local variable就省在new里
面不可以么？
f*y
3 楼
不知道这种工作好不好申请，貌似工作稳定，每天看看五花八门的申请材料，而且有着
生死大权在握的感觉
如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
生死大权在握的感觉
如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
l*i
4 楼
cancel, 然后等买家回应。
j*o
5 楼
不算
y*l
6 楼
有前途！
l*i
7 楼
买家如果想给你留差评，随时可以留。就算寄出去，也是早晚的事。
f*t
8 楼
第一条是必须的。
第二条你们应该先反省为啥会写出这种垃圾代码
第二条你们应该先反省为啥会写出这种垃圾代码
R*6
9 楼
yes.
【在 f****y 的大作中提到】
: 不知道这种工作好不好申请，貌似工作稳定，每天看看五花八门的申请材料，而且有着
: 生死大权在握的感觉
: 如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
: 不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
【在 f****y 的大作中提到】
: 不知道这种工作好不好申请，貌似工作稳定，每天看看五花八门的申请材料，而且有着
: 生死大权在握的感觉
: 如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
: 不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
d*y
10 楼
多谢多谢！
【在 l********i 的大作中提到】
: 买家如果想给你留差评，随时可以留。就算寄出去，也是早晚的事。
【在 l********i 的大作中提到】
: 买家如果想给你留差评，随时可以留。就算寄出去，也是早晚的事。
t*4
11 楼
不算，改不了太多撒
如果每次给你抓一点错就有挑刺嫌疑，一次把问题说完，改好就approve就不算
[在 uno88 (Ut) 的大作中提到：]
：要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
：
：...........
如果每次给你抓一点错就有挑刺嫌疑，一次把问题说完，改好就approve就不算
[在 uno88 (Ut) 的大作中提到：]
：要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
：
：...........
A*o
12 楼
据说pay的很低，估计你兴趣不大
【在 f****y 的大作中提到】
: 不知道这种工作好不好申请，貌似工作稳定，每天看看五花八门的申请材料，而且有着
: 生死大权在握的感觉
: 如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
: 不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
【在 f****y 的大作中提到】
: 不知道这种工作好不好申请，貌似工作稳定，每天看看五花八门的申请材料，而且有着
: 生死大权在握的感觉
: 如果我当IO，规规矩矩的中国人全过，烙印先tmd来个RFE，然后据他个80%
: 不多久我的代码就会在烙印论坛里变为一个杀手传说，哈哈
u*8
13 楼
但是这家伙就是明显挑刺，写很多不痛不痒的。
组要downsize，manager就要他们几个code review给我狂写。
【在 t******4 的大作中提到】
: 不算，改不了太多撒
: 如果每次给你抓一点错就有挑刺嫌疑，一次把问题说完，改好就approve就不算
: [在 uno88 (Ut) 的大作中提到：]
: ：要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: ：
: ：...........
组要downsize，manager就要他们几个code review给我狂写。
【在 t******4 的大作中提到】
: 不算，改不了太多撒
: 如果每次给你抓一点错就有挑刺嫌疑，一次把问题说完，改好就approve就不算
: [在 uno88 (Ut) 的大作中提到：]
: ：要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: ：
: ：...........
y*z
14 楼
How much?
j*o
15 楼
第一个代码是bug，你转的帖子里说的很清楚了，和performance无关。
第二个就是你们公司的规范的问题了，你要保证随便一个新来的人能看懂你的代码。公
司需要的是工作能够顺利交接，而不是你的code写的有多fancy。
第二个就是你们公司的规范的问题了，你要保证随便一个新来的人能看懂你的代码。公
司需要的是工作能够顺利交接，而不是你的code写的有多fancy。
w*y
16 楼
看在哪个地区。 一般从GS-5开始做， 最高可做到GS-9.
2012年的数据：
GS-5, $31,315 ~ $40,706.
GS-7, $38,790 ~ $50,431.
GS-9, $47,448 ~ $61,678.
应该还有联邦的福利。
【在 y****z 的大作中提到】
: How much?
n*1
17 楼
一次全说完就不是鸡蛋里挑骨头
c*f
18 楼
不算挑刺
我做CR的话，我会加上"\d{4}-\d{2}-\d{2}"为什么不是constant
我做CR的话，我会加上"\d{4}-\d{2}-\d{2}"为什么不是constant
u*8
19 楼
team要砍掉，要去别的组分流。
这几个coder被manager威胁，要给我bad review，好开我走人，不给package。
有个哥们给我说了，但是他不做这种事情。
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
这几个coder被manager威胁，要给我bad review，好开我走人，不给package。
有个哥们给我说了，但是他不做这种事情。
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
s*r
20 楼
1 yes
2 如果constructor很复杂，可以用builder pattern，比较清楚
2 如果constructor很复杂，可以用builder pattern，比较清楚
p*g
21 楼
第一个代码根本不对啊，此模式不能查出所有非法日期。建议用joda
j*r
22 楼
1. Not needed unless it's in a loop or in critical path. But you can't say
reviewer is picky.
2. Point to Java 8 streaming and ask them to fuck off.
reviewer is picky.
2. Point to Java 8 streaming and ask them to fuck off.
b*y
23 楼
算，但是给人打工就是这样了。。。工作要么忍要嘛滚
u*8
24 楼
谢谢。
还有
for (String line; (line = reader.readLine()) != null;) {
他说要
Use a while loop as is standard. Or use the IOUtils to read fully.
这算么？
【在 b*******y 的大作中提到】
: 算，但是给人打工就是这样了。。。工作要么忍要嘛滚
还有
for (String line; (line = reader.readLine()) != null;) {
他说要
Use a while loop as is standard. Or use the IOUtils to read fully.
这算么？
【在 b*******y 的大作中提到】
: 算，但是给人打工就是这样了。。。工作要么忍要嘛滚
p*r
25 楼
哥们，遇到这种情况，速度招下家吧。
头铁了心要你走，你早晚都得走。
之前我头找我茬的时候，
我就直接和头说，你要只是不满意我骂人，我以后就不骂人
你要不想我这里呆着了，那你直说，我就去找工作了。
给个痛快话，大家以后还是朋友。
【在 u***8 的大作中提到】
: team要砍掉，要去别的组分流。
: 这几个coder被manager威胁，要给我bad review，好开我走人，不给package。
: 有个哥们给我说了，但是他不做这种事情。
头铁了心要你走，你早晚都得走。
之前我头找我茬的时候，
我就直接和头说，你要只是不满意我骂人，我以后就不骂人
你要不想我这里呆着了，那你直说，我就去找工作了。
给个痛快话，大家以后还是朋友。
【在 u***8 的大作中提到】
: team要砍掉，要去别的组分流。
: 这几个coder被manager威胁，要给我bad review，好开我走人，不给package。
: 有个哥们给我说了，但是他不做这种事情。
j*8
26 楼
那你头怎么回答你的？
【在 p**r 的大作中提到】
: 哥们，遇到这种情况，速度招下家吧。
: 头铁了心要你走，你早晚都得走。
: 之前我头找我茬的时候，
: 我就直接和头说，你要只是不满意我骂人，我以后就不骂人
: 你要不想我这里呆着了，那你直说，我就去找工作了。
: 给个痛快话，大家以后还是朋友。
【在 p**r 的大作中提到】
: 哥们，遇到这种情况，速度招下家吧。
: 头铁了心要你走，你早晚都得走。
: 之前我头找我茬的时候，
: 我就直接和头说，你要只是不满意我骂人，我以后就不骂人
: 你要不想我这里呆着了，那你直说，我就去找工作了。
: 给个痛快话，大家以后还是朋友。
p*r
27 楼
头说，如果你不骂人，你就是完美人才。
然后俺就混混日子刷刷买买提，看着FLG的贫困线流口水，
一年又过去了，唉，俺的青春啊。。。
【在 j*****8 的大作中提到】
: 那你头怎么回答你的？
然后俺就混混日子刷刷买买提，看着FLG的贫困线流口水，
一年又过去了，唉，俺的青春啊。。。
【在 j*****8 的大作中提到】
: 那你头怎么回答你的？
b*5
28 楼
this is correct
【在 p**********g 的大作中提到】
: 第一个代码根本不对啊，此模式不能查出所有非法日期。建议用joda
【在 p**********g 的大作中提到】
: 第一个代码根本不对啊，此模式不能查出所有非法日期。建议用joda
h*I
29 楼
是不是挑刺每个人的感觉也不一样的，因为主观标准不一样。
以上几点我觉得都不是在挑刺。
尤其是这个：“for (String line; (line = reader.readLine()) != null;) {”
如果是我的话我会给出同样的review。
以上几点我觉得都不是在挑刺。
尤其是这个：“for (String line; (line = reader.readLine()) != null;) {”
如果是我的话我会给出同样的review。
j*o
30 楼
看对方的目的了，如果只是纠正代码那正常。如果是要以此赶你走的话就算挑刺了，不
过你也没什么办法。
过你也没什么办法。
e*0
31 楼
不算
撇开你老板要不要开人的政治上原因，这两行代码我们组code review都会被打回来
我也看到好多其他组根本就不在乎类似的，却绝于每个组leader对coding的态度
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
x*1
32 楼
有joda 不用joda 么？
x*1
33 楼
lz 估计也不写UT阿
t*c
34 楼
我这种readability都是给人敲章水过的人也忍不住喷你一下你的代码了
三个点都该喷
真正鸡蛋挑骨头的你没见过
变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
三个点都该喷
真正鸡蛋挑骨头的你没见过
变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
u*8
35 楼
此人原来做过类似的review给我。
结果自己打脸。
【在 t********c 的大作中提到】
: 我这种readability都是给人敲章水过的人也忍不住喷你一下你的代码了
: 三个点都该喷
: 真正鸡蛋挑骨头的你没见过
: 变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
: 代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
: ，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
结果自己打脸。
【在 t********c 的大作中提到】
: 我这种readability都是给人敲章水过的人也忍不住喷你一下你的代码了
: 三个点都该喷
: 真正鸡蛋挑骨头的你没见过
: 变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
: 代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
: ，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
y*d
36 楼
就事论事的话，他提这两点都有道理
如果是检查输入的话，一般来讲，performance根本就不是个事。但是，用regular
expression的时候先compile出个pattern再用是个好习惯
第二条，你那个代码确实难看
不过都不是什么大不了的问题。在代码质量要求高的地方，确实都需要改了。如果不是
什么特别强调代码质量的地方，这么写也可以了。
如果是检查输入的话，一般来讲，performance根本就不是个事。但是，用regular
expression的时候先compile出个pattern再用是个好习惯
第二条，你那个代码确实难看
不过都不是什么大不了的问题。在代码质量要求高的地方，确实都需要改了。如果不是
什么特别强调代码质量的地方，这么写也可以了。
f*n
37 楼
问题就是，之前都这么写啊。现在要裁人了，这斯就开始叽歪了。
【在 y*d 的大作中提到】
: 就事论事的话，他提这两点都有道理
: 如果是检查输入的话，一般来讲，performance根本就不是个事。但是，用regular
: expression的时候先compile出个pattern再用是个好习惯
: 第二条，你那个代码确实难看
: 不过都不是什么大不了的问题。在代码质量要求高的地方，确实都需要改了。如果不是
: 什么特别强调代码质量的地方，这么写也可以了。
【在 y*d 的大作中提到】
: 就事论事的话，他提这两点都有道理
: 如果是检查输入的话，一般来讲，performance根本就不是个事。但是，用regular
: expression的时候先compile出个pattern再用是个好习惯
: 第二条，你那个代码确实难看
: 不过都不是什么大不了的问题。在代码质量要求高的地方，确实都需要改了。如果不是
: 什么特别强调代码质量的地方，这么写也可以了。
y*d
38 楼
为这么点鸡毛蒜皮的小问题，扯到裁员就太扯了
但是裁员这事吧，就好像毛主席抓右派。上面定了名额，要抓出1/3的右派出来，他不
把你搞成右派，他自己就有成右派的危险。把你抓了，他就安全了
【在 f*****n 的大作中提到】
: 问题就是，之前都这么写啊。现在要裁人了，这斯就开始叽歪了。
d*k
39 楼
如果你的matches是在一个loop里头，在外面compile the pattern很提高速度。
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
u*8
40 楼
问题就是不在loop里面。
【在 d******k 的大作中提到】
: 如果你的matches是在一个loop里头，在外面compile the pattern很提高速度。
【在 d******k 的大作中提到】
: 如果你的matches是在一个loop里头，在外面compile the pattern很提高速度。
u*8
41 楼
改变量名，这个reviewer之前也做过类似的事情哦。
【在 t********c 的大作中提到】
: 我这种readability都是给人敲章水过的人也忍不住喷你一下你的代码了
: 三个点都该喷
: 真正鸡蛋挑骨头的你没见过
: 变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
: 代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
: ，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
【在 t********c 的大作中提到】
: 我这种readability都是给人敲章水过的人也忍不住喷你一下你的代码了
: 三个点都该喷
: 真正鸡蛋挑骨头的你没见过
: 变量命名要改，class名要改，改了之后 突然又觉得另外一个名字更好，继续改
: 代码结构要改，但又讲不出道理，只是是说他觉得那样更好，哼哧哼哧搞了一两天之后
: ，我发现此路不通，再费半个小时口舌告诉他为什么不能这么改，
u*8
42 楼
对的。我说的就是这么回事。
【在 f*****n 的大作中提到】
: 问题就是，之前都这么写啊。现在要裁人了，这斯就开始叽歪了。
【在 f*****n 的大作中提到】
: 问题就是，之前都这么写啊。现在要裁人了，这斯就开始叽歪了。
u*8
43 楼
但是裁员这事吧，就好像毛主席抓右派。上面定了名额，要抓出1/3的右派出来，他不
真相。
真相。
m*u
44 楼
这算个p的挑刺
都是很正常的建议，楼主这么defensive怪不得你头不爽你
都是很正常的建议，楼主这么defensive怪不得你头不爽你
s*1
45 楼
总之一句话，最好的代码是本科二年级就能看懂能改的代码。
别玩花，别玩火。
别玩花，别玩火。
s*n
46 楼
你应该感谢给你写这些review的人
自己出错还嫌弃
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
自己出错还嫌弃
【在 u***8 的大作中提到】
: 要检测日期，2015-10-11 是不是YYYY－MM－DD的格式。
: if (!startDate.matches("\d{4}-\d{2}-\d{2}")) {
: throw new Exception("ERROR: startDate " + startDate + " is
: invalid.");
: }
: reviewer说，Please compile the pattern + matcher, and use the matcher to
: match the dates.
: 我问，值得么？
: 他说。In terms of performance, yes.
: 这个真的很大的performance 的关系么？
u*8
47 楼
他也没看出来那个不能检查date。
【在 s******n 的大作中提到】
: 你应该感谢给你写这些review的人
: 自己出错还嫌弃
【在 s******n 的大作中提到】
: 你应该感谢给你写这些review的人
: 自己出错还嫌弃
S*w
48 楼
第一个条说的很对啊。 performance是个big concern
不过如果你这个method 几秒中或者几分钟才call一次 也就无所谓了。
不过如果你这个method 几秒中或者几分钟才call一次 也就无所谓了。
相关阅读
请问 IEEE Senior Member 能加分吗？EB1B如何让uscis把 收条邮寄 给 自己？包子求审稿机会，太阳电池方向citation少的可以试试这些方法独立推荐人的问题请问推荐信电子签名可以吗？efile eb1b i140 的话公司 如何 签字？拿到绿卡，首先要干什么？请大家帮忙出主意OPT EAD马上要过期了485-都收到FPnotice并walk-in 打了指纹，网上状态怎么还是acceptance，为LD的485求祝福9/27 NSC 双绿 + TimelineWalk-in FP 能加快绿卡批准，附本人报绿 time line 加背景要审稿被踢皮球咋办？求助：I140被笨蛋律师递到NSC了准备递交入籍申请，但是要回国几个月google scholar没有列出的citation 怎么处理推荐信签名弱EB1A TSC 应该是批了吧请问485递上去，多久可以请议员去催？3x