神奈川県にも感染者拡大中…。
日本に限らず、世界でもどんどん感染拡大している状態になっています。
東京五輪までもう数日と言った状況。
バッハIOC会長も失言がもとで、もはや日本では何を行っても炎上しそうな感じになっちゃっています。何しに日本に来たんでしょうか？・・・という状態。
オリンピックも、今までに中止するという方向に舵を切るタイミングは何度もあったはずですが、結局この不安定な状況で開催というには本当にどうなんでしょうかね…と思ってしまいます。
無観客で開催するのはいいとしても、選手がクラスタ感染して死者が出た・・・なんてことになったら果たしてどうするつもりなのか。
せめてワクチン接種が終わることとオリンピック開催の引き換えの約束をして、それができないなら潔く諦める…というのも必要だったのではないかと思いますので。
命の危険を犯してまで開催に踏み切るのは、少なくとも私は正解とは思えません。
東京五輪を開催して、これでパンデミックや病床圧迫、医療崩壊を招いたら、誰が責任を取るのでしょうか？
このオリンピック、果たして一体誰のためにやるのでしょうか・・・。
そして、菅内閣の支持率も順調に低下中。
そりゃ、ここまでに何もしてこなかった、何もリーダーシップを発揮できなかった首相が支持されるわけもなく。
申し訳ないですが、官房長官として大変優秀だった人は、参謀としては向いていても、リーダーとしては向いていないことがより印象付けられた気がします。（以前の福田総理もそうでしたし）
この１年、振り返ってみても、結局、政府としてコロナ対策としてやったことといえば、緊急事態宣言の発令・解除の繰り返しだけしか印象がありませんので…。
菅政権が不時着する時も確実に近づいている気がしますね。
新人OJTネタです。
コーディングレビューでの一コマ。
今のプロジェクトでは、複数のプログラム言語を併用した開発を行っております。
今回は、C++でのコーディングで、コマンドのオプションの拡張。
オプションを追加することになり、その実装を担当させることになり、実装工程も終わり、コーディングレビューの工程へ。
オプションを追加して、そのオプション名のチェック処理ですが、あがってきたソースはこんな感じ。
引数ごとにbuf 変数に値を格納し、そのbuf変数の値がオプション名に一致しているかをチェックする処理ですが…
よくよく見るとこれはおかしいですよね。
文字列*s1 と文字列*s2 を先頭からn文字比較。'\0'以降の比較は行われない。
s1 > s2 で正の値が，s1 < s2 で負の値が，s1 = s2で 0 が返される。
sizeof(OPTION_NAME_STR) のサイズは '\0' までのサイズになるため、そのため、上記実装をよく見てみると、比較するサイズを sizeof(buf) - 1 とすると’\0' は比較対象に含まれないことになります。
仮に
とマクロ定義していたとすると、strncmp関数で sizeof(OPTION_NAME_STR) で比較されるのは '\0' ですが、「-1」しているために 'n' までになってしまいます。
こうすると、完全一致判定条件ではなく前方一致判定条件になってしまい、「option」で始まっている文字列であれば、何でも条件を満たしてしまいます。
例えば buf変数に「option2」とか「option test」とかが入っている場合でも、条件に当てはまってしまいます。
本人にこの実装おかしくない？と聞いてみると、「？？？？」という感じでした。
他に同じような処理があり、それを持ってきて実装してみてとりあえず動いたので大丈夫みたいな感じだったようですね。
なんで「-1」したのかについては、
があったからのようです。
strncpy関数で、コピー先の領域サイズから１引く理由は、strncpy関数は '\0' は自動付与してくれないので、sizeof(tmp) > sizeof(buf) の場合、buf変数の最後に '\0' が付与されないままになってしまいます。
このまま参照してしまうと危ないですね。
コーディング時間を短縮するためには、既存のコードを流用するというのは勿論アリだと思います。
ただ、何も考えずにコピペしてもダメで、その処理をきちんと理解した上で流用しているかどうかになりますね。
