visual studioでstrtokをstrtok_sに変換する方法について

Question

現在苦しんで覚えるC言語でC++を学んでいるのですが、strtokをstrtok_sに変換する方法がよくわかりません.
整数をカンマで区切って入力した時、縦方向に入力した順に数値を出力したいです.入力値が10以下の時は入力してないところはNULL状態で出力したいです.一応太字の様に(上:記載されていたコード , 下:自分で変えたコード)しましたが&の後にどの変数を入れたらいいかわかりません.どうかアドバイスをお願いします.

#include<stdio.h>
#include<stdlib.h>
#include<string.h>

void main(){
int i, j, val[10];
char str[32], * ch;

fgets(str, sizeof(str), stdin);


**ch = strtok(str, ",\n");**
              ↓
    **ch = strtok_s(str,"\n,&ch);**



for (i = 0; i < 10; i++) {
	if (ch == NULL) {
		break;
	}
	else {
		val[i] = atoi(ch);


	}
	**ch = strtok(NULL, ",\n");**
                      ** ↓**

　　　　　　　　　ch = strtok_s(NULL, ",\n",&ch);
}
for (j = 0; j < i; j++) {
printf("%d\n", val[j]);
}
return;
}

Comments

[episteme]

> 苦しんで覚えるC言語でC++を学んでいるのですが

C++じゃない...

[iwa28]

すいません、ありがとうございます！

Answer 1

入力値が10以下の時は入力してないところはNULL状態で出力したいです.

入力の個数が 10以下なのではありませんか？
入力の値は 10より大きくてもいいんですよね。100,200,300 とか。

strtok_s の第3引数には、char *p; と宣言して、&p を与えてください。

C++ なら istringstream と getline が使えます。

#include <iostream> // cin, cout, endl
#include <string>   // getline, stoi
#include <sstream>  // istringstream
using namespace std;

int main()
{
	string line, str;
	getline(cin, line);
	istringstream iss(line);
	while (getline(iss, str, ',')) {
		int i = stoi(str);
		cout << i << endl;
	}
}

strtok と違って、11,,,,22,33 などはダメです。

Comments

[iwa28]

すいません、そーですね入力個数が10です。。ありがとうございました!

Answer 2

[回答ではありません] せっかくのC++なんだから、C++らしく(?)書いてみた:

#include <iostream>
#include <string>
#include <vector>

int main() {
  using namespace std;
  vector<int> val;

  string line;
  getline(cin, line); // 標準入力から一行読み込む

  const string delimiter = ",\n"; // 区切り文字
  string::size_type bpos = 0;
  string::size_type epos = 0;
  // bpos : eposを起点とし、delimiterに含まれない最初の文字の位置
  // epos : bposを起点とし、delimiterに含まれる最初の文字の位置
  // 切り出すtokenは 範囲 [bpos, epos) となる
  while ( (bpos = line.find_first_not_of(delimiter, epos)) != string::npos ) {
    std::string token = line.substr(bpos, (epos = line.find_first_of(delimiter, bpos))-bpos);
    val.emplace_back(std::stoi(token)); // tokenをintに変換しvalの末尾に挿入
  }

  // できたかな?
  for ( auto item : val ) {
    std::cout << item << std::endl;
  }
}

[別解] 正規表現使うと楽(オーバースペックかしら...)

#include <iostream>
#include <regex>
#include <string>
#include <vector>

int main() {
  using namespace std;

  vector<int> val;

  string line;
  getline(cin, line);
  regex re("[^,\n]+"); // ,でも改行でもない文字のひとつ以上の連続

  for ( sregex_iterator it(line.begin(), line.end(), re), end; it != end; ++it) {
    val.push_back(stoi(it->str()));
  }
  
  for ( auto item : val ) {
    cout << item << endl;
  }
}

Comments

[iwa28]

ありがとうございます!まだ理解が難しいですが写してみます!

Answer 3

strtok_sはstrtokに似ていますが、単純な置き換えでは動きません。
Microsftのリファレンスに簡単なサンプルがありますが、質問者さんのコードを直すとすると、

void main(){
    int i, j, val[10];
    char str[32], *ch;
    char *next_token = NULL; /* 追加 */

    fgets(str, sizeof(str), stdin);

    ch = strtok_s(str,"\n”,&next_token);

    for (i = 0; i < 10; i++) {
        if (ch == NULL) {
            break;
        }
        else {
            val[i] = atoi(ch);
        }
        ch = strtok_s(NULL, "\n",&next_token);
    }
    for (j = 0; j < i; j++) {
        printf("%d\n", val[j]);
    }
    return;
}

となります。

デリミタで分割されたトークンを受け取るポインタと、現在の分轄位置を保持するポインタは別の変数にしておかないと、正常に動作しません。
上記ソースはstrtokとstrtok_sを置き換えただけで意図した通りに動作するかまでは保証できませんので、デバッグは適宜行ってください。

Comments

[iwa28]

非常にわかりやすくて助かりました!ありがとうございました!