文字列の逆順

Question

#include<iostream> 
#include<string.h> 
using namespace std; 
char * reverse (char *); 
int main() 
{ 
    char * a; 
    cout<<"enter string"<<endl; 
    gets(a); 
    cout<<a; 
    cout<<"the reverse of string is"<<reverse(a); 
    return 0; 
} 
char * reverse (char * b) 
{ 
    int i,j=strlen(b); 
    for(i=0;b[i]!='\0';i++) 
    { 
     char temp; 
     temp=b[j-i-1]; 
     b[j-i-1]=b[i]; 
     b[i]=temp; 
    } 
    return b; 
} 

このプログラムはコンパイル時にエラーが発生しません。しかし、実行時エラーが発生し、出力が得られません。原因を説明してください.C++でそれほどうまくいきませんので、私の質問が印ではないなら、私を許してください。

Answer 1

文字列にメモリを割り当てていません。 getsはあなたのためにメモリを割り当てないので、ランダムな場所を読み込んでいるだけです。結果は未定義です。

それとは別に、そこに多くの問題です：

• これがタグ付けされたC++が、それはCOUTとよりCのようなものだあなたが代わりにfgetsのgetsを使用している
• （stringを使用することは良いアイデアでしょう）。 getsは非常に危険であり、避けるべきです。

Answer 2

最初の文字列にスペースを割り当てる必要があります。そうでなければ、エーテルを指し示し、上書きするだけで神はどのスペースを知っているのですか。しかし、よりイディオム的なC++は標準的な文字列を使用することをお勧めしますか？単にchar *をstd :: stringに置き換えてください。

Answer 3

referenceの適切な使用方法はgets()です。初期化されていないchar*を渡すことはできません。あなたはstd::stringを使用する必要が

char a[256]; 
cout<<"enter string"<<endl; 
gets(a); 

Answer 4

：あなたはとのより良い運を持っているかもしれません。

#include <iostream> 
#include <string> 
#include <algorithm> 
#include <iterator> 
int main() 
{ 
    std::string a; 
    std::cout<<"enter string"<<endl; 
    std::cin >> a; 
    std::cout << "The reverse of a string is "; 
    std::copy(a.rbegin(), a.rend(), std::ostream_iterator<char>(cout)); 
    return 0; 
} 

標準は、使用しようとしているC-文字列のような非常識ではない文字列処理を提供し、それはまた、かなり自明逆反復を提供します。このモデルでは、独自のメモリを割り当てる必要はなく、独自の逆転を行う必要もありません。

Answer 5

これまでの回答に加えて、実際には文字列を2回反転しています。あなたは弦の半分しか歩かなくてはなりません。