Arduinoでのclassの書き方が分からない。

Question

前提・実現したいこと

いま教本を使って勉強して、自分でクラスを書いてみているのですが、エラーが出ています。
どこが間違っているのか検討が着かなくなってしまいました。
クラスの正しい書き方、メンバー変数とメンバー関数の正しい書き方を具体的に知りたくて質問しました。

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

classes:16:1: error: 'm' does not name a type

 m.x = 10;

 ^

classes:17:1: error: 'm' does not name a type

 m.y = 20;

 ^

classes:18:1: error: 'm' does not name a type

 m.go(10, 20);

 ^

c:\Users\a\Dropbox\!Arduino\class\classes.ino: In function 'void setup()':

classes:23:48: error: no matching function for call to 'HardwareSerial::println(const char [17], int&, int&)'

     Serial.println("x is %d, y is %d", m.x, m.y);

                                                ^

In file included from C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Stream.h:26:0,

                 from C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/HardwareSerial.h:29,

                 from C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Arduino.h:233,

                 from sketch\classes.ino.cpp:1:

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:77:12: note: candidate: size_t Print::println(const __FlashStringHelper*)

     size_t println(const __FlashStringHelper *);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:77:12: note:   candidate expects 1 argument, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:78:12: note: candidate: size_t Print::println(const String&)

     size_t println(const String &s);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:78:12: note:   candidate expects 1 argument, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:79:12: note: candidate: size_t Print::println(const char*)

     size_t println(const char[]);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:79:12: note:   candidate expects 1 argument, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:80:12: note: candidate: size_t Print::println(char)

     size_t println(char);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:80:12: note:   candidate expects 1 argument, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:81:12: note: candidate: size_t Print::println(unsigned char, int)

     size_t println(unsigned char, int = DEC);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:81:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:82:12: note: candidate: size_t Print::println(int, int)

     size_t println(int, int = DEC);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:82:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:83:12: note: candidate: size_t Print::println(unsigned int, int)

     size_t println(unsigned int, int = DEC);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:83:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:84:12: note: candidate: size_t Print::println(long int, int)

     size_t println(long, int = DEC);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:84:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:85:12: note: candidate: size_t Print::println(long unsigned int, int)

     size_t println(unsigned long, int = DEC);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:85:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:86:12: note: candidate: size_t Print::println(double, int)

     size_t println(double, int = 2);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:86:12: note:   candidate expects 2 arguments, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:87:12: note: candidate: size_t Print::println(const Printable&)

     size_t println(const Printable&);

            ^~~~~~~

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:87:12: note:   candidate expects 1 argument, 3 provided

C:\Program Files (x86)\Arduino\hardware\arduino\avr\cores\arduino/Print.h:88:12: note: candidate: size_t Print::println()

     size_t println(void);

            ^~~~~~~

該当のソースコード

class Move
{
public:
    int x;
    int y;
    void go(int w, int h);
};

void Move::go(int w, int h)
{
    x += w;
    y += h;
}

Move m;
m.x = 10;
m.y = 20;
m.go(10, 20);

void setup()
{
    Serial.begin(9600);
    Serial.println("x is %d, y is %d", m.x, m.y);  
}

void loop()
{
    
}

補足情報（FW/ツールのバージョンなど）

C/C++のクラスは生まれて初めて書きました。つっこみ満載かと思われますが、ご指摘頂けると助かります。

何卒よろしくお願いいたします。

Answer 1

逆(?)に、実行文は関数外には書けません。

また、Serial::printlnはprintfとは要求する引数が違います。

Comments

[episteme]

ホントだ。Arduino の Serial.println は "いわゆるprintf"とはちゃいますね。
https://www.arduino.cc/reference/en/language/functions/communication/serial/println

Answer 2

きっとこんなコトやりたいんだろなー...

class Move {
public:
    int x;
    int y;
    void go(int w, int h);
};

void Move::go(int w, int h) {
    x += w;
    y += h;
}

Move m;

void setup() {
    Serial.begin(9600);
    m.x = 10;
    m.y = 20;
    Serial.println("at start: x is %d, y is %d", m.x, m.y);  
}

void loop() {
    m.go(10, 20);
    Serial.println("x is %d, y is %d", m.x, m.y);  

}

Comments

[kalon]

まさにエスパーさん、助かります。

[kalon]

コンパイルしたところ、
```
classes:19:58: error: no matching function for call to 'HardwareSerial::println(const char [27], int&, int&)'

Serial.println("at start: x is %d, y is %d", m.x, m.y);
```
printlnでエラーが出てしまいますね。printlnって今まで数値と文字列を直接しか入れたことがないので、
教えて頂けると助かります。

[episteme]

https://www.arduino.cc/reference/en/language/functions/communication/serial/println
ココ↑に書いてある。

Answer 3

クラスは関数の中に書くもんじゃないです。

というか、「C++　入門」あたりでぐぐって一通り読んでみよう

Comments

[kalon]

稚拙な質問にご回答ありがとうございます。void setup{}自体が関数でしたね。
気付きませんでした。コードを修正してみて、色々試しましたがエラーが出て、やはりぐぐってもよく分かりませんでした。

[y_waiwai]

ArduinoであればArduinoIDEの標準関数（？）のソースがどこかに付属しています。
それらをみていけば参考になりますよ。

[y_waiwai]

ただ、Arduinoでクラスを使うというのは筋がよくありません。
というのは、対象のデバイス（ArduinoUNOとか）のメモリが絶望的に貧弱なため、限定的なクラスの使い方しかできません
付属のソースを読んでいっても、Cにちょっと毛の生えたような使い方しかしてないことがわかると思います

[kalon]

そうなんですね。アドバイスありがとうございます。