0

こんにちは私はC++とOOPに頭を悩ませるために最善を尽くしています。すべてをメインループに入れたときに機能するコードをいくつか作成しました。私は今それを片付けて、自分の簡単なライブラリとクラスを作ろうとしています。

私のプロジェクトには、Arduinoボードに接続された8個のLEDセンサーアレイがあります。光センサーの状態を読み取ろうとしています。クラスをインスタンス化するときにクラスから呼び出したいcalibrateメソッドを持つQTRSensorsライブラリを使用しています。

現在、私は次のコードを持っています。

ヘッダ

#ifndef SEDIMENTLOGGER_H
#define SEDIMENTLOGGER_H

#include <QTRSensors.h>
#include <Arduino.h>


#define NUM_SENSORS 8


class sedimentSensor
{
 public:
  QTRSensorsRC sensor1;//(unsigned char [8], int&, int&, int&);
  QTRSensorsRC sensor2;//(unsigned char [8], int&, int&, int&);

  sedimentSensor(); //default constructor
  ~sedimentSensor();
  void  calibrateSensors();
  unsigned int *SensorValues1(){return sensorValues1;};
  unsigned int *SensorValues2(){return sensorValues2;};
  int SampleDelay(){return sampleDelay;};
  int EmitterPin1();
  unsigned int Position1(){return position1;};
  void Position1(unsigned int value){position1 = value;};
  unsigned int Position2(){return position2;};
  void Position2(unsigned int value){position2 = value;};

 private:
  // Map pins to LED numbers
  // first LED array
  int led1;
  int led2;
  int led3;
  int led4;
  int led5;
  int led6;
  int led7;
  int led8;
  int emitterPin1;
  //second Array
  int led9;
  int led10;
  int led11;
  int led12;
  int led13;
  int led14;
  int led15;
  int led16;
  int emitterPin2;

  int  timeout; // waits for 2.5 seconds for sensor outputs to go low
  long sampleDelay; // 1 second
  int numSensors;

  unsigned int sensorValues1[];
  unsigned int sensorValues2[];

  unsigned int position1;
  unsigned int position2;
};
#endif

クラス実装での私の試み

#include "sedimentLogger.h"
#include <Arduino.h>
#include <QTRSensors.h>

sedimentSensor::sedimentSensor() 
{
  //Constructor
  led1 = 24;
  led2 = 26;
  led3 = 28;
  led4 = 30;
  led5 = 32;
  led6 = 34;
  led7 = 36;
  led8 = 38;
  emitterPin1 = 22;
  //second Array
  led9 = 25;
  led10 = 27;
  led11 = 29;
  led12 = 31;
  led13 = 33;
  led14 = 35;
  led15 = 37;
  led16 = 39;
  emitterPin2 = 23;

  timeout = 2500; // waits for 2.5 seconds for sensor outputs to go low
  sampleDelay = 1000; // 1 second

  sensorValues1[NUM_SENSORS];  
  sensorValues2[NUM_SENSORS];
};

sedimentSensor::~sedimentSensor()
{ 
};

int sedimentSensor::EmitterPin1()
{return emitterPin1;};

void sedimentSensor::calibrateSensors()
{
  sensor1((unsigned char[]){led1,led2,led3,led4,led5,led6,led7,led8},numSensors,timeout,emitterPin1);
  sensor2((unsigned char[]) {led9,led10,led11,led12,led13,led14,led15,led16},numSensors,timeout,emitterPin2);


  unsigned int _delay = 500;
  delay(_delay);
  pinMode(13,OUTPUT);
  for(int i = 0; i<400; i++)
    {
      sensor1.calibrate();
      sensor2.calibrate();
    }
  digitalWrite(13,LOW);

  //TODO WRITE MAX AND MIN VALS TO FILE
  Serial.print("Array 1 Min :: ");
  for (int i = 0; i < NUM_SENSORS; i++)
  {
    Serial.print (sensor1.calibratedMinimumOn[i]);
    Serial.print(' ');
  }
  Serial.println();
  Serial.print("Array 2 Min :: ");
  for (int i = 0;i< NUM_SENSORS; i++)
  {
    Serial.print (sensor2.calibratedMinimumOn[i]);
    Serial.print(' ');
  }

  Serial.println();
  Serial.print("array 1 Max :: ");
  // print the calibration maximum values measured when emitters were on
  for (int i = 0; i < NUM_SENSORS; i++)
  {
    Serial.print(sensor1.calibratedMaximumOn[i]);
    Serial.print(' ');
  }
  Serial.println();
  Serial.print("array 2 Max :: ");
  // print the calibration maximum values measured when emitters were on
  for (int 
  i = 0; i < NUM_SENSORS; i++)
  {
    Serial.print(sensor2.calibratedMaximumOn[i]);
    Serial.print(' ');
  }

  Serial.println();
  Serial.println();
  delay(_delay); 
};

そして私の主な機能とループ

#include <QTRSensors.h>
#include <Arduino.h>

#include "sedimentLogger.h"
#include <Wire.h>
#include "/home/marrabld/Programming/Arduino/arduino-1.0/libraries/RTClib/RTClib.h"

sedimentSensor sedLog;//= sedimentSensor();

void setup()
{
  Serial.begin(9600);
  sedLog.calibrateSensors(); 
}

void loop()
{
  sedLog.Position1(sedLog.sensor1.readLine(sedLog.SensorValues1()));
  sedLog.Position2(sedLog.sensor2.readLine(sedLog.SensorValues2()));

  unsigned char i;
  for (i = 0; i < NUM_SENSORS; i++)
    {
      Serial.print(sedLog.SensorValues1()[i] * 10/10001);      
      Serial.print(' ');
    }
  Serial.println(sedLog.Position1());
  for (i=0;i < NUM_SENSORS; i++)
    {
      Serial.print(sedLog.SensorValues2()[i]*10/1001);
      Serial.print(' ');
    }
  Serial.print("  ");
  Serial.println(sedLog.Position2());
  delay(sedLog.SampleDelay());

  Serial.println(sedLog.SampleDelay());
  Serial.println(sedLog.EmitterPin1()); 
}

次のエラーが発生します

沈殿物Logger.cpp:メンバー関数'voiddesetSensor :: calibrateSensors()':sedateLogger.cpp:47:102:エラー:'(QTRSensorsRC)(unsigned char [8]、int&、int&、int&)'の呼び出しに一致しません沈殿物Logger.cpp:48:108:エラー:'(QTRSensorsRC)(unsigned char [8]、int&、int&、int&)'への呼び出しに一致しません</ p>

センサーのインスタンス化をdesetSensorコンストラクターに入れるなど、いくつかの異なるバリエーションを試しましたが、運が悪かったのです。

これを機能させるための助けとアドバイスをいただければ幸いです。ここでも概念が明らかに欠けているので、コーチングのアドバイスも同様に役立ちます。

4

2 に答える 2

3

sensor1とのコンストラクターを呼び出そうとしているようですがsensor2、コードのその時点で既に構築されています。init代わりにそれらの関数を使用してみてください:

sensor1.init(
    (unsigned char[]){led1,led2,led3,led4,led5,led6,led7,led8},
    numSensors,
    timeout,
    emitterPin1);

sensor2.init(
    (unsigned char[]){led9,led10,led11,led12,led13,led14,led15,led16},
    numSensors,
    timeout,
    emitterPin2);
于 2012-05-21T07:49:38.000 に答える
1

47 行目と 48 行目は次の 2 つだと思います。

sensor1((unsigned char[]){led1,led2,led3,led4,led5,led6,led7,led8},numSensors,timeout,emitterPin1);
sensor2((unsigned char[]) {led9,led10,led11,led12,led13,led14,led15,led16},numSensors,timeout,emitterPin2);

問題は、オブジェクト関数として呼び出そうとすることです。次のように、オブジェクトをコンストラクターで初期化する必要があります。

sedimentSensor::sedimentSensor() 
    : led1(24), led2(26), /* other variables needing initialization */
      sensor1(/*arguments*/), sensor2(/*arguments*/)
{
    /* .... */
}

上記を機能させるには、クラス メンバー変数の宣言の順序を変更する必要があります。メンバー変数が初期化される順序は、宣言されている順序と同じであるためです。sensor1つまり、 andの宣言は、それらを初期化するために使用される変数の後に配置する必要がsensor2あります

于 2012-05-21T08:00:40.367 に答える