0

SDカードにあるファイルには1行ずつ値が入っており、その値を割り込み関数内にある val に代入してモータを回転させたいです。
ファイルには 1500, 2500, 500... と1行ずつ入っていて、val に代入されるとそれぞれ1.5s, 2.5s, 0.5s... おきにモーターを回転させます。

しかし、今の動きはファイルにある最後の数 1500 が val に入って 1.5s おきに回転してしまいます。

setup内に書かれているファイルの読み取りコードこれを1行ずつ val に値入れたらその都度回せるようにしたいです。
スケッチの改善点あれば教えていただきたいです。
よろしくお願いします。

#include<SD.h>
#include<SPI.h>

int APHASE=2;
int AENBL=3;
int BPHASE=6;
int BENBL=7;
volatile int count=0;           // LED用のカウンター
volatile int RotCounter=0;      // 現在の回転角(0,1,2,3)
volatile int TimeDivCounter=0;  // 割り込みのカウンター用
volatile int datacount=0;       // データ用のカウンター
volatile int val;

ISR(TIMER1_COMPA_vect){         // 1ms(0.001s)で読みだす
    // 18度回転の処理
    if(TimeDivCounter==val){    // ==のあとの数字はSDの値
        switch(RotCounter){
            case 0:
                digitalWrite(APHASE, LOW);
                digitalWrite(AENBL, HIGH);
                digitalWrite(BPHASE, LOW);
                digitalWrite(BENBL, HIGH);
                break;
            case 1:
                digitalWrite(APHASE, HIGH);
                digitalWrite(AENBL, HIGH);
                digitalWrite(BPHASE, LOW);
                digitalWrite(BENBL, HIGH);
                break;
            case 2:
                digitalWrite(APHASE, HIGH);
                digitalWrite(AENBL, HIGH);
                digitalWrite(BPHASE, HIGH);
                digitalWrite(BENBL, HIGH);
                break;
            case 3:
                digitalWrite(APHASE, LOW);
                digitalWrite(AENBL, HIGH);
                digitalWrite(BPHASE, HIGH);
                digitalWrite(BENBL, HIGH);
                break;
        }
        RotCounter++;

        if(RotCounter>=4){
            RotCounter=0;
        }
        TimeDivCounter=0;
    }else{
        TimeDivCounter++;
    }
}

void setup() {
    pinMode(APHASE,OUTPUT);
    pinMode(AENBL,OUTPUT);
    pinMode(BPHASE,OUTPUT);
    pinMode(BENBL,OUTPUT);
    digitalWrite(AENBL,HIGH);
    digitalWrite(BENBL,HIGH);

    Serial.begin(9600);
    while (!Serial) {
        ;
    }
    Serial.println("SDカードの初期化");

    if (!SD.begin(4)) {
        Serial.println("初期化に失敗");
        while(1);
    }
    Serial.println("初期化完了");

    File dataFile = SD.open("data.txt");
    if(dataFile){
        char buffer[65];

        while(dataFile.available()){
            int length=dataFile.available();

            if(length>64){
                length=64;
            }

            dataFile.read(buffer,length);
            buffer[length]='\0';
            Serial.write(buffer,length);
            val=atoi(buffer);
        }
        dataFile.close();
    }else{
        Serial.println(F("error opening data.txt"));
    }

    DDRB=0b00000100;
    TCCR1A = 0b00000010;
    TCCR1B = 0b00000011;
    TIMSK1 = 0b00000010;
    OCR1A = 249;
}

void loop(){
}
2
  • see also teratail.com/questions/291635 オイラも結局のところ全部作り直しになるに1票
    – 774RR
    Commented 2020年9月14日 23:23
  • 提示のコードではどういう不具合があるんでしょうか。 問題が見えません
    – y_waiwai
    Commented 2020年9月15日 1:26

2 件の回答 2

2

setup関数は最初に1度だけ実行されるのですから、「1行ずつvalに値入れたらその都度」という動作をsetup関数に書くのは不適切です(1度きりの動作に、その都度はありえない)。

繰り返し実行されるloop関数の中に、「valに値をいれて、その時間だけモーターを回す」というコードを移したら良いです。

2

組み込み系の基礎知識として

  • 初期化処理を1回だけして (提示 setup() が相当)
  • メインループをぐるぐるまわって終わることは無い loop()
  • 割り込み処理は最短で終わらせないと破綻する ISR()

提示プログラムはこの鉄則を守っていないので期待通りに動かなくて当然・ではどう直すかの方針だけ提示

  • 割り込み処理の中でファイル内容を使いたい
  • が、割り込み内でファイルを読むのは処理時間的に論外
  • ファイル内容を事前に読み込んでおき割り込み内ではメモリ上のデータを使う
  • 事前の処理のための関数が setup()
  • ついでに言うとメモリは有限

ということで、やるべきは

  • 秒数を保持するテーブルを作る (要素は有限個数になるだろう)
  • setup() で値を読んでテーブルに記憶しておく(読んだ要素数も必要:ファイル内容が多かったらどうするかを仕様策定する必要がある)
  • ISR() でそのテーブルの値を1つ1つ使う
  • どこまで使ったかのインジケータが別途必要だろう(読んだ要素数と別に)
  • 読み込んだ個数を使い切ったらどうするかを仕様策定する必要がある

のではないかなー、ということで改善点の提案とします。

この質問に回答するには、ログインする必要があります。