1

我有以下一段代码。

    char str[MAXS];
    gets(str);
    N = strlen(str);

    for (i = N / 2 - 1; i >= 0; i--) {
        printf("%c", str[i]);
    }

    for (i = N - 1; i > N / 2 - 1; i--) {
        printf("%c", str[i]);
    }
    printf("\n");

对于输入字符串，例如“I ENIL SIHTHSIREBBIG S”，它应该简单地打印“THIS LINE IS GIBBERISH”。但它只打印第二个循环的内容，但是，如果我在两个循环的中间插入一个简单的“\n”，那么第一个循环内的内容，printf 就可以工作。这是怎么回事，这里是“工作”代码（至少，我不希望中间出现“\n”）

#include <iostream>
#include <string>
#include <sstream>
#include <vector>
#include <set>
#include <map>
#include <list>
#include <queue>
#include <stack>
#include <memory>
#include <iomanip>
#include <numeric>
#include <functional>
#include <new>
#include <algorithm>
#include <cmath>
#include <cstring>
#include <cstdlib>
#include <cstdio>
#include <climits>
#include <cctype>
#include <ctime>

#define REP(i, n) for(int (i) = 0; i < n; i++)
#define FOR(i, a, n) for(int (i) = a; i < n; i++)
#define FORR(i, a, n) for(int (i) = a; i <= n; i++)
#define for_each(q, s) for(typeof(s.begin()) q=s.begin(); q!=s.end(); q++)
#define sz(n) n.size()
#define pb(n) push_back(n)
#define all(n) n.begin(), n.end()

template<typename T> T gcd(T a, T b) {
    if(!b) return a;
    return gcd(b, a % b);
}
template<typename T> T lcm(T a, T b) {
    return a * b / gcd(a, b);
}

template<typename T> void chmin(T& a, T b) { a = (a > b) ? b : a; }
template<typename T> void chmax(T& a, T b) { a = (a < b) ? b : a; }
int in() { int x; scanf("%d", &x); return x; }

using namespace std;

typedef long long Int;
typedef unsigned uint;

const int MAXS = 107;

int N, T;
char str[MAXS];

int main(void) {
    scanf("%d ", &T);

    int i;

    for ( ; T--; ) {
        gets(str);
        N = strlen(str);

        for (i = N / 2 - 1; i >= 0; i--) {
            printf("%c", str[i]);
        }

        for (i = N - 1; i > N / 2 - 1; i--) {
            printf("%c", str[i]);
        }
        printf("\n");
    }
    return 0;
}
4

2 回答 2

2

你的程序中有 73 行代码，其中 50 行与程序的工作无关。请研究如何创建一个 SSCCE（简短、独立、正确的示例），这样人们就不必费力地阅读 3 倍的代码。

这是一个有效的 22 行程序：

#include <cstring>
#include <cstdio>

const int MAXS = 107;

int main(void)
{
    char str[MAXS];

    while (fgets(str, sizeof(str), stdin) != 0)
    {
        int N = strlen(str);
        if (str[N-1] == '\n')
            str[--N] = '\0';
        for (int i = N / 2 - 1; i >= 0; i--)
            printf("%c", str[i]);
        for (int i = N - 1; i > N / 2 - 1; i--)
            printf("%c", str[i]);
        printf("\n");
    }
    return 0;
}

它只是名义上的 C++；它使用<cstring>and<cstdio>标头，但如果您将它们更改为<string.h>and <stdio.h>，它将是 C 代码。请注意，它会检查缓冲区溢出（使用fgets()— 从不，永远使用gets()！）并测试是否读取了数据。它不需要计算输入的行数；它可以可靠地检测EOF。它删除了fgets()离开但gets()没有的尾随换行符。然后它向后打印字符串的前半部分，然后向后打印字符串的后半部分。我没有对两个打印循环的逻辑进行任何实质性更改。我确实删除了全局变量；您也应该避免使用它们，尽管有时全局变量是必要的（而gets()这只是毒药，永远不应该使用）。

于 2013-08-18T06:10:12.310 回答
1

您的代码似乎有效。至少这样：

#include <stdio.h>
#include <string.h>

#define MAXS 1024
int main() {
  char str[MAXS];
  int N;
  int i;

  gets(str);
  N = strlen(str);

  for (i = N / 2 - 1; i >= 0; i--) {
    printf("%c", str[i]);
  }

  for (i = N - 1; i > N / 2 - 1; i--) {
    printf("%c", str[i]);
  }
  printf("\n");
}

检查您的 MAXS 定义。没有其他线索，对不起！

编辑：哦！你在做C++，实际上......

于 2013-08-18T02:51:02.540 回答