X

+ JavaScript の質問用スレッド vol.122 + [転載禁止]©2ch.net

■ このスレッドは過去ログ倉庫に格納されています
0001Name_Not_Found
垢版 |
2014/12/12(金) 23:52:28.79ID:???
JavaScript に関する何でも質問スレです。
お気軽にどうぞ。
0041Name_Not_Found
垢版 |
2015/01/02(金) 02:09:48.36ID:???
>>39
それはどうでもいいところを統一しようとするから
そんなことになるんだよ。

コードレビューで指摘するのは、どうでもいい規約を守らせるためじゃなくて
問題点。悪い所を直すこと。AとBがあったとき、明確にAの方が
優れている場合に指摘すればいいんだよ。

AもBもどっちも書き方の違い的な場合は、指摘する必要はない。
こっちのほうがいいんじゃないですか？ぐらいの提案はするけどな。

コーディング規約でどうでもいいことを厳密に守らせようとするから
レビューアの好みとかそういう話が出てくるんだよ。

な？ （絶対守る）規約はデメリットばかりだろ？
0042Name_Not_Found
垢版 |
2015/01/02(金) 08:06:26.70ID:???
コードレビューで質が維持されるってのは同意だな
ゲームの攻略wikiのような参加者の文章レベル低そうなところでも
更新のたびに多数が相互に推敲繰り返すから文体が統一されたり、自然とそつのない文になってくだろ
複数人作業の醍醐味だな
酷いものは自然と淘汰されんだよ

大抵の人間にはそういう最低限の判断センスがあるんだが…
>>30の例えを借りると、原稿の中身の良さはレビューしないとどうにもならんが、原稿サイズでさえ合わせてこないやつがたまにいるので規約がある

世の中にはいるんだよ、既存のスペース4インデントのソースに、タブ混ぜて来るようなやつが
スペース2かスペース4かというのは宗教だが、スペースにタブを混ぜてなにも感じないのはおかしい
プログラミングのレベルの問題じゃなくて、そういう違和感が絶望的にないやつがいるから規約がなくならない
あとはどっちでもいいけど宗教論争になるとどっちでもいいだけに不毛な所を先に決めておく的なやつ
0043Name_Not_Found
垢版 |
2015/01/02(金) 09:53:37.57ID:???
>>40
レビュー者の評価が正しい事は誰が保証するんだ？
レビュー者AとBの評価が違ったらどうする？
独断と偏見のレビューなど、何の役にも立たん
0044Name_Not_Found
垢版 |
2015/01/02(金) 10:01:42.11ID:???
>>41
> コーディング規約でどうでもいいことを厳密に守らせようとするから
> レビューアの好みとかそういう話が出てくるんだよ。
逆だろ
レビュアーに一任するからレビュアーの好みが反映されるんだろ
個人間でチームを組むなそれでいいが、会社としてやっていくなら個人の好みが反映される仕組みにするべきではない
何のためにGoogleがガイドラインを作っていると思ってるんだ
守らなくていいガイドラインなら作る必要はないんだよ
0045Name_Not_Found
垢版 |
2015/01/02(金) 11:13:20.85ID:???
>>43
> レビュー者AとBの評価が違ったらどうする？
そんなことAとBで話し合って決めろよ。
コミュ障かよ？

それともコーディング規約を作れば解決できる問題なのか？
今度はコーディング規約内容をどうするかでAとBが対決するだけだろ。
0046Name_Not_Found
垢版 |
2015/01/02(金) 11:22:10.95ID:???
>>42
> 世の中にはいるんだよ、既存のスペース4インデントのソースに、タブ混ぜて来るようなやつが

そういうのは、技術が低いからなだけ。
スペース派、タブ派の違いはいるが、混ぜる派はいない。
技術が低いから混ざってしまう派だ。

そういう奴は技術が低いのが原因。たかがスペースなどという
どうでもいいことを揃えた所で技術力は高くならない。
どちらにしろそいつのコードは信用出来ないんだよ。
0047Name_Not_Found
垢版 |
2015/01/02(金) 11:28:50.32ID:???
いまどきそんな混雑させるエディタを使うのが惡い
0048Name_Not_Found
垢版 |
2015/01/02(金) 14:59:59.34ID:???
レビュー出来るぐらい優秀ならば、その分他人が書くべきコードを引きとって
コードをバリバリ書いた方が結果的にはいいものが出来るというジレンマ
0049Name_Not_Found
垢版 |
2015/01/02(金) 15:01:20.19ID:???
コードレビューできる＝優秀なのか・・・
普段どんな酷いコードばかり書いて
それでシステムが動いているんだろうか。
客には言えないな。
0050Name_Not_Found
垢版 |
2015/01/02(金) 15:08:30.03ID:???
>>49
コードレビューできる＝優秀
もちろんそうだ
0051Name_Not_Found
垢版 |
2015/01/02(金) 15:13:12.40ID:???
>>49
他人がするコードレビューはほとんど無意味だと思ってる
それをする意味があるのは、本当に優秀なやつがコードを本気で直す時だけだ
0052Name_Not_Found
垢版 |
2015/01/02(金) 15:27:10.57ID:???
>>45
おまえはコーディング規約が不要という意見ではなかったのか？
「規約は不要、レビューすれば十分」といっていただろうが
規約が必要ならレビュー前から守らせろよ
0053Name_Not_Found
垢版 |
2015/01/02(金) 15:28:46.49ID:???
>>52
だから規約は不要だって。
AとBが話し合えばいいだけの話。
そんな細かいことにいちいち規約を作るな。
0054Name_Not_Found
垢版 |
2015/01/02(金) 15:30:42.28ID:???
>>50-51
プログラマなら普通だれでも相互にレビューするだろ
まさかプログラマの壁を超えた奴が一人もいないのか？

プログラマがいないのに、
どうやって規約なんて作れるんだ？
0055Name_Not_Found
垢版 |
2015/01/02(金) 15:32:50.42ID:???
正しいコードを採用すればいいだけの話。
どちらが正しいかは論理的に考えれば
だれでも同じ答えにたどり着くだろ。

レビューで指摘するのはそういうこと。

それでもなお、意見が別れるものに関しては
どちらでもいいから、そんなものは規約で決める必要がない。

という話なんだが？
0056Name_Not_Found
垢版 |
2015/01/02(金) 15:40:32.74ID:???
>>54
しないよ
どんだけ暇なんだよ
大量のコードを書く職場だったらどんだけ非効率な事か分かるだろ
もちろん本人はレビューとテストを徹底的にするがな
0057Name_Not_Found
垢版 |
2015/01/02(金) 15:51:46.94ID:???
> 大量のコードを書く職場だったらどんだけ非効率な事か分かるだろ

大量のコードを書かなくていいようにするために
レビューするんだけど？

レビューしないと、質の悪いコードになる
質の悪いコードとは、コードの量が多くて
コードの見通しが悪いコード

誰もそれを指摘しない。

そしてそれを参考にして、他の人が書く。
その時、コピペしてそのまま使えないからと一部だけ修正する
最初のコードよりもさらに質が落ちる。

そしてそれを参考にして、以下繰り返し。
でどんどん破綻していくんだけど。
0058Name_Not_Found
垢版 |
2015/01/02(金) 16:12:03.24ID:???
>>57
> 大量のコードを書かなくていいようにするために
仕事の規模が大きかったらどうすんだよw

> 質の悪いコードとは、コードの量が多くて
完全にダウト
コードの量と質は全く関係無い
非常に複雑で大きなコードを書く事だってあるが、複雑で大きいコードだからと
いって悪いコードなんて事は全くない
それを他人が一々レビューするのは全く非効率

誰でも担当外のコードをすぐ見て理解出来る程度の仕事規模なら、レビューでもすればいいよ
0059Name_Not_Found
垢版 |
2015/01/02(金) 16:12:47.48ID:???
>>53
事前に話し合いでルール(規約)を決めておかないわけ？
問題が起きてから相談してルールを決めるのは効率が悪すぎる上に致命的な問題だったら放題な時間を費やすことになる
0060Name_Not_Found
垢版 |
2015/01/02(金) 16:23:46.02ID:???
バグを他人に発見されるのは恥かしい事なんだよ
他人にレビューしてもらってバグが発見されるようなら、プログラマとして終わってる
ってぐらいの緊張感とプライドでコードを書く必要があるんだよな

それでも、ある時期には一旦コードの更新を止めて全員でバグチェックとレビューはした方がいいだろうね

その時はマジでやるかやられるかぐらいの緊張感があるべきだね
0061Name_Not_Found
垢版 |
2015/01/02(金) 16:24:46.91ID:???
>>58
> コードの量と質は全く関係無い
関係ある。コードの質は客観的な方法で計測可能。そのための指標はいくつかあるが、
コードの量が計測に入っているものも多い

例えば
http://msdn.microsoft.com/ja-jp/library/bb385914.aspx
> コード行 - コード内の行の概数を示します。
> 数が非常に大きい場合、型またはメソッドでの処理が多すぎるため、
> 分割が必要であることを示すことがあります。

>>59
決めておかなくても、技術力があれば
問題ないコードをかけるから、心配する必要はないし、
コーディング規約なんかで防げるのは
コードの質の全体の5％ぐらいなもん。

問題が起きるならば、起きる人の技術力が低いわけで
技術力が低い人が、決められたルールを守った所で
もっと大きな問題が起きる。

前提として、コーディング規約では致命的な問題を解決することは出来ないってことを知るべき。
0062Name_Not_Found
垢版 |
2015/01/02(金) 16:28:49.84ID:???
> それを他人が一々レビューするのは全く非効率

レビューに時間が掛かるから非効率だと考えてしまう。

ではなぜレビューに時間がかかるかというと

まさに、コードの量が多いから

このことで、コードの量が多いということが問題であると
はっきりわかっただろう？


指摘するべき問題があるのに、非効率という言葉で問題を解決しないことになる。
コーディング規約があってもそうなってしまっているということだ。

だからコーディング規約なんかに頼ることはやめて
レビューによる技術力アップをするべき。
0063Name_Not_Found
垢版 |
2015/01/02(金) 16:33:09.04ID:???
>>60
> バグを他人に発見されるのは恥かしい事なんだよ

念の為に言っておくと、コードレビューで行うのは
バグの発見ではない。だからコードレビューでテストをする必要もない。

もちろん質の悪いコードをみて、バグを見つけたら
指摘していいが、そっちはおまけ。

コードレビューでやるのは、コード質の向上や開発効率の向上。
レビュー時間を短縮するためにも、可読性が高いコードが必要になる。
レビューに時間が掛かるなんて言ってる時点で、
コードに大きな問題があるってことなんだよ。

コードレビューをやらないとどんどん開発効率は落ちていく。
コーディング規約程度では守れない問題。
0064Name_Not_Found
垢版 |
2015/01/02(金) 16:44:22.04ID:???
なお、コードレビューよりも効果が高いものがあって、
それはペアプロ。

実質リアルタイムにコードレビュー状態になるから
早いうちに問題点を取り除ける。

「早いうちに問題点を取り除く」の反対（悪い方）は
問題があるのにそれに気づかずに、コードがどんどん作られ
後になってバグの修正などでコードをみた時に
長く複雑なコードと対峙してコードを解析するのに時間がかかること。
問題を直すのは後になればなるほど時間がかかるからね。

普段他人のコードを"解析"することが多いならば大いに反省するべき。
コードはスラスラと読むものであって、解析するものではない。
っていうか、解析が必要な難問を自分らで生成してどうするよｗ
0065Name_Not_Found
垢版 |
2015/01/02(金) 16:46:17.71ID:???
コーディング規約は精神安定剤みたいなもんだろう。

これさえ守っていれば、コードの質は保たれるんだーみたいな。

実際には質が悪いにももかからわず、ただただ
精神を安定させるために使われてる。
0066Name_Not_Found
垢版 |
2015/01/02(金) 17:01:58.99ID:???
>>64
正直それは理想論だな
理想を否定する気はないが、本当に専門性の高いコードは他人が見ても分からない

単にコーディング規約を守ってるかぐらいはレビューすれば分かるが
それがコードの質を高める事に繋がるのか？

コードが長いか短かいかは誰がどういう判断で決めるんだ？
何度も言うが専門性が高いコードは安易に良い悪いは決められない

・コーディング規約を守ってるか？
・1つの関数が長くないか？

これだけだったら、新人教育という範疇だろう
それだったら、ちゃんと毎回レビューして指摘すべきだろうけど
0067Name_Not_Found
垢版 |
2015/01/02(金) 17:05:56.45ID:???
「とりあえず、やってみて問題が起きてから話し合う」って日本人だから出来る考え方だよな
空気を読んで足並みを揃えるってやつ
アメリカで同じ事が起きたら「決めておかなかったのが悪い」で一蹴される
0068Name_Not_Found
垢版 |
2015/01/02(金) 17:09:38.03ID:???
あと、新人教育やレビューは他人の為にやる事だから、ちゃんと職場の理解がないと
単に仕事が遅い使えないやつと思われかねない
難しい問題ではある
実力主義でやってるようなところは特にやりずらいだろうね
006943
垢版 |
2015/01/02(金) 17:13:24.88ID:???
結局、話し合いで決める規約が存在するんだろ
それをコーディング規約と呼ぶか呼ばないかの違いでしかない
文書化せずに口約束レベルで統一をはかろうってのは証拠が残らないし、後で齟齬が発生しても仕方がないと思うがね
0072Name_Not_Found
垢版 |
2015/01/02(金) 18:30:07.73ID:???
レベル差がなくても他人の目を通すとわかること結構あるよね
イラスト左右反転したらデッサンの狂いに気づくのに似てる

自分は5年位独学でやってて、そのあと偶然入った仕事でペアプロやる事になったんだけど
それまで自分のプログラミングってどっかおかしい？って自信持てなくなってたのが
あれ俺思ってたより悪くない？って確認できたのと
刺激と緊張感でスキルアップできたな
0073Name_Not_Found
垢版 |
2015/01/02(金) 19:59:38.93ID:???
>>72
ペアプロが有効なのは否定しない
人件費やお互いのプライドなんかの問題が無ければやるべきだとは思うよ
0074Name_Not_Found
垢版 |
2015/01/02(金) 20:06:17.47ID:???
ペアプロで試されるのはコーディング力より人間性ｗ
0075Name_Not_Found
垢版 |
2015/01/02(金) 20:09:36.46ID:???
>>72
スキルアップに有効なのは否定しないが、規約の必要性は別問題
0076Name_Not_Found
垢版 |
2015/01/02(金) 20:10:55.76ID:???
>>74
人間性も重要だろう
プログラムに限った事じゃないけど、平気でバグ出すやつとか滅茶苦茶気を付けてるやつとか
の違いって結局人間性の違いだしね
0077Name_Not_Found
垢版 |
2015/01/02(金) 21:39:46.24ID:???
>>67
> 「とりあえず、やってみて問題が起きてから話し合う」って日本人だから出来る考え方だよな
> 空気を読んで足並みを揃えるってやつ
> アメリカで同じ事が起きたら「決めておかなかったのが悪い」で一蹴される

それ、逆だぞ。

一個人のブログだが、いろいろ調べてみるといい。同じような話が見つかるはず。
http://mojix.org/2010/10/05/yatteminai-to-wakaranai

> ＜私は社内でアメリカ人からうんざりされることが多い。プロジェクトの業務要件を決める会議などで、
> レアケースも含めて網羅的に考慮ポイントを説明したりすると、最初のうちはうなずきながら
> 興味深そうに聞いているのだが、次第に疲れが顔にではじめ、ついには「いや、色々あるのはわかったけど、
> とりあえずやってみて、駄目ならその時考えよう」と言われることが非常に多い＞。
>
> ＜アメリカ人は総じて、事前に考慮事項を洗い出し、シミュレーションして対策を講じるというのが苦手で、
> あれこれ考える前にとりあえずやってみようというアプローチをとることが殆ど。そういう点でアメリカ人は
> 事後にストップする仕組みをきちんと整備しているというよりむしろ、うまくいかなかったら軌道修正したり、
> 場合によってはストップすることを前提に物事に取り組むといったほうが正確だろう＞。
>
> これは面白い。アメリカ人は総じて、「とりあえずやってみて、ダメだったらやめる」という考え方をする、とのこと。

だいたい、最初に問題なんて全部洗い出せるわけがないわけで。
0078Name_Not_Found
垢版 |
2015/01/02(金) 21:40:47.53ID:???
>>66
> コードが長いか短かいかは誰がどういう判断で決めるんだ？

コードメトリクスで検索しろ。
0079Name_Not_Found
垢版 |
2015/01/02(金) 21:42:58.45ID:???
>>75
> スキルアップに有効なのは否定しないが、規約の必要性は別問題

規約を過大評価しすぎ。あってもなくても
コードの質に大きな影響を与えない。

コードレビュー（ペアプロ含む）をやれば、
大きな効果があるし、規約の内容をカバーできる。
0080Name_Not_Found
垢版 |
2015/01/02(金) 23:18:15.86ID:???
aaaっていうclass名のついた要素のidを取得したいんですけど、どうやったらできますか？

<div id="????(変わるが常に何かしらのIDがついている)" class="aaa"> </div>

<script>
element = document.getElementsByClassName("aaa");
alert(element.id);
</script>

だとunderfinedになってしまいました。
よろしくお願いします。
0081Name_Not_Found
垢版 |
2015/01/02(金) 23:27:31.33ID:???
>>80
aaaが複数ある場合はどうするの？

var id = $(".aaa").attr("id");　// aaaが複数ある場合は最初のものを参照する
0082Name_Not_Found
垢版 |
2015/01/02(金) 23:30:35.72ID:???
aaaがない場合も考えられるな
0083Name_Not_Found
垢版 |
2015/01/03(土) 09:21:40.43ID:???
>>80
<script>
elements = document.getElementsByClassName("aaa");
alert(elements[0].id); //最初のclass="aaa"のidを取得
</script>

elements.lengthでclass="aaa"の個数が分かるから、条件分岐とかで適宜使うといい

あと>>81はjQueryっていうプラグインを使っている
そのままじゃ使えないから注意な
0084Name_Not_Found
垢版 |
2015/01/03(土) 09:25:39.14ID:???
jQueryはプラグインじゃなくてライブラリな。
JavaScriptではDOM操作をする時によく使われているライブラリ。
Microsoftも採用しているよ。
0085Name_Not_Found
垢版 |
2015/01/03(土) 09:35:07.94ID:???
>>83さんのでできました！ありがとうございます！
81,82,84さんもありがとうございました！
0086Name_Not_Found
垢版 |
2015/01/03(土) 10:21:40.85ID:???
また質問で申し訳ないのですが、
var hoge="aaa";
document.getElementsByClassName(hoge);
ってすることはできないのでしょうか？
0088Name_Not_Found
垢版 |
2015/01/03(土) 10:45:56.57ID:???
聞く前にコンソールで実行してから聞いたの？
試しもせずに何でも聞くなよ？
0090Name_Not_Found
垢版 |
2015/01/03(土) 10:53:41.81ID:???
自己解決したのならその詳細を事細かに書け
他の人の参考にならないだろ。
0091Name_Not_Found
垢版 |
2015/01/03(土) 12:19:15.94ID:???
>>88
実行したのですが、動きませんでした
どこか間違ってないか確認してみます
>>89は私じゃないです
0092Name_Not_Found
垢版 |
2015/01/03(土) 12:46:52.13ID:???
じゃあ動かない。が答えですよ
0093Name_Not_Found
垢版 |
2015/01/03(土) 13:19:36.11ID:???
>>91
まず基礎から勉強したほうがいいよ
はっきりいうけど基礎も分かってない
0094転載禁止アフィカ.ス死ね
垢版 |
2015/01/03(土) 14:24:15.15ID:ZWtfUb5U
正月から勢い強すぎだろ
0096Name_Not_Found
垢版 |
2015/01/06(火) 16:30:46.08ID:???
var x=5;
if(
x.match(/\d/)
)
{
alert("数字")
}

これだと動かないけど
var x="5";
にすると動きます

matchって数値の変数には対応してないんですか？
0097Name_Not_Found
垢版 |
2015/01/06(火) 16:47:45.37ID:???
String型のメソッドだからそうだろうね
0099Name_Not_Found
垢版 |
2015/01/11(日) 21:56:56.00ID:???
ES6のmoduleって名前空間と何か違うんですか？
ES4の名前空間は二度と考慮しないって言うのはどうなるんですか？
0101Name_Not_Found
垢版 |
2015/01/22(木) 20:52:50.10ID:???
lodash 3.0 リリース間近！

https://github.com/lodash/lodash
3.0-preから-preが外れました！


スレが多すぎてどこに書けばいいかわからないので
関連スレすべてにマルチポストしています。m(__)m
0102Name_Not_Found
垢版 |
2015/04/22(水) 21:08:11.10ID:a1K1Ys+O
☆ 日本の核ブ装は絶対に必須ですわ。☆
http://www.soumu.go.jp/senkyo/kokumin_touhyou/index.html

☆ 日本国民の皆様方、2016年7月の『第２４回 参議院選挙』で、改憲の参議院議員が
3分の2以上を超えると日本国憲法の改正です。皆様方、必ず投票に自ら足を運んでください。
私たちの日本国憲法を絶対に改正しましょう。☆
0104Name_Not_Found
垢版 |
2015/06/04(木) 14:37:17.17ID:???
PhantomJSに詳しい人、ボタンのクリックの仕方教えてください。

入力はうまく出来ますが、なぜかボタンが押せません。
0105Name_Not_Found
垢版 |
2015/06/04(木) 23:26:01.68ID:???
>>104
どんなページで、どんなコードを書いたのか
再現できるコードを載せろよ
0106age
垢版 |
2015/08/09(日) 22:54:27.68ID:???
age
0108Name_Not_Found
垢版 |
2015/11/07(土) 17:20:50.82ID:vT73by48
(´；ω；`)
0109Name_Not_Found
垢版 |
2016/01/28(木) 12:12:56.20ID:KHYBW4Xb
　
お世話になります。
私、責任者の加茂と申します。以後、宜しくお願い致します。
http://homepage2.nifty.com/e-d-a/scurl/ays.html
　
　http://homepage2.nifty.com/e-d-a/scurl/SW-pos.html
　http://homepage2.nifty.com/e-d-a/scurl/SW-sp.html
　http://homepage2.nifty.com/e-d-a/scurl/SW-BB8.html
　
浪速建設様の見解と致しましては、メールによる対応に関しましては
受付しないということで、当初より返信を行っていないようで、今後につい
てもメールや書面での対応は致しかねるというお答えでした。
　http://www.o-naniwa.com/
このように現在まで６通のメールを送られたとのことですが、結果一度も
返信がないとう状況になっています。
　
　http://homepage2.nifty.com/e-d-a/scurl/ia-1-3.html
　http://homepage2.nifty.com/e-d-a/scurl/ia-2-1.html
　http://homepage2.nifty.com/e-d-a/scurl/ia-3-1.html
　
私どものほうでも現在までのメール履歴は随時削除を致しております
ので実際に１１通のメールを頂戴しているか不明なところであります。
　
●クリスタル通り122号室の入居者
●浪速建設 女事務員 南野 東条　●アパマンショップ八尾店　加茂正樹
　
!!!!!!!!!!!!!!!
0115Name_Not_Found
垢版 |
2018/05/01(火) 20:43:02.30ID:l1wYHpV1
誰でもできる在宅ワーク儲かる方法
少しでも多くの方の役に立ちたいです
グーグルで検索するといいかも『金持ちになりたい 鎌野介メソッド』

8BV06
■ このスレッドは過去ログ倉庫に格納されています
ニューススポーツなんでも実況