Arduinoでオブジェクト指向

Question

Arduinoでオブジェクト指向を勉強中なのですが、自作したプログラムがエラーになります。
仕様としては5に3を足して返してその値をシリアルで表示するという単純なものです。

何が間違っているのでしょうか？

該当のソースコード

//ヘッダーファイル
//sumple.h

#ifndef _sumple_h
#define _sumple_h
#include "Arduino.h"
 
class Csumple{
   
   public:
  void set(int data);
   int GET(void);
   
   private:
   int data;
   
    
 };
 
 #endif 

//ソースファイル
//sumple.cpp

#include "Arduino.h"
#include "sumple.h"

Csumple::Csumple(int pin)　// コンストラクタ（初期化処理）
{    
  pin=0;
   
 }
   Csumple::set(pin){  
   data = pin + 3; 
 }
 
 int Csumple::GET(){
   return data;
   
 }
 
 

 //メイン
 //main.ino

 //数値を渡してその値に3を足して返すプログラム
 
 #include"Arduino.h"
 #include"sumple.h"
 
 
   
   Csumple a;  //インスタンス生成


void setup() {
   Serial.begin(9600);
 
 }
 
 void loop() {
    int num=5;
    a.set(num);
   Serial.println(a.GET());
 } 

Comments

[y_waiwai]

エラーが出るならエラーメッセージを提示しましょう
また、Arduinoのなにを使ってるんでしょうか。

[episteme]

「エラーになります」を説明しなさい。

なにをしたら/どこで/どんなエラーになりますか?
エラーメッセージが出力されたなら、それをそのまま呈示しなさい。

[PHENIXa]

すみません。エラーメッセージを追記します

Answer 1

そもそもオブジェクト指向というのは、コンピュータ資源をじゃぶじゃぶ使って、ニンゲンがわかりやすいよーにコードの構成を組むことができる、という環境です。
なので、資源が極端に限られてるArduinoを使ってそれを勉強するというのはムリムリかと思います。

素直にPCを使って勉強しましょう。

Answer 2

回答をいろいろもらいましたが、まともな回答をして頂ける方(女又)が少ないので自己解決しました。

thkanaさん曰くteratailはエンジニア専用サイトらしいのでこんなことは知っていて当然のはずなので解決方法は皆さんは既にご存知と思うのでど素人の私の回答は差し控えさせていただきます。

Comments

[episteme]

まともな回答ならついてたよ。

「デフォルト(引数なし)以外のコンストラクタを使うなら、宣言と定義をセットで用意してください。」
件のケースではコンストラクタに意味がないから "コンストラクタを宣言/定義しない" が正解でしょね。

# エラーメッセージを示してくんなかったから断言はできんけど、きっとそう。

Answer 3

オブジェクト指向を勉強中

クラスはオブジェクト指向プログラミングをやりやすくするための仕掛けですが、クラスを使えばオブジェクト指向になるということは全くないです。今後オブジェクト指向プログラミングをするためにクラスを試している、ということかと思いますが、質問のプログラムがオブジェクト指向的だとかの勘違いはしないように。

一番どうでもいいところからいうと、プログラム言語的には全く問題にならないけれど、英語の問題としてsumpleはちょっと。
あと、習慣的にC/C++で全部大文字の関数名ってあまり使わないかなぁ...
不必要なファイルをインクルードするのもあまりよいことではないと思うし、
インクルードガードには#pragma onceというプリプロセッサ命令も使えるというのは覚えておいても損はないと思います。

とここまでは「間違い」ではないですが、以下本番。

最近のC++は実はシンボルにはUTF-8文字が使えることになっていたりするけれど、あくまでシンボルだけだし、しかもgccはいろいろとあるので、プログラム内では文字列/文字リテラルとコメント以外は
いわゆる半角文字だけで記述しましょう。UTF-8の全角空白(コード0xE3 80 80, 0343 200 200)が紛れ込んでいます。

クラスのメンバ関数の宣言と定義はシグニチャ(戻り値の型、関数名、引数リスト)が一致していなければいけません。これはCでのプロトタイプ宣言と関数定義の関係と同様。

デフォルト(引数なし)以外のコンストラクタを使うなら、宣言と定義をセットで用意してください。
逆に、デフォルトでないコンストラクタを宣言・定義した場合はデフォルトコンストラクタは暗黙的には使えなくなります。引数あり、引数なしのコンストラクタを両方使いたいのなら明示的に宣言定義しましょう。ただし、

Csumple::Csumple(int pin)// コンストラクタ（初期化処理）
{    
  pin=0;
}

は一体なにをしたいのですか? そこらへんを整理する方が先ですね。

Comments

[PHENIXa]

＞一体なにをしたいのですか?
＞コメントに書いてある通りです。そのためのコメントなのでは？

[episteme]

＞コメントに書いてある通りです。そのためのコメントなのでは？

初期化処理なのはコメントでわかった。
で、これのどこが初期化処理なの? なにを初期化してるの?

って意味で「一体なにをしたいのですか?」と問われているのよ。

[thkana]

では、「初期化処理」ってなに? と聞きましょうか。
初期化した結果、なにがどこに得られるのでしょう?

[PHENIXa]

それがよくわあからないのですが
不明なので、変数(正確には変数ではない？)pinを初期化

一応Arduinoではなくc++で同じようなプログラを記述すると初期化をしなくてもうまくいくのですが。

[episteme]

「一体なにをしたいのですか?」に答えられないのね。
説明できないコードは一行たりとも書いてはならんのだけど。

> 初期化をしなくてもうまくいくのですが。
なら初期化処理なんて要らんやん。なにするかもわかってないんだし。

[thkana]

> それがよくわあからないのですが
> 不明なので、変数(正確には変数ではない？)pinを初期化
う～ん。つまり、なんだかなんにもわからないけどそれっぽく文字を並べてみた、ということですか。

> 初期化をしなくてもうまくいくのですが。
うまくいく、とはなにがどうなることでしょう?

[PHENIXa]

仕様のとおりです。

＞なんだかなんにもわからないけどそれっぽく文字を並べてみた、ということですか。
はい、そうですけど。それがわからないから質問してるのですが。ここはそういう場なのでは？

[thkana]

"Hello worldを表示するプログラム"が、表示が"Hello world"だったとしても、その内部で1～100までの和を計算していたら、そのプログラムは「仕様通り」と言い切れるでしょうか?

では、もう少しミクロに「コンストラクタの関数仕様」を示してください。何を入力として、何を出力...コンストラクタなので戻り値はありませんが、要求される結果とするのでしょう。

> ここはそういう場なのでは？
googleでteratailを検索すると"ITエンジニア特化型Q&Aサイト"という副題付きで(まぁ広告ですけど)挙がってきます。エンジニアなら、「なんだかわからないけどそれっぽく文字を並べる」ことは「やってはいけない」でしょう。わからないことはかまいませんが、だからといってめちゃくちゃをやるのはダメです。