C++ ヘッダの実装部について

こんにちは

ヘッダの実装部についての質問です。
以下のようなコードがうまく実行されません。

a.h----------------------------------------------

class A
{
public:
int a(vector<int>& x) ;
}

class B :public A
{
int b(vector<int>& y);
}

a.cpp(実装部)-----------------------------------------------

int A::a(vector<int>& x)
{
int m = 0;
m = x.capacity() / 2;
if(m * 2 == x.capacity()){return 0; }
else{return 1; }
}

int B::b(vector<int>& y)
{
if(!(a(y))){cout << "奇数\n";} /この部分がうまく実行されません/
else{cout << "偶数\n";}
return 0;
}

b.cpp---------------------------------------------

A* aa=new A;
vector<int> a;
a->b(a);

実行しても関数aがちゃんと動きません。

行動規範の内容に同意します

回答2件

ベストアンサー

こんにちは。

maisumakunさんへのコメントのソースなら、微妙に追加すると動作するようです。
偶奇判定が逆なのは単純ミスと思いますよ。

C++
1#include <iostream>
2#include <vector>
3using namespace std;
4
5class A
6{
7public:
8    int a(vector<int>& x) ;
9};
10
11class B :public A
12{
13public:
14    int b(vector<int>& y);
15};
16
17int A::a(vector<int>& x)
18{
19    int m = 0;
20    m = x.capacity() / 2;
21    if(m * 2 == x.capacity()){return 0; }
22    else{return 1; }
23}
24
25int B::b(vector<int>& y)
26{
27    if(!(a(y))){cout << "奇数\n";}    /*この部分がうまく実行されません*/
28    else{cout << "偶数\n";}
29    return 0;
30}
31
32int main(void)
33{
34#if 0
35    A* aa=new A;
36    vector<int> a;
37    a->b(a);
38#else
39    B* b=new B;
40    vector<int> a;
41    b->b(a);
42#endif
43}

ところで、ソースは可能な時はコンパイルに通ったものを示すとスムーズですよ。
例えば、質問に記載されているソースは多少補った程度ではコンパイルできません。
aはvectorですので、当然b()関数はありません。
aa->b()のミスとしても、クラスAにb()関数はありません。
そのため、「実行しても関数aがちゃんと動きません。」と書かれても、実行できない筈なので回答できなくなります。

投稿2017/09/16 05:17

Chironian

総合スコア23272

退会済みユーザー

2017/09/16 08:31

こんにちは、回答ありがとうございます。 "コンパイルに通ったものを示すと…" かくしておおまかな代替のソースを載せていました。 "クラスAにb()関数はありません"とは、実際は"#if 0" は通らないで"else"以降が実行されるということでしょうか？

Chironian

2017/09/16 09:08

> "コンパイルに通ったものを示すと…" > かくしておおまかな代替のソースを載せていました。元のソースを開示する必要はありません。「おおまかな代替のソース」をコンパイルに通しておくとスムーズですよというお話です。ちょっとした手間を惜しまないことって意外に重要なのです。 > "クラスAにb()関数はありません"とは、読んだままです。もしかしてクラスAにb()関数を定義しているつもりでしょうか？もしそうなら質問に記載されているクラスAの定義を再確認すると良いと思います。 > 実際は"#if 0"は通らないで"else"以降が実行されるということでしょうか？ roueさんが質問に書かれたコードとmaisumakunさんへのコメントに書かれたコードを載せているだけです。

退会済みユーザー

2017/09/19 08:23

こんにちは返信遅れてしまい申し訳ありません。 main中の"#if","else","end"は必要なのでしょうか？また、main中の"#if"等はmain中でコンパイラが優先的に実行するものなのですか？

退会済みユーザー

2017/09/19 08:41

こんにちはどうにかちゃんと動作してくれました。単純ミスだったようです。(maisumakunさんQでミスがとれたようです。) "#if 0"へさせてもらった質問は気が向いたら返信ください。どうもありがとうございました。

Chironian

2017/09/19 08:42

必要ではありません。「roueさんが質問に書かれたコードとmaisumakunさんへのコメントに書かれたコードを載せているだけです。」これ以上の意味は無いです。 > また、main中の"#if"等はmain中でコンパイラが優先的に実行するものなのですか？ #if 0~#elseまでをコメントアウトしているのと同じです。プリプロセッサについて調べるとよいと思います。http://ftc.suki.net/tpb/saruc09.html

退会済みユーザー

2017/09/19 08:54

追記 static(?)かconstつければどうにかなると思っていました。値のアクセスの問題と考えていました。(abort()というものから)

退会済みユーザー

2017/09/19 09:02

"猿でもわかる…"そんなプログラミングのCDROM 兄が10年くらい前持っていた気がします…(関連は不明ですが) サイト、概観してみたのですが読みやすいです。さっそく見てみます。

行動規範の内容に同意します