設定した実引数が反映されません

Question

下記コード(Fixed)に修正したところ正常に出力させるようにはできたのですが、なぜ下記コード(Error)では正常に出力されないのか分からず質問させて頂きました。

関数Print_Color(String color)で出力させる文字(色について)を設定しており、関数main()でPrint_Color(String color)を呼び出して実引数(BLUE)を代入しています。
しかし、確認したところmain()内で代入した実引数が、Print_Color(String color)で反映されておらず。REDが出力されてしまいます。

【Error】

String RED;
String BLUE;
String GREEN;

void Color(String color);

void main()
{
  Color(BLUE);
}

void Color(String color)
{
  if(color == RED)
  {
    printf("RED");
  }
  else if(color == BLUE)
  {
    printf("BLUE");
  }
  else if(color == GREEN)
  {
    printf("GREEN");
  }
}

//REDが出力されます

【Fixed】

void Color(String color);

void main()
{
  Color("BLUE");
}

void Color(String color)
{
  if(color == "RED")
  {
    printf("RED");
  }
  else if(color == "BLUE")
  {
    printf("BLUE");
  }
  else if(color == "GREEN")
  {
    printf("GREEN");
  }
}

// BLUEが出力されます

Comments

[itagagaki]

Stringは何ですか？typedefした型ならその定義を示してください。

[dodox86]

C言語でもC++でもJavaでもProcessing(≒Java)でも無さそうですが、一体何のプログラミング言語をお使いなのでしょうか。ソースコードのマークダウン記法でソースコードを提示する時は
```プログラミング言語の名前
コード...
```
のようにします。"Error"とか"Fixed"とかを入れるところではありません。

String RED;
String BLUE;
String GREEN;
はたぶん、グローバル変数かフィールド扱いになっているので、値がnullとかになってしまい、BLUEもREDも値自体が同じになり、if (RED == BLUE)が真になるのではないかと。

[karin10]

※Stringについての記載をせず申し訳ありませんでした。
Stringについて定義したのは自分ではなく、定義内容を確認したのですがclass String{省略}で定義されており、クラス内のコードの意味が理解できず勝手に省略してしまいました。申し訳ありません。
クラス内のコードが150行ほどありますので重要そうなところを抜粋して記載させていた頂きます。
検討違いでしたらすみません。
typedef void (String::*StringIfHelperType)() const;
void StringIfHelper() const {}

[jimbe]

C に"クラス"は無いと思います。 C++ ではないでしょうか。

[karin10]

今回の質問は、購入したロボット作成の学習キットで作成中に生じたことなのですが、マニュアルにはC言語と記載があります.
すでに定義されている内容はC++で定義されているのかもしれません。。

[episteme]

マニュアルが嘘ついてる。コンパイラは何使ってます?

[jimbe]

> ロボット作成の学習キット
商品名をお聞きしてもよろしいでしょうか？

[karin10]

プログラミング学習 iCarです。
https://icar.kernel-se.co.jp/

[dodox86]

ご提示のURLの説明を見ると
> Arduino【C言語初級者向け】
とあるので、Arduino部分を使っていませんか？　ArduinoにはStringクラスがあります。（ベースはC++です）まぁまぁ、C言語の知識で開発を進められる、という程度のことだと思います。

[karin10]

コンパイラはgccを用いています

[karin10]

自分はエミュレータも用いてC言語【上級者向け】のコースですすめています。
すでに定義されているところはC++も用いられているという感じなんですね！

[dodox86]

C言語【上級者向け】に関する説明では
> 「C言語開発には、別途オンチップデバッギングエミュレータ(ルネサスエレクトロニクス社製 E2 エミュレータLite)が必要です。

とあり、別途 https://icar.kernel-se.co.jp/download/ からダウンロードできる「C言語(e2 studio)開発環境」や
「C言語(e2 studio)入門編・サンプルプログラム・演習課題」の関数リファレンス - e2 studioを見るとベースはArduinoのソフトの開発方法と同等に見えます。setup()があり、loop()があります。 C++です。C++だと敷居が高そうに思われるのでC言語とマニュアルに書いているのかもしれませんが、そうだとするとかえって混乱を呼んでいますね。

[karin10]

細かいところまで確認して頂きありがとうございます。。
そうだったんですね。関数loop内のコードを勝手に関数mainに置き換えて記載してしまっていました。
勉強不足な点が多く、今回の質問で皆様に混乱を招いてしまい申し訳ありませんでした。

[jimbe]

ロボット車の操作のプログラミングから「実際の仕事でプログラミング」に至るという謳い文句が、結構な大風呂敷に思います ^^;

[karin10]

そうですね。。。
LEDやモータを制御するための関数まで用意されていたので、「実際の仕事でプログラミング」どころか、C言語やハードの勉強もあまり進みませんでした。こんな質問しておいてですが。

[jimbe]

一応 BA 終了ということにされましたが、今後のご質問があれば、iCar やエミュレータ、開発環境(Arduino?, e2 studio?) 等の情報も含めたうえで、コードは改変せずにご提示頂くのが宜しいかと思います。

それにしても「ロボット プログラミング」で検索したら結構出てるモンですね～。

[karin10]

ありがとうございます。
開発環境などについても記載してコードもそのまま記載するようにします。

結構多いですよね！

Answer 1

質問するときは、コードを省略しないでください。
String が何かが分からないと回答ができません。

#include <stdio.h>

typedef char *String;

これを質問のコードの先頭に追加した場合について説明します。

Errorのコードでは、RED、BLUE、GREEN の 3つのポインタ変数が
すべてグローバルなので、NULL に初期化されています。
Color(BLUE); の呼出しで、変数BLUE の値が NULL なので、
Color関数の引数 color に NULL が渡されます。
最初の if文で、変数RED の値が NULL なので、引数color の値と一致します。

Fiexedのコードでは、Color("BLUE"); の呼出しで、
Color関数の引数 color に文字列 "BLUE" の先頭アドレスが渡されます。
今使っているコンパイラが同じ文字列を 1つにまとめるようになっているので、
else if(color == "BLUE") の文字列 "BLUE" の先頭アドレスが
引数color の値と一致します。

同じ内容の文字列を 1つにまとめず別々の文字列とするコンパイラの場合、
一致しません。

Comments

[karin10]

Stringについて申し訳ありませんでした。
Stirngの定義は自分ではなく、内容が理解できず勝手に省略してしまいました。
Stringについてclass String{省略}で記載されているのですが、中身のコードが150行ほどでありますので一部記載させて頂きます。的外れな内容でしたら申し訳ありません。
typedef void (String::*StringIfHelperType)() const;
void StringIfHelper() const {}

[kazuma-s]

C に class はありません。C++ の間違いでしょう。質問を編集して、
class String の中のコンストラクタ String() の定義とメンバ変数の宣言を見せてください。
ヘッダファイルも全部見せてください。(#include の行)

[karin10]

購入した学習キットですでに定義されていた内容なのですが、ネットに乗せても問題ないでしょうか。。。

Answer 2

ErrorのほうのREDにはなにがはいってるんでしょうか

なにか入れてる形跡ありませんが