>>667
＞strcpy(cWriteSpell, "\0");
二重に非効率。
＞if(nAsciiCode >= 48 && nAsciiCode<= 57){
環境依存。不適当な変数名の使用。
＞nSpellLen = atoi(cWriteSpell);
不適当な変数名の使用。
＞return -1;
環境依存。
＞strcat(cWriteSpell, "\0");
全く無意味。無駄。
＞(*(argv+1))
可読性を損なっている。
＞*(cReadSpell+counter)
表記の不統一。

総合評価：
ダーメダメじゃ〜ん。