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

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

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

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

Q&A

解決済

2回答

1115閲覧

stack で文字列逆順表示

tokutok

総合スコア17

C++

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

0グッド

0クリップ

投稿2018/12/03 12:49

文字列を逆順で表示するプログラムです。
なぜ途中までしか表示されないか教えてください。
お願いします。

c++

1#include<iostream> 2#include<stack> 3using namespace std; 4int main() { 5 stack<char> s; 6 char c; 7 while (cin >> c) { 8 s.push(c); 9 } 10 for (size_t i = 0; i < s.size(); i++) { 11 cout << s.top(); 12 s.pop(); 13 } 14 cout << endl; 15 return 0; 16}

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

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

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

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

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

guest

回答2

0

stack の size() より empty() をつかったほうが良いと思います。
(stack の深さを知る必要はないはず。
空でなければ top を取り出すというロジックで十分なはず。
そして質問にあるような間違いが起こることを防げます。

c++

1#include<iostream> 2#include<stack> 3using namespace std; 4int main() { 5 stack<char> st; 6 char c; 7 while (cin >> c) { 8 st.push(c); 9 } 10 while (!st.empty()) { 11 cout << st.top(); 12 st.pop(); 13 } 14 cout << endl; 15 return 0; 16}

投稿2018/12/03 16:15

katoy

総合スコア22324

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

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

0

ベストアンサー

for (size_t i = 0; i < s.size(); i++) {

s.size()がどんどん小さくなっていきます
条件式の度にs.size()が評価されるので、およそ半分で打ち切られます。

対処としては

c++

1 for (size_t i = 0, size = s.size(); i < size; i++) {

c++

1 for (size_t i = s.size(); i > 0; i--) {

になります。

投稿2018/12/03 13:10

編集2018/12/03 13:16
asm

総合スコア15147

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問