texからhtmlへの変換プログラムについて

Question

前提・実現したいこと

c言語を用いてtexの数式モードであらわされたものをhtmlの数式に変換するというプログラムを作成しています
条件はギリシャ文字と上付き、下付き文字の変換ですなぜうまくいかないのでしょうか。
また、この機能を実現したいとき、どのようなコードを書けばよいのでしょうか

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

\alpha　x_1とコマンドラインで実行すると　&alpha　x<sub>1</sub>と出力してほしいが　alpha x_1という出力結果になる　そもそもこのコードは欠陥が多い気がします

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
void get_Greek_Mathsymbol(char *arg);
void get_Lgreek(char *arg);
void get_Subscript(char *arg);
void get_Superscript(char *arg);

int main(int argc, char *argv[]) {
	if (argc > 9) {
		printf("Illegal number of argument.\n");
		return (-1);
	}
	int k, a;

	for (k = 1; k < argc; k++) {

		get_Greek_Mathsymbol(argv[k]);
		get_Lgreek(argv[k]);
		get_Subscript(argv[k]);
		get_Superscript(argv[k]);

	}

	for (a = 1; a <= argc; a++) {
		printf("%s\n", argv[a]);
	}
	return 0;
}

void get_Greek_Mathsymbol(char *arg) {

	char greek[][10] = { "alpha", "beta", "gamma", "delta", "epsilon", "zeta",
			"eta", "theta", "iota", "kappa", "lambda", "mu", "nu", "xi", "pi",
			"rho", "sigma", "tau", "upsilon", "phi", "chi", "psi", "omega",
			"Gamma", "Delta", "Theta", "Lambda", "Xi", "Pi", "Sigma", "Upsilon",
			"Phi", "Psi", "Omega" "geq", "leq" };

	static int Cnt1 = 0;
	int cnt1 = 0;

	if (*arg == '\') {
		arg++;
		while (*arg != '\0') {   // これをすべての文字についてループ

			if (*arg != greek[Cnt1][cnt1]) {
				break;
			} else if ((*(arg + 1) == '\0') || (*(arg + 1) == '_')
					|| (*(arg + 1) == '^')) {
				*(arg - cnt1 - 1) = '&';
				if (*arg == 'q') {
					*arg = '\0';
				}
			}

			else {
				arg++, cnt1++;
				continue;
			}

		}

	}
	Cnt1++;
}

void get_Lgreek(char *arg) {

	char Greek[][10] = { "Alpha", "Beta", "Gamma", "Epsilon", "Zeta", "Eta",
			"Iota", "Kappa", "Mu", "Nu", "Rho", "Tau", "Chi", "omicron" };
	static int Cnt2 = 0;
	int i = 0;

	if ((*arg == Greek[Cnt2][0])
			&& ((*(arg + 1) == '\0') || (*(arg + 1) == '_')
					|| (*(arg + 1) == '^'))) {
		*arg = '&';
		arg++;
		while (Greek[Cnt2][i] != '\0') {
			*arg = Greek[Cnt2][i];
			arg++, i++;
		}
	}
	Cnt2++;
}

void get_Subscript(char *arg) {
	char rest1[10];
	int i, j, l, m;
	int cnt2 = 0;
	char subscript[][10] = { "<sub>", "</sub>" };

	if (*arg == '_') {
		arg++;
		for (i = 0; i <= 9; i++) {

			if ((*arg == '\0') || (*arg == '^')) {
				break;
			}
			rest1[i] = *arg;
			arg++, cnt2++;
		}

		for (j = 0; subscript[0][j] != '\0'; j++) {
			*(arg - cnt2 - 1 + j) = subscript[0][j];
		}
		for (l = 0; l < cnt2; l++) {
			*(arg - cnt2 + 4 + l) = rest1[l];
		}
		for (m = 0; subscript[1][m] != '\0'; m++) {
			*(arg + 4 + m) = subscript[1][m];
		}
	} else {
		arg++;
	}

}

void get_Superscript(char *arg) {
	char rest2[10];
	int i, j, l, m;
	int cnt3 = 0;
	char *superscript[] = { "<sup>", "</sup>" };
	if (*arg == '_') {
		arg++;
		for (i = 0; i <= 9; i++) {

			if (*arg == '\0') {
				break;
			}
			rest2[i] = *arg;
			arg++, cnt3++;
		}

		for (j = 0; superscript[0][j] != '\0'; j++) {
			*(arg - cnt3 - 1 + j) = superscript[0][j];
		}
		for (l = 0; l < cnt3; l++) {
			*(arg - cnt3 + 4 + l) = rest2[l];
		}
		for (m = 0; superscript[1][m] != '\0'; m++) {
			*(arg + 4 + m) = superscript[1][m];

		}
	} else {
		arg++;
	}
}

試したこと

関数を一つ一つ実行しても変換がうまくいかない

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

ここにより詳細な情報を記載してください。

Comments

[int32_t]

具体的にどんな入出力が必要なのか書いてください。例「〜というコマンドラインで起動すると、〜という出力になるはずが、〜という出力になってしまう」。あと、コードのインデントがデタラメでひどく読み辛いので、エディタの整形機能などを使ってください。

[tumayouji]

すみません、修正しました

Answer 1

alpha x_1という出力結果になる

コードを見るとそうはならない気がします。\alpha が入力にあると永久ループになりませんか?

            } else if ((*(arg + 1) == '\0') || (*(arg + 1) == '_')
                    || (*(arg + 1) == '^')) {
                *(arg - cnt1 - 1) = '&';
                if (*arg == 'q') {
                    *arg = '\0';
                }
            }

この条件に合うとargを変更しないのでwhileループから抜け出せません。

また、x_2 の方はget_Subscript()関数の引数argがxを指しているため、何も変更されません。

ところで、

int main(int argc, char *argv[]) {

main関数のargv引数はconstが付いておらず書き換え可能なのですが、文字数が増えるような変更はできません。

get_Subscript()とget_Superscript()は文字数を増やそうとしているため、文字列書き換えが起きるとメモリ破壊が起きます。文字数が増えるなら、別の十分な長さのバッファを用意する必要があります。

Comments

[tumayouji]

バッファはmain関数内に用意するということでしょうか。またループの件については理解をしました

[int32_t]

そうですね、自分しか使わないようなプログラムなら、main内に固定長のバッファを用意してget_*()に渡せば簡単ですね。

for (k = 1; k < argc; k++) {
char buffer[1024];
strlcpy(buffer, argv[k], 1024); // get_*()がどれも動かないときのため
get_Greek_Mathsymbol(argv[k], buffer, 1024);
get_Lgreek(argv[k], buffer, 1024);
get_Subscript(argv[k], buffer, 1024);
get_Superscript(argv[k], buffer, 1024);
printf("%s\n", buffer);
}

のように。

[tumayouji]

すみませんがループに関して、解決しません。どうしたらよいですか？

[int32_t]

引用したifブロック内の最後に arg++; を足せば良さそうです。

[tumayouji]

} else if ((*(arg + 1) == '\0') || (*(arg + 1) == '_')
|| (*(arg + 1) == '^')) {
*(arg - cnt1 - 1) = '&';
if (*arg == 'q') {
*arg = '\0';

}
arg++;
}
という感じでしょうか

[int32_t]

そうです。
ここで確認するより、自分で試して動かしたほうが早いですよ。

[tumayouji]

そうですね、試したらうまくいきそうな挙動をしました。