【うーん】プログラマー新人おれ「変数名はaとかbでええやろ」 上司「あのさぁ」

if(navigator.userAgent.indexOf(‘iPhone’) > 0){
var adstir_vars = {
ver : “4.0”,
app_id : “MEDIA-c21e3821”,
ad_spot: 9,
center : true
};
} else {
var adstir_vars = {
ver : “4.0”,
app_id : “MEDIA-c21e3821”,
ad_spot: 10,
center : true
};
}

1: 億ったーがお送りします 2021/04/14(水) 14:00:23.888 ID:67UZX8lm0
上司「わかりにくいからこれ」

おれ「しかし、このほうが簡潔ですしミスも少ないと思います…」

上司「変数名で1文字とかありえないから。まじで気をつけて」

おれ「…はい」

何故なのか
古臭いしきたりかなんかあるのかこの会社

   var group = document.getElementsByClassName(“kijinaka”);
   if(/smartnews.(com|be)/.test(document.referrer)) {
       group[0].innerHTML = ‘

‘;
   } else {
       group[0].innerHTML = ‘

‘;
   }

27: 億ったーがお送りします 2021/04/14(水) 14:08:23.048 ID:b1sqYeWpd
>>1
わからないことがあったら略

28: 億ったーがお送りします 2021/04/14(水) 14:08:49.031 ID:xYMgFPF40
>>1
コーディング規約があるだろ

52: 億ったーがお送りします 2021/04/14(水) 14:18:48.889 ID:e5RxJLSJM
>>1は情報系の学校に入ったピカピカの1年生と見た

75: 億ったーがお送りします 2021/04/14(水) 14:35:45.115 ID:v9/vVzZW0
これは>>1が無能

3: 億ったーがお送りします 2021/04/14(水) 14:01:14.374 ID:kxWNCT/N0
動くか確認するときは適当につけるけど上げるときはちゃんとした名前にする

6: 億ったーがお送りします 2021/04/14(水) 14:01:29.402 ID:43BQ0iJQ0
iは?

13: 億ったーがお送りします 2021/04/14(水) 14:03:23.067 ID:yE1qPuxP0
>>6
ループ専用ならまあ

7: 億ったーがお送りします 2021/04/14(水) 14:01:29.676 ID:yTWxUAK6a
普通だな

8: 億ったーがお送りします 2021/04/14(水) 14:01:37.512 ID:kBVrmKPwd
他の人がわからんだろ

9: 億ったーがお送りします 2021/04/14(水) 14:02:15.596 ID:ISHpFkla0
aとかbとかでも問題ないよな
わかりやすければ

10: 億ったーがお送りします 2021/04/14(水) 14:02:19.917 ID:Kjhf6rpAd
1文字だと検索かけるときに不便じゃね?

11: 億ったーがお送りします 2021/04/14(水) 14:03:00.173 ID:iR0xAhhJa
>>10
これ

15: 億ったーがお送りします 2021/04/14(水) 14:03:52.194 ID:ISHpFkla0
フレームワークとかアーキテクチャとか作るようなのだと名前つけたほうが分かりにくいこともある
業務ロジックのような末端のコードだと名前に意味をもたせたほうが分かりやすい

16: 億ったーがお送りします 2021/04/14(水) 14:03:53.566 ID:MUJewvTrd
短いブロックでの局所変数ならまだいいけど、
ブロックもない長い関数でそんなん出てきたらイライラする

20: 億ったーがお送りします 2021/04/14(水) 14:04:51.604 ID:QOnA3jst0
後から書き直す奴が見にくいってのはあるがお前しか見ないならそれでいいと思う

25: 億ったーがお送りします 2021/04/14(水) 14:07:56.874 ID:FqmJr3Gad
上司のフルネームにしとけ

62: 億ったーがお送りします 2021/04/14(水) 14:24:09.774 ID:Ygj3Cmp60
>>25
これすき

33: 億ったーがお送りします 2021/04/14(水) 14:09:09.768 ID:UIoX+neg0
思いやりプログラミングだよ自分のコードも次の日見たら何これなんだから

35: 億ったーがお送りします 2021/04/14(水) 14:09:51.038 ID:GAbkcQiO0
val s
s=Castle in the Sky.

43: 億ったーがお送りします 2021/04/14(水) 14:14:17.530 ID:3Ak+kbhtd
>>35
目がー

76: 億ったーがお送りします 2021/04/14(水) 14:36:15.610 ID:GAbkcQiO0
>>43
有能

40: 億ったーがお送りします 2021/04/14(水) 14:12:57.763 ID:kTA4WE8l0
そういうの全部規約があるだろ

53: 億ったーがお送りします 2021/04/14(水) 14:19:01.696 ID:dhJn200l0
>>40
必ずしも規約があってもそこまで規定されてない可能性は高い
aとかbとかつけてると多分契約断られるとおもう

45: 億ったーがお送りします 2021/04/14(水) 14:15:24.164 ID:O7muTR4bp
自分が何やってるかわからなくなるからお名前つけるわ
部分一致の処理もよくやるから名前決まってた方がやりやすい><

60: 億ったーがお送りします 2021/04/14(水) 14:22:12.169 ID:0ssUr8ej0
1年後2年後にバグとか問い合わせとかで調査するとき、一目で何やってるか分かる名前にしないと死ぬぞ
お前がやるならまだいいけど別の新人が担当したらそいつが死ぬぞ

引用元: https://hebi.5ch.net/test/read.cgi/news4vip/1618376423/

if(navigator.userAgent.indexOf(‘iPhone’) > 0){
var adstir_vars = {
ver : “4.0”,
app_id : “MEDIA-c21e3821”,
ad_spot: 9, center : true
};
} else {
var adstir_vars = {
ver : “4.0”,
app_id : “MEDIA-c21e3821”,
ad_spot: 10,
center : true
};
}

1001:人気記事紹介

.ninki{ clear:both; margin-bottom: 15px;}
.mm-ttl{ font-size:14px;}
.mm-ttl .Ttl-mil{ font-size:15px ; color: gray;}
.mm-ttl .Ttl-mil span{ color: rgb(0 128 0); font-size: 15px ; font-weight: bold;}
.at_out ul{-webkit-padding-start: 0; margin: 0 ; padding: 0;}
.at_out li{ list-style-type:none ; font-size:18px; line-height:1.2; font-weight: bold; margin-left: 0 ; margin-bottom: 20px ;}
.at_out li a{ text-decoration:underline; color:rgb(0 102 204); list-style-type:none;}
.pc_none{ display:none;}
.sp_none{ display:block;}

$(function(){
var c = 0;
if($(“div.mtpro-tweet-outer”).length){
var ad = “div.mtpro-tweet-outer”
}if($(“.twitter-tweet”).length){
var ad = “.twitter-tweet”
}if($(“div.t_h”).length){
var ad = “div.t_h”
}
$(ad).each(function() {
if (c !== 0) {
if(c == 4 ){
$(this).before(‘

‘);
}
else if(c == 8 ){
$(this).before(‘

‘);
}
else if(c == 12 ){
$(this).before(‘

‘);
}
else if (c >= 16 && c%4 == 0) {
$(this).before(‘

‘);
}
}
c++;
});
});

Source: 億ったー
【うーん】プログラマー新人おれ「変数名はaとかbでええやろ」 上司「あのさぁ」

okutta