プログラマ俺、コードを読み込めていないまま修正をしコードレビューでボコボコにされる

[0001 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:54:25.110 ID:MQesB6xNM]

コメントもドキュメントもない巨大コードに修正入れるのキツすぎる

[0002 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:55:46.004 ID:8esln431a]

レビュアーは何を根拠にレビューしてんの？

[0003 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:57:02.151 ID:MQesB6xNM]

ロジック3回ほど確認しても確認漏れがあってつらい

[0004 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:57:54.856 ID:MQesB6xNM]

>>2
コード
レビュワーはその部分に詳しい人に頼む

[0005 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:58:14.201 ID:fd+4vZOG0]

>>1
修正したものを手元で走らせてみないの？

[0006 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:58:57.025 ID:MQesB6xNM]

>>5
走らせられないコードがあってさ
詳しくはいえんのだけど

[0007 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 22:59:25.585 ID:btGJFZEza]

正直みてらんねえコードは確かにある

[0008 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:00:01.237 ID:8esln431a]

修正しろって言われる部分がコーディング技術によるものか仕様の不理解によるものかで変わってきそうだな。どっちにしてもやりたくない仕事だな

[0009 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:00:53.110 ID:fd+4vZOG0]

自分はこのぐらいの長さまでしか動作未確認で出せる自信ないよ。
uint8_t timerIRQ[TIM_NUM] = {TIMER_IRQ_0, TIMER_IRQ_1, TIMER_IRQ_2, TIMER_IRQ_3};
typedef struct timerInfo {
bool used = false;
uint32_t delayUS = 0;
bool isOneShot = false;
// pointer to alarm callback function.
void (*callback)(void) = NULL;
} timerInfo_t;
timerInfo_t gTimerInfo[TIM_NUM] = {0};

// alarm interrupt handlers.
static void commonAlarmHandler(void) {
for(int16_t i = 0; i < TIM_NUM; i++) {
// which timer IRQ fired?
if(timer_hw->intr & (1u << i)) {
hw_clear_bits(&timer_hw->intr, 1u << i);
gTimerInfo[i].callback();
// set next alarm if interval mode.
if(gTimerInfo[i].isOneShot == false) {
restartIntervalTimer(i);
// clear timer info if oneshot mode.
} else {
gTimerInfo[i].used = false;
}
// done.
break;
}
}
}


// set interval timer.
int8_t startIntervalTimer(uint32_t delayUS, void (*callback)(void)) {
for(int16_t i = 0; i < TIM_NUM; i++) {
if(gTimerInfo[i].used == false) {
hw_set_bits(&timer_hw->inte, 1u << i);
gTimerInfo[i].used = true;
gTimerInfo[i].delayUS = delayUS;
gTimerInfo[i].isOneShot = false; // set to interval mode.
gTimerInfo[i].callback = callback;
irq_set_exclusive_handler(timerIRQ[i], commonAlarmHandler);
irq_set_enabled(timerIRQ[i], true);
uint64_t target = timer_hw->timerawl + delayUS;
timer_hw->alarm[i] = (uint32_t)target;
return (int8_t)i;
}
}
return -1;
}

[0010 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:01:15.186 ID:7jbwiv9BM]

走らせられないで修正とかどう考えても無理だろ

[0011 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:01:23.448 ID:8esln431a]

見てらんねえコードに修正入れなきゃいけない時は泣ける。まるまる書き直したくてもできないとき

[0012 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:01:40.397 ID:v4GRpBIt0]

LGTM

[0013 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:02:45.679 ID:8esln431a]

>>9
その長さでも自信ないな

[0014 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:03:11.106 ID:btGJFZEza]

変数名を変数で生成すんじゃねーよ

[0015 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:03:15.779 ID:MQesB6xNM]

>>8
俺の経験不足のせいでも仕様の理解不足と巨大で複雑なコードのせいでもある

[0016 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:03:48.344 ID:MQesB6xNM]

>>10
ある程度のテストは書かれてるけどそれ以上のテストは無理

[0017 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:04:23.692 ID:MQesB6xNM]

>>11
本当にそれ

[0018 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:04:41.589 ID:MQesB6xNM]

>>9
10倍はあるよ

[0019 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:06:28.552 ID:MQesB6xNM]

プログラミングの技術はあるけど読みにくいことこの上ないコードを書く人、なんなん
勘弁してほしい

[0020 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:06:51.548 ID:fd+4vZOG0]

>>18
それはかなりつらい作業だね。
適当なドライバーとスタブ書いてとりあえず走らせることも不可？状況が良く分からないんだよね。
実行環境無しでコーディングはやったことないな。
プロ中のプロなのかな？

[0021 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:07:01.269 ID:MQesB6xNM]

テスト困難なコードを書くのやめてくれ

[0022 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:08:05.195 ID:MQesB6xNM]

>>20
外部の処理に依存していてかつテスト出来ない設計になってて無理
書き直すには工数がでかすぎる
ある程度安定はしているので正直動かしたくない

[0023 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:09:03.661 ID:f08vEeXH0]

分からないのに分からないと言えない人？
犯罪者だな

[0024 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:09:04.273 ID:pmsAw0iO0]

>>19
成績は良いけど教えるのは下手だった人いるじゃん？
あれと本質は一緒

[0025 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:09:11.060 ID:fd+4vZOG0]

へー、そんなクソ言語も世の中にあるんだ。やったことないな。

[0026 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:09:56.122 ID:4/8LMM3f0]

最近はChatGPT4に頼りっぱなしだ。基本的なコードレビューならやってくれて助かるよ。

API連携したり、プロンプトがもうちょっと巨大なデータを把握できるようになれば、人間でのレビュー作業がほんとに少なくなると思う。

[0027 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:12:55.135 ID:fd+4vZOG0]

>>22
どんな言語・環境か分からんけど

外部の処理に依存していて
→この部分を自分勝手にスタブとドライバで代用して、
かつテスト出来ない設計になってて
→ここは自分で書いたコードだからいくらでもデバッグ出力なりなんなり埋め込めるはずだけど、
こういうのが許されない開発現場ってあるんだね。世の中広いな。

[0028 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:16:58.899 ID:MQesB6xNM]

>>25
うーん
そもそもAPIを叩くとかではなく提供されているライブラリを使うので一応モックアップは出来るんだけどそうするためには8千行のクラスを書き換えなきゃいけなくてとても辛いんだよ

[0029 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:18:24.920 ID:MQesB6xNM]

>>27
特殊な部分なので無理なんよ
別の部分はここまでテストしにくくない

[0030 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:19:13.126 ID:fd+4vZOG0]

難しいシステムを扱っているんだね。自分には無理だわ。

[0031 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:19:48.564 ID:MQesB6xNM]

>>30
コードを見ずに理解するほうが無理だわ

[0032 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:31:21.984 ID:gW4+5Lgh0]

転職したほうがいいよ

[0033 以下、5ちゃんねるからVIPがお送りします 2023/08/14(月) 23:44:58.081 ID:MQesB6xNM]

>>32
それしか言わんよね