教えてください！！

質問

テキストファイル aesop.txt には、イソップ寓話の一つが英語で書かれている。このファイルを読み込み、各単語を左右反転して標準出力に出力するプログラムを作成したい。つまり、以下の入力

The Dog and the Shadow

It happened that a Dog had got a piece of meat and was
carrying it home in his mouth to eat it in peace. Now on his way
:
が以下のようになればよい。

ehT goD dna eht wodahS

tI deneppah taht a goD dah tog a eceip fo taem dna saw
gniyrrac ti emoh ni sih htuom ot tae ti ni ecaep. woN no sih yaw
:
以下の指示に従って、プログラムを作成せよ。

コード
#include <stdio.h>
#include <stdlib.h>

#define STACKSIZE 256

char st[STACKSIZE];
int top=0;

int push(char d)
{

if (top < STACKSIZE)
{
st[top]=d;
top=top+1;
return(0);
}
if(top >= STACKSIZE)

return (-1);

}

char pop(void)
{
if(top>0){
top=top-1;
return(st[top]);
}
if(top==0) return('\0');
}

int main(void){
FILE *fp;
int c;
char moji;

fp = fopen("aesop.txt", "r");
if (fp == NULL){
printf("入力ファイルを開けません\n");
exit(1);

while ((c = fgetc(fp)) != EOF){

if(('A'>=c && c<='Z')||('a'>=c && c<='z')){
if(push(c)==0) {push(c);}
if(push(c)==-1){ exit(1);}
}
else {
while(top != 0){
moji=pop();
if(moji != '\0')
printf("%c",moji);
}

}

printf("%c",c);
}
}
fclose(fp);
return (0);
}

というコードを書いたのですが、実行できませんどこがまちがっているのでしょうか？？

回答No.4

echoes_x86

こんばんは．

質問のタイトルは「具体的に」書いた方が良いと思いますが…

他の方も指摘されていますが，データを二重にpushしている点が問題である，というものが回答です．
また，他にもいくつか修正すべき点があります．
(1)単語ごとにスタックにpushして，区切り文字が出てきたらpopすべき
(2)main関数で popの戻りとtopを同時に調べるのは無駄
(3)pop，pushの両関数でifを二つ書くのは無駄
(4)文字のチェックは ctype.h 内のマクロで行った方が良い

以下がこれらの方針に基づく実装例です．

#include <stdio.h>
#include <stdlib.h>
#include <ctype.h> /* isalnum()，isspace()マクロのため */

#define STACKSIZE 256

char st[STACKSIZE];
int top=0;

int push(char d)
{
if (top < STACKSIZE)
{
st[top]=d;
top++; /* この書き方の方が自然 */
return(0);
}
else { /* どんな場合でも値を返すように修正 */
return (-1);
}
}

char pop(void)
{
if(top>0){
top--; /* この書き方の方が自然 */
return(st[top]);
}
else{ /* どんな場合でも値を返すように修正 */
return('\0');
}
}

int main(void){
FILE *fp;
int c;
char moji;

if ((fp = fopen("aesop.txt", "r")) == NULL){
printf("入力ファイルを開けません\n");
exit(1);
}

do{
c = fgetc(fp);

if(isalnum(c)){ /* 文字か数字だったらスタックに積む */
if(push(c)==-1){/* スタックが溢れたら終了 */
exit(1);
}
}
/* 単語の区切り文字(スペース，改行とピリオド)を検出したら単語のpop開始 */
else if ((isspace(c))||(c=='.')){
while((moji = pop()) != '\0'){
printf("%c",moji); /* 文字列をpopして */
}
printf("%c",c); /* 区切り文字を出力 */
}
}while (c != EOF);

/* EOF直前の単語をpopする */
while((moji = pop()) != '\0'){
printf("%c",moji); /* 文字列をpopして */
}
fclose(fp);

return (0);
}

回答No.3

redfox63

どこがどのように希望に沿わない結果なのかを明記しましょう

if(('A'>=c && c<='Z')||('a'>=c && c<='z')){
　　if(push(c)==0) {
　　　　push(c);
　　}
　　if(push(c)==-1){
　　　　exit(1);
　　}
}
この部分は何を期待してるのでしょう
ファイルから読み込んだ文字をpush関数でバッファにためる処理のように思いますが
pushを何回も同じ文字に対して実行していますがこれは意図した動きでしょうか

>The Dog and the Shadow
の Tを読み込んで

　　if(push(c)==0) {
で1回目のpush
これが成功するので push(c)==0 条件が成立
if文の中の 2回目のpush(c); を実行
　　if(push(c)==-1) {
で3回目ののpushを実行
といった具合になるので配列stには
st[0] = "T", st[1] = "T", st[2] = "T"
と格納されてしまいます
これが意図した動きでしょうか?

最初にpush(c)の戻り値を取得してこれが失敗を表す-1かどうかを判断するのではないでしょうか

回答No.2

asuncion

＞というコードを書いたのですが、実行できませんどこがまちがっているのでしょうか？？

実行できません、とは、どういう状態ですか？
１）コンパイルできない。コンパイル時にエラーが出る。そのときのエラーメッセージは？
２）コンパイルはできて、実行ファイルは作れたが、その実行ファイルが正しく実行できない。エラーは出る？出ない？出る場合、メッセージの内容は？

どこかが間違っているとして、まず自分の頭でどこが正しくないかを考えましたか？
「動かない」→○→「人に聞く」
○印のところに「まずはとことん自分で考えてみる。それでもダメだったら」という過程があるはずですが、
それをすっ飛ばしていませんか？

回答No.1

Yanch

main関数にて、
「{」と「}」の数が一致していないようですよ。
よくコードを確認しましょう。

そして、ソース全体において、字下げの書き方が統一されていないみたいです。
読みにくいので、何かに統一すると良いです。