質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Arduino

Arduinoは、AVRマイコン、単純なI/O(入出力)ポートを備えた基板、C言語を元としたArduinoのプログラム言語と、それを実装した統合開発環境から構成されたシステムです。

Q&A

解決済

2回答

696閲覧

arduino nanoでのコンパイルエラー

zzzzzz

総合スコア6

Arduino

Arduinoは、AVRマイコン、単純なI/O(入出力)ポートを備えた基板、C言語を元としたArduinoのプログラム言語と、それを実装した統合開発環境から構成されたシステムです。

0グッド

0クリップ

投稿2019/12/08 11:29

前提・実現したいこと

Arduino nanoでテキストメッセージ(nmea0183形式の風向風速データ※MWV)を別のテキスト(同左、VWR)に変換するスケッチを作成しています。

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

コンパイルエラーになってしまっているのですが原因、修正方法がわかりません。

エラーメッセージ

exit status 1 'vdir' was not declared in this scope ```ここに言語名を入力

#include <PString.h>
#include <nmea.h>

NMEA nmeaDecoder(ALL);

void setup(){
Serial.begin(4800);
}

// calculate checksum function (thanks to https://mechinations.wordpress.com)
byte checksum(char*str)
{
byte cs = 0;
for (unsigned int n=1; n < strlen(str) - 1; n++)
{
cs ^= str[n];
}
return cs;
}

void loop() {
if(Serial.available()){
if (nmeaDecoder.decode(Serial.read())) {
char*title=nmeaDecoder.term(0);
if (strcmp(title,"WIMWV")==0) { // only run the following code if the incoming sentence is MWV
Serial.println(nmeaDecoder.sentence()); // prints the original MWV

int wdir=atoi(nmeaDecoder.term(1)); // declares a integer from a string if (wdir<=180){ int vdir=wdir;} // and converts 360 degrees to 180 degrees else {int vdir=wdir-180;} char rol=nmeaDecoder.term(1); if (wdir<=180){ char rol="R";} else {char rol="L";} float wvel=atof(nmeaDecoder.term(3)); // declares a float from a string float vveln=wvel; float vvelm=wvel * 1852/3600; float vvelk=wvel * 1.852; // Time to assemble the sentence char vwrSentence [36]; // the VWR sentence can be up to 36 characters long byte cst; PString strt(vwrSentence, sizeof(vwrSentence)); strt.print("$XXVWR,"); strt.print(vdir); strt.print(","); strt.print(rol); strt.print(","); strt.print(vveln,1); strt.print(",N,"); strt.print(vvelm,1); strt.print(",M,"); strt.print(vvelk,1); strt.print(",K,"); cst = checksum(vwrSentence); if (cst < 0x10) strt.print('0'); // Arduino prints 0x007 as 7, 0x02B as 2B, so we add it now strt.print(cst, HEX); Serial.println(vwrSentence); } }

}
}

### 試したこと ここに問題に対して試したことを記載してください。 ### 補足情報(FW/ツールのバージョンなど) ここにより詳細な情報を記載してください。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

変数は宣言された中括弧{}の中だけしか有効ではありません

else {int vdir=wdir-180;}

このように宣言された vdir 変数は、このelse節の中でだけしか有効でありません
つまりは、このelseのあとの中括弧を抜けると未定義となります
ということで、このカッコを抜けたあとにある

strt.print(vdir);

では、この変数は無効となっているためにエラーが出ています

ついでに、

char rol=nmeaDecoder.term(1);

if (wdir<=180){ char rol="R";} else {char rol="L";}

ここでは、全く別個のrolという変数を3つ宣言してますね
すべて、別々のカッコの中にあるために、エラーにもならずに同じ名前の変数が共存できてます

# 本来、同一名の変数は宣言でないはず、ですね

投稿2019/12/08 13:19

y_waiwai

総合スコア87719

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

zzzzzz

2019/12/09 01:31

有難うございます。やはり基本を理解しないと駄目ですね。
y_waiwai

2019/12/09 01:50

というころで、これらの変数の宣言を、 void loop() { の直後でまとめて行うといい、ということですね
guest

0

{ }の中で宣言された変数の有効範囲は{ }の中に限られます。
プログラムの「骨組み」構造だけ取り出してみると、

if (strcmp(title,"WIMWV")==0) { // int vdir; //候補地 if (wdir<=180){ int vdir=wdir;} //ここでは宣言された{ }の外なのでvdirは有効でない else {int vdir=wdir-180;} //ここでは宣言された{ }の外なのでvdirは有効でない char rol=nmeaDecoder.term(1); //ついでに言えばこのrolと... if (wdir<=180){ char rol="R";} //このrolは別のもの。{ }内ではそこで宣言されたものが優先される else {char rol="L";} //このrolも別もの。 // if文の中の処理では結局rolは変更されていない strt.print(vdir); //というわけで、ここでは有効なvdirは存在しない

ということになります。「候補地」のあたりで宣言してみてはいかがでしょう。

投稿2019/12/08 12:06

thkana

総合スコア7610

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

zzzzzz

2019/12/09 01:32

有難うございます。ご指摘の点も踏まえて修正してみます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問