指定した文字数、文字列がコピーされない

Question

前提・実現したいこと

C言語でコマンドラインのような入力処理を作っています。

発生している問題・エラーメッセージ

そこで、入力した文字列を1文字ずつ取り出して、判断しようと次のようなプログラムを組んだのですが、2つ目に取り出したい"obj(char型)"がなぜか指定した文字列数取り出してくれません。
例)"move GameSettingFile"と入力すると、cmdには"move"が格納されますが、objは"GameSett"までしか格納されませんでした。

該当のソースコード

#include <stdio.h>
#include <string.h>
#include <mbstring.h>
#include <windows.h>

char* extract_letter_func(char *);
int main() {
	char line[256];
	char cmd[8] = { "\0" };
	char obj[16] = { "\0" };
	printf("入力:");
	fgets(line, sizeof(line), stdin);//コマンドラインの取り出し
	rewind(stdin);

	char *h = line;
     //cmdの取得
	strncpy(cmd, extract_letter_func(h), 8);

	if (*h != '\0') {//objの取得(なしの場合、スルー)
		*(h += (strlen(cmd) + 1));
		strncpy(obj, extract_letter_func(h), 16);
	}



	printf("%s\n%s\n", cmd, obj);
}
char *extract_letter_func(char *h) {
	char letter[16] = { "\0" };
	char tmp[3];
	while (*h != ' ' && *h != '\0'&&*h != '\n') {
		switch (_mbclen((BYTE*)h)) {
		case 1:
			tmp[0] = *(h++);
			tmp[1] = '\0';
			break;
		case 2:
			tmp[0] = *(h++);
			tmp[1] = *(h++);
			tmp[2] = '\0';
			break;
		default:
			//error
			break;
		}
		strncat(letter, tmp, 16);
	}
	return letter;
}

試したこと

cmd,objはもちろん、strncatのバイト数を増やしてみたのですが、変わりませんでした。

補足情報（FW/ツールのバージョンなど）

Visual Studio 2017

Answer 1

extract_letter_funcの引数でcmdやobjのポインターと最大長を渡して、そのまま文字を設定していけばいいのでは。
たとえば、こんな感じ。

#include <mbstring.h>
#include <stdio.h>
#include <string.h>
#include <windows.h>

char *extract_letter_func(char *, char *, int);

int main(void)
{
    char line[256];
    char cmd[9] = "";
    char obj[17] = "";

    printf("入力:");
    fgets(line, sizeof(line), stdin);
    rewind(stdin);

    char *h = extract_letter_func(line, cmd, sizeof(cmd));
    if (*h == ' ') {
        extract_letter_func(h + 1, obj, sizeof(obj));
    }

    printf("%s\n%s\n", cmd, obj);
}

char *extract_letter_func(char *h, char *dst, int len)
{
    char tmp[3];
    while (*h != ' ' && *h != '\0' && *h != '\n') {
        switch (_mbclen((BYTE *)h)) {
        case 1:
            tmp[0] = *(h++);
            tmp[1] = '\0';
            break;
        case 2:
            tmp[0] = *(h++);
            tmp[1] = *(h++);
            tmp[2] = '\0';
            break;
        default:
            break;
        }
        if (strlen(dst) + strlen(tmp) >= len) {
            break;
        }
        strcat(dst, tmp);
    }
    return h;
}

Comments

[ShoYama]

想定通り動作できました！
hはそのまま関数内で操作してそのまま代入したほうが便利ですね。
ありがとうございました。

Answer 2

これではいけないのでしょうか？

#define _CRT_SECURE_NO_WARNINGS
#include <stdio.h>

int main(void)
{
    char line[256], cmd[9], obj[17];

    printf("入力:");
    if (!fgets(line, sizeof(line), stdin)) return 1;
	if (sscanf(line, "%8s%16s", cmd, obj) != 2) return 2;
    printf("%s\n%s\n", cmd, obj);
}

実行例

入力:move GameSettingFile
move
GameSettingFile

Answer 3

extract_letter_func()でローカル変数letterのポインタを返していますが、スコープを抜けた後は無効です。
strncpy()でコピー元文字列長がコピー先バッファサイズ以上の場合、'\0'終端されません。