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

Question

前提・実現したいこと

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/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

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

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という変数を３つ宣言してますね
すべて、別々のカッコの中にあるために、エラーにもならずに同じ名前の変数が共存できてます

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

Comments

[zzzzzz]

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

[y_waiwai]

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

Answer 2

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

     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は存在しない

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

Comments

[zzzzzz]

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