arduinoを使った出力最大値のリアルタイム処理について

Question

前提・実現したいこと

arduinoを使ってリアルタイムにアナログ値を取得しながら、一定データ数ごとの最大値を求めるプログラムを作成したい。（下記は１００データごとの最大値を求めるプログラム）

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

コンパイルは通るが、計算結果が表示されない。

該当のソースコード

int sensorpin[100];
const int sensorPin = A5;

void setup() {
Serial.begin(57600);
Serial.println(maxv(sensorpin,100));

}

void loop() {
int sensorValue = analogRead(sensorPin);
sensorpin[100]=sensorValue;
}

int maxv(int data[], int num){
int maxnum = data[0]; //仮の最大値
for(int i=1; i < num; i=i++){
if(maxnum < data[i]) maxnum=data[i];

}
return maxnum;
}

よろしくお願いします。

Comments

[episteme]

"１００データごとの最大値を求め"ている箇所が見当たらんのですが。

[Uka]

void setup() { Serial.begin(57600); Serial.println(maxv(sensorpin,100));　までいったら、一番下のブロックのmaxv関数に引数を渡すように設定しているつもりなのですが、、

[episteme]

setup() ってloop() に"先立って一回だけ"呼び出されるんじゃないん?

[Uka]

とりあえず動作確認するにあたって、最初の計算結果だけ確認できればいいかなと思い、setup()のなかに入れています。これだとmaxv関数の計算は行ってくれないのでしょうか？

[episteme]

setup() は loop() が動き出す"前"に呼ばれるんでしょ? ゴミ(意味のない値)が一回出力されるだけじゃない?

[Uka]

たしかにそのとおりですね、だとすると関数の計算はされずに素通りしてしまっているということですね。loop()のなかに入れて再度確認してみます。

Answer 1

void loop() {

int sensorValue = analogRead(sensorPin);
sensorpin[100]=sensorValue;
}

これはなにをやってるんでしょうか

Comments

[Uka]

読み込んだ１００個の値を要素数１００の配列にしています。

[y_waiwai]

これでは１００の配列になってません

[y_waiwai]

それでは、
int sensorValue = analogRead(sensorPin);
これはなにをしてますか

[Uka]

そもそも配列にする必要があるのか、配列に置き換える方法であればどのように記載すればよいのでしょうか？

[y_waiwai]

あなたはなにをしたいのでしょうか。
まずは、C言語の基礎から学習の必要があると思いますよ

[Uka]

int sensorValue = analogRead(sensorPin);
についてですが、これはA5ピンからの入力をsensorPinとして読み込み、sensorValue変数として宣言しています。
C言語は勉強しているところで、初歩的な質問で申し訳ありません。

[y_waiwai]

そして、
sensorpin[100]=sensorValue;
というのは、sensorpin配列の１０１番めにsensorValueを代入してます（配列の範囲外エラー）

Answer 2

void loop(){
int MAX_Value=maxv(100);
Serial.println(MAX_Value);
}
です。

Answer 3

配列を使用しないのであれば、maxvの中で
int sensorValue = analogRead(sensorPin);
を行えばよく、
int maxv(int num){
int maxnum = 0;
for(int i=1; i < num; i++){
int sensorValue = analogRead(sensorPin);
if(maxnum < sensorValue) maxnum=sensorValue;
}
return maxnum;
}
で、出来るし配列から最大値を探さないので早いです。

Comments

[Uka]

回答いただきありがとうございます。
配列を使わなくてもできるんですね、そうするとvoid loop() のなかも
int sensorValue = analogRead(sensorPin);
だけでよい気がしますね。
確認してみます。