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

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

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

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

解決済

2回答

1154閲覧

出力が正確に動作しない

progokina8

総合スコア4

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2020/11/09 09:01

編集2020/11/09 10:35

前提・実現したいこと

現在、複数の文字列をファイルinput.txtからパイプラインにより入力し、文字列を辞書順にソートするコードを作成しています。
正しい入力・出力は以下の通りです。

【入力】(input.txt)
cat
bench
dog
apple
e-mail

【出力】
before:
cat
bench
dog
apple
e-mail

after:
apple
bench
cat
dog
e-mail(最後のみ改行なし)

自分のコードだとソートは成功するのですが、出力の形式(改行)がこの通りにいきません。
理由・修正方法を教えて頂けると幸いです。

該当のソースコード

C

1#include <stdio.h> 2#include <string.h> 3 4void strSort(int size, char str[30][31]){ 5 char tmp[31]; 6 for(int i=0; i<size-1; i++){ 7 for(int j=i+1; j<size; j++){ 8 if(strcmp(str[i], str[j]) > 0){ 9 strcpy(tmp, str[i]); 10 strcpy(str[i], str[j]); 11 strcpy(str[j], tmp); 12 } 13 } 14 } 15} 16 17void strlist(int size, char str[30][31]){ 18 for(int i=0; i<size; i++){ 19 printf("%s", str[i]); 20 if(i != size) printf("\n"); 21 } 22} 23 24int main(){ 25 int i, size; 26 char str[30][31]; 27 28 for(i=0; i<30; i++){ 29 if(scanf("%s", str[i]) == EOF){ 30 size = i+1; 31 break; 32 } 33 } 34 35 printf("before:\n"); 36 strlist(size, str); 37 printf("\n"); 38 39 strSort(size, str); 40 41 printf("after:\n"); 42 strlist(size, str); 43 44 return 0; 45} 46

###コンソール画面(VSCode)

Get-Content input.txt | .\a.exe before: cat bench dog apple e-mail after:  (←ここに改行は本来ない) apple bench cat dog e-mail

補足情報(FW/ツールのバージョンなど)

実行環境:VSCode
gccでコンパイル。

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

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

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

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

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

guest

回答2

0

sizeにEOF分を入れているからだと考えます。
sizeは、以下になるのでは。

c

1if (scanf("%s", str[i]) == EOF) { 2 size = i; // +1 は必要ないのでは 3 break;

投稿2020/11/09 09:33

akiruno-oneone

総合スコア815

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

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

progokina8

2020/11/09 10:26

回答有り難うございます。 残念ながら直りませんでした。 解決策がないかもう少し考えてみます。
guest

0

ベストアンサー

strlist の定義の中のループで、このループの継続条件は i<size です。 isize より小さい間しか繰り返さないのですから i==size であることはあり得ません。 つまり常に i!=size が真です。

ループの最後は改行を付けないことにするのなら if 文は if(i < size-1) printf("\n"); といったような書き方をする必要があります。

投稿2020/11/09 09:18

SaitoAtsushi

総合スコア5684

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

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

progokina8

2020/11/09 10:28

回答有り難うございます。 ご指摘の通りですね...最後の改行は直りました。 追加で申し訳ないのですが、コード内ではprintf("after:\n");と書いたので改行は1行のはずですが、何故かafter:の後に改行が2行分になってしまいます...。 理由が分かれば教えてください。
SaitoAtsushi

2020/11/09 12:46

それは akiruno-oneone さんの指摘のほうも関係してますね。 size がひとつ大きい数になっているために、何も格納してない場所も含めてソートしたらそれが先頭に来てしまったということだと思います。
progokina8

2020/11/09 13:20

なるほど、そういうことだったのですね...。 つまり、SaitoAtsushiさんのご指摘と、akiruno-oneoneさんのご指摘を両方直せば良いということでしょうか?
progokina8

2020/11/12 14:14

その通りでした。 期待していた通りの出力が得られました。 有り難うございました。ベストアンサーにさせて頂きます。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問