質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

2回答

463閲覧

(C++)入力した文字列によって結果が変わるプログラム

mol

総合スコア32

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

0クリップ

投稿2017/07/13 13:32

###前提・実現したいこと
こんばんは。文字列を入力して、それが"one"なら"正解"、"one"以外の場合"不正解"と表示するプログラムを作りたいのですが、ビルドしたところ、"初期化されていないローカル変数'bird'が使用されます。"というエラーメッセージが表示されてしまい、実行できませんでした。初期化とはどうすればいいのでしょうか?誰か教えてください・・・

###該当のソースコード

C++

1#include <iostream> 2 3using namespace std; 4 5class animal 6{ 7public: 8 char *cat; 9 void dog() 10 { 11 cout << cat; 12 } 13} obj ; 14 15int main() 16{ 17 char *bird; 18 19 obj.cat = "1を英語で言うと?\n"; 20 obj.dog(); 21 cin >> bird; //ここでエラーが出ました。 22 23 if (!strcmp(bird, "one")) 24 cout << "正解\n"; 25 26 else cout << "不正解\n"; 27 28 return 0; 29 30}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

こんにちは。

char *bird;はchar型変数へのポインタですね。
しかし、初期化していないため、どこか謎の空間を指し示しています。
そこへデータを読み込もうとすると確実におかしなことになります。

C++でしたら、#include <string>した上で、std::string bird;とすることをお勧めします。std::string型は文字列を取り扱うのに非常に便利ですよ。

if (!strcmp(bird, "one"))ではなくif (bird == "one")と書けば良い筈です。

投稿2017/07/13 13:40

編集2017/07/13 14:23
Chironian

総合スコア23272

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

_Victorique__

2017/07/13 13:52

揚げ足取りで申し訳ないですけど凄く強そうなライブラリをインクルードしてますよ!
mol

2017/07/13 13:53

教えていただいた通りにコードを書き直したら、無事できました!ありがとうございます!
mol

2017/07/13 14:00

強そうなライブラリとは・・・?
Chironian

2017/07/13 14:22

_Victorique__さん、ご指摘ありがとうです。 あああ、ほんとうですね。 molさん。これは#include <string>の間違いです。修正しておきます。 最近、Wordpressで<strong>とよく書くから、つい手が滑ってしまったようです。orz
guest

0

birdの実体が存在しないからです。

C++ならstd::stringを使った方がよいです。
特段使わない事情があるので無い限り。

投稿2017/07/13 13:38

編集2017/07/13 13:39
LouiS0616

総合スコア35658

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

mol

2017/07/13 13:57

やっぱりstd::stringを使ったほうがいいんですね。中途半端にCを勉強して、そこからC++の勉強を始めたのでついcharを使っていました。
LouiS0616

2017/07/13 14:00 編集

あるあるですね。 std::string、std::vector、std::unique_ptrの使い方は最低限覚えておいた方が良いです。 あとは必要に応じてlistとか他のスマートポインタですね。
mol

2017/07/13 14:13

わかりました!まだstd::stringしか使えませんが、他の2つも使いこなせるように勉強します。アドバイスありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問