「ファイル名に”yyyymmdd_hhmmss”がついてますが同タイミングで動かしたらどうなりますか」とレビューされて震えた話

リンク 朝日新聞デジタル マイナカードで誤交付、自治体・富士通に報告求める 個人情報保護委：朝日新聞デジタル マイナンバーカードを使った証明書のコンビニ交付サービスで、他人の住民票などが誤って交付された問題をめぐり、個人情報保護委員会は11日、関係自治体やシステムを納入した富士通Japanに対して、報告や資… 9 ゆーしゃん.py(bot運用中) @Mr_mura_ura 客「ファイル名に「yyyymmdd_hhmmss」がついてますが、完全に同タイミングで動かしたらどうなります？」 私「こんな小規模な社内システムでそんな事無いと思いますが…エラー落ちしますね…」 客「まぁ99%無いと思いますが、念のためにね」 というレビューをつい最近受けた私、震えてます← 2023-05-11 00:02:16

[morimarii]

重複しない方式で実装するのも大事なんだが問題は「重複したときどうなるか」なんだよ。別に大したことにならないなら「yyyymmdd_hhmmss」でもよい。リスク評価してそれに応じた方法を取ることが重要

[naoto_moroboshi]

この話はさておき、現実的に考えてありえんやろ……みたいなシーンを想定してどこまで金と時間と労力かけるのかって話はかんがえんとあかん。

[Fushihara]

重複を想定して、重複したらちゃんとエラーになるとか 1秒待ってリトライするとか 想定済みなら全然問題ないよ

[punychan]

ファイルの有無確認じゃ、両者同時に確認して、ないと判断してしまうだけだぞ。まっとうにロックするしかない。あるいはUUIDみたいに確率的にゼロとみなすか。

[nanakoso]

実際の数字は忘れたけどタイムスタンプのミリ秒は当てにならないぞ（同じタスク／スレッドコンテキスト内だと同じ値になることが多いので思ったよりずっと重複しやすい）

[prozorec]

ファイル名をユニークにしたいならtmpfileの様なライブラリ関数を使うべきで、新たに作るのは無駄だ

[the108simulacra]

これの対応策が「ミリ秒にする」な人はちょっとセンスない感じ・・/乱数はテスタビリティ下がるのでやめて欲しい

[itotto]

ファイルの有無を確認してあったら再度名前とればいいだけでは？乱数振ったってかぶるときはかぶるでしょ。/ ファイルの名前の話だからこれしか書いてないだけでこれだけでいいとは言ってないんだけど。

[sabacurry]

まぁおれは(poorな)乱数で重複された経験あるからな…。適切な乱数であって欲しい。

[Knoa]

増田の記事idなんかもYYYYMMDDhhmmssになってるけど、ssまで重複したら自動でユニークになるまで+1されるんかなぁとか思ってた。検証はしてない。毎秒1記事を超えたら繰り上がりで25時とか32日とか出てくるんかな。

[versatile]

かぶったら (1) をつけろよデコ助野郎

[theta]

システム時計のミリ秒なんて16ms単位だったりするので秒単位に比べて同時刻実行時の衝突確率が60分の1程度になるだけで衝突はしうるでしょ

[miruto]

PHPならdate('YmdHis')に加えてuniqid関数を使えばよい。

[abstruct3431]

「uuidなら重複した時の処理とか書かないけど、あり得るとか言う人は書いてるもんなの？」ってコメントがあるけど、書くだろ？ほんの僅かでも可能性がある以上起こることを想定するのが設計の常識でしょ？

[Finding]

へっぽこエンジニアとしてはコメント欄がほんと勉強になっている

[masaru_al]

『tmpfile関数やGetTempFileNameA関数みたいな物が用意されているのに』ただ単に知らないだけなんだと思う。いま知っている文法とAPIだけを駆使して既に有るものも頑張って作ってしまうのは初心者あるある

[kondoly]

重複する可能性は保持してる対応容量内だけなので重複確認してから発行するか自動加算のID付与するだけでも良いかと。双方向なのに初動が取得情報のみを基軸にするのは油断が過ぎるかと。

[civicpg]

uuidなら重複した時の処理とか書かないけど、あり得るとか言う人は書いてるもんなの？

[anigoka]

客ｓ「せーの、リリース！」

[onefootinthegrave]

windowsだとそもそもtimerの精度が15msしかなかったような。

[kagehiens]

まぁ、DB使っていいなら採番テーブルも援用すれば良いし、無理ならセマフォとかで整理券番号取る仕組みを組めば万全だわ。

[ryouchi]

端末番号をファイル名に入れておけば重複しないような気がする

[yoshitsugumi]

a

[kura-2]

PIDくらいは入れといたらどうか

[prjpn]

ファイル作るなよ

[metamix]

タイムスタンプだろうがUUIDだろうが重複はあり得る（理屈上は天文学的数字だろうがID生成の実装がそうであるとは限らない）んだから、重複しても大丈夫なようにするべき

[vanish_l2]

データクレンジングするとありえない重複なんていくらでも出てきがちなんで、まあ、そこまて考えていくべきかどうかは要件によるかも

[cl-gaku]

普通そう。富士通にはそのレベルの技術者はおらんということ

[hatomugicha]

ここ数年でTwitterを見だしてこの手のIT系の話題にも明らかな広告目的なり集客目的なものがあってしばらく見てた後にbioを見れば何となく分かると知識を得てこの方はたぶん違うかなと思った

[byaa0001]

"事象が10件/日くらいしか発生しなくとも、秒までのタイムスタンプは３年に１度は衝突する。事象が平日の昼間に集中している場合は毎年のように衝突する。N個の選択肢があると概ね√N 回の無作為選択で衝突する。"

[otoan52]

"正直に申し上げて「yyyymmdd_hhmmss」が付いていれば重複しないと思い込んでいた" "私作成の業務改善ツールをブログに纏めてます" どういうこと…？/UUIDとかちゃんと研究されてるもの使おうよ。種類色々あるからさ…。

[harumomo2006]

「管理者を全部削除することなんてありえないから！」という客の必死の説明を受けて、管理者アカウントを含めて全部ユーザー管理画面から削除できるという恐ろしいシステムを納品したことあるんだけどどうなったかな

[Cru]

プロセス番号付加でいいんでない?？乱数は重複するが。1秒以内にプロセス番号一周はありえない。つかこれ川崎のコンビニ戸籍謄本不具合の報道資料から客がつけたクレームじゃん？むしろ富士通のレベルの低さに震えた

[beyoooooooon]

排他制御入れてれば乱数なんて言い出すわけないし、排他制御入れればいいだけなのになんで余計なことを考えるのか謎。大体、排他制御入れてないのにエラーで落ちると言い切れるのはなんでだ。

[tomokatz]

卒論書いてる時に、ファイル_[0:99]をファイル_[1:100]にリネームするスクリプトを書いて実行したらファイル_100の1ファイルだけになって戦慄したことを思い出した。1を2に上書き、以下同となってた。実行前で良かったね。

[everybodyelse]

重複してもリカバーできてりゃいいし、最悪ユーザーがリトライボタンを押せばなんとかなるなら問題ない。まずいのはそれでシステムの可用性が失われたり、重大なインシデントが起きないかということ。

[m50747]

富士通Japanの件はこれじゃないかと思ってる。 webサービスは絶対に時刻をユニーク情報に使わない。言語の乱数も。データベースのオートインクリメントのPKかOpenSSLを使おう。さらに自システムのユーザーIDをプラスする。

[yuuAn]

排他制御も忘れずにやろうね。

[vegnpomn]

ミリ秒まで取ればタイムスタンプでユニークになるだろ！と思うようなエンジニアでも知ってる時刻を取る処理って、あんまり精度良くないから処理が早いと実際msレベルでも見た目ほどはバラけないんよね

[nisisinjuku]

良い子のみんなは「なんとかjapan」みたいにならないように一意のID番号を採番しようね☆