文字列で取得したものを数値に変換したい

質問

現在ATmega88というマイコンでC言語プログラミングを行っています。

PCとこのマイコンをRS232Cケーブルでつないで通信を行い、PCからのコマンドでATmega88内のEEPROMメモリ内にアクセスして、データ変更などを行いたいと考えています。

たとえばPC側から”wr id 00 123456”という文字列を渡して、ATmegaがその文字列を受け取ったら、u8 rx_char[40];この中にいれて、

下のようなプログラムでコマンドを参照して

static reg_slave greg_slave[100] __attribute__((section(".eeprom")));

typedef struct _reg_slave_
{
u8 serial[3];
u8 slave_ctl_set;
}reg_slave;

このプログラムでEEPROM内を書き換えたいと思っています。
ただ、したのプログラムはうまく動作していないようなのです。どのように変更すれば良いかご教授お願いできませんでしょうか。よろしくお願いいたします。

char * p;
uint8_t n = 0;

p = strtok(rx_char," ");
while(*p)
{
strcpy(&words[n++], p);
p = strtok(NULL, " ");
}

u8 i=0;
u8 temp;

#if 1
if(strstr(words[0],cmds[0])!=NULL)//cmd wr
{

if(strstr(words[1],options[0])!=NULL)//option -id
{
for(i=0;i<3;i++)
my_eeprom_write_byte(&greg_slave[atoi(&words[2])].serial[i],atoi(&words[3+i]));
}
else if(strstr(words[1],options[1])!=NULL)//option -id
{
my_eeprom_write_byte(&greg_slave[atoi(&words[2])].slave_ctl_set,atoi(&words[3]));
}
else
{
putc(0x65);//e
putc(0x72);//r
putc(0x72);//r
}
}
else//cmd rd
{
if(strstr(words[1],options[0])!=NULL)//option -id
{
for(i=0;i<3;i++)
{
temp = my_eeprom_read_byte(&greg_slave[atoi(&words[2])].serial[i]);
puthex(temp);
}
}
else if(strstr(words[1],options[1])!=NULL)//option -id
{
//my_eeprom_read_byte(atoi(&words[2]),&slave_ctl_set);
temp = my_eeprom_read_byte(&greg_slave[atoi(&words[2])].slave_ctl_set);
puthex(temp);

}
else
{
putc(0x65);//e
putc(0x72);//r
putc(0x72);//r
//putc(0x0d);//cr

}
}
putc(0x0d);//cr
putc(0x0a);//lf
gClient.status.status=WAIT_RX;

回答No.1

Wr5 · ベストアンサー

該当のマイコンは知りませんが…(Arduino とは違うか…)

>while(*p)
>{
>　strcpy(&words[n++], p);
>　p = strtok(NULL, " ");
>}

strtok()の仕様を確認した方がよいでしょう。
元々の文字列内に" "が含まれなかった場合、NULLを返却するハズです。
となると、「while(*p)」はNULLポインタアクセスです。
# 最初のstrtok()で該当する文字が発見できなかった場合も当然NULLポインタアクセスです。

>my_eeprom_write_byte(&greg_slave[atoi(&words[2])].serial[i],atoi(&words[3+i]));

ループ中「atoi(&words[2])」に関しては値が変わらないのですから、ローカル変数なりに入れた方が
実行時の効率が上がるかと思われます。
マイコンの為、メモリは厳しいかと思われますが…

あとは…どこがどう「うまく動作していないよう」なのか不明ですので…なんとも。

回答No.4

asuncion

そこかしこに登場している
words
という変数をどのように定義しているかがわからないので、なんとも…。

回答No.3

D-Matsu

strtokは「デリミタが見つからない場合は（残りの）全文を返す、残りが空ならNULLを返す」なので第1引数の中身がNULLや空文字列でなければ戻り値がNULLでないのは保証されます。

問題なのはwhileの条件式の方で、この式では「参照先がヌル文字」の場合のみ抜けるということになるので空文字列を返すことがないstrtokでは絶対に正常終了できません。

要するに「ほぼ確実に最初のwhileで既にメモリ例外になっている」ということです。

回答No.2

titokani

sscanfが使えるなら、そちらのほうが楽ですよ。
char s1[40];
char s2[40];
int d;
int serial;
sscanf(rx_char,"%s %s %d %d",s1,s2,&d,&serial);
という感じで。