yogomi/dotfile

dotfile's Issues

変数名を略しすぎ

関数内で使われている一時的な変数は、関数を作った人が責任を持つものなのである程度、作成者だけがわかればいいところはあるのですが、流石にちょっと略しすぎな部分があります。

std::string buffer
std::vectorstd::string v;
std::ifstream ifs(inifile);

あたりです。

item もちょっと分からないかも。

例えば、 buffer ですが、これが使われている行

index = buffer.find("=");

を見ても、bufferがなんなのかは

while (std::getline(ifs, buffer)) {

まで遡らないと分かりません。

buffer を reading_line とかにするだけで、使われている行を見るだけで一目瞭然です。

簡単な変数名は、変数名をつけるときには時間を取られないので書くときは速いのですが、何かバグがあったりして修正しようとして読み直すときに時間がかかるようなコードになっていきます。
工夫して行って見てください。