1

このコードを機能させるために、今から 2 日間試しています。エラーに次ぐエラーです。

誰かが私が間違っていることを指摘できますか?

#include "stdafx.h"
#include <string>
#include <iostream>
using namespace std;

int main()
{
    int h = 0;
    for(int a = 100; a<1000; a++)
        for(int b = 100; b<1000; b++)
            int c = a * b;
// Error: "c" is undefined
            if ((c == reverse(c)) && (c > h))
                h = c;
    cout << "The answer is: " << h << endl;
}

int reverse (int x)
{
// Error: "'itoa' : function does not take 1 arguments"
    string s = string(itoa(x));
    reverse(s.begin(), s.end());
  return (x);
}

std::to_string を使用すると、さらにエラーが発生するだけです。

4

3 に答える 3

1

別の解決策を提案できますか? int<->string 変換を行う代わりに、次の方法で数値が回文であるかどうかをテストできます。

bool is_palindrome(int number, int base = 10)
{
    int rebmun = 0;
    for (int temp = number; temp != 0; temp /= base) {
        rebmun = (rebmun * base) + (temp % base);
    }
    return number == rebmun;
}

次に、テストは次のようになります。

if (is_palindrome(c) && (c > h))
于 2012-07-29T05:09:55.983 に答える
1

コンパイラがエラー メッセージで何かを説明した場合、それを信じるべきです。itoa実際、次のリンクでわかるように、複数の引数を取ります。

http://www.cplusplus.com/reference/clibrary/cstdlib/itoa/

編集：ああ、これは標準のC++スタイルのコードを使用して達成可能です（コメントの提案に従って少しコードを修正しました）：

int reverse(int x)
{
    std::stringstream ss;
    ss << x;

    std::string s = ss.str();
    std::reverse(s.begin(), s.end());

    ss.clear();
    ss.str(s.c_str());

    ss >> x;

    return x;
}

ここ。それが最もクリーンなソリューションかどうかはわかりませんが、私のコンパイラでは機能します。

編集:ここで1つの文字列ストリームのみを使用する方法を見つけました:文字列ストリームをクリアするには?

于 2012-07-29T04:38:15.580 に答える
0

最初の問題については、正しいインデントによって明確になる場合があります。

int h = 0;
for(int a = 100; a<1000; a++)
    for(int b = 100; b<1000; b++)
        int c = a * b;

if ((c == reverse(c)) && (c > h))
    h = c;

いくつかの追加のブラケットを使用:

int h = 0;
for(int a = 100; a<1000; a++)
{
    for(int b = 100; b<1000; b++)
    {
        int c = a * b;
        if ((c == reverse(c)) && (c > h))
            h = c;
    }
}

問題に関してitoaは、その署名は次のとおりです。

char *  itoa ( int value, char * str, int base );

itoa(x)そのため、単に書いて文字列を返すことを期待することはできません。

ただし、C++ でanintを aに変換するより良い方法があります。string

  • C++11 を使用している場合は、std::to_string
  • それ以外の場合は、astd::stringstreamがその役割を果たします。

そのようです：

#include <sstream>

int reverse (int x)
{
    std::stringstream ss;
    ss << x;
    string s(ss.str());
    reverse(s.begin(), s.end());
    return (x);
}

intただし、これは逆を返さないことに注意してください。

于 2012-07-29T04:36:20.570 に答える