C/C++の宿題やらせてください。6代目。
■ このスレッドは過去ログ倉庫に格納されています
011184
NGNG本題とは少しずれるが、GetItem関数の第一引数は
constとしてポインタを変化させない方が親切だな。
同じ値を返しているんだから。
こうすることで、入力値を変化させずに
結果が得られるので少しは汎用的になる。
当然、GetItemにポインタ変数が1つ増えるが。
ちなみにGetItem関数のfor内にある初めのif文は無駄だと思う。
サイズが決まっているならポインタではなく
配列にしてもいいと思う。サイズ小さいし。
だから...
enum { PERSON_FNAME_SIZE=10, PERSON_LNAME_SIZE=12, PERSON_AGE_SIZE=3 };
struct person{
char all[PERSON_FNAME_SIZE+PERSON_LNAME_SIZE+PERSON_AGE_SIZE];
char fname[PERSON_FNAME_SIZE];
char lname[PERSON_LNAME_SIZE];
char age[PERSON_AGE_SIZE];
}p[NUMBER];
だね。で、GetItemにあるマジックナンバーは
enumで宣言したものに置き換える。
これでサイズ修正の際の修正箇所が減るね。
■ このスレッドは過去ログ倉庫に格納されています