Question

我有以下代码。

    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”，它应该简单地打印“这条线是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;
}

Answer 1

您的程序中有73行代码，其中50行与程序的运行无关。请研究如何创建一个SSCCE（Short, Self-Contained, Correct Example），以便人们不必跋涉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>和<cstdio>标头，但如果您将其更改为<string.h>和<stdio.h>，那么它将是C代码。请注意，它检查缓冲区溢出（使用fgets() - 永远不要使用gets()！）并测试数据是否被读取。它没有考虑输入线的数量;它可以可靠地检测EOF。它会移除fgets()离开的尾随换行符，但gets()不会。然后它向后打印字符串的前半部分，然后向后打印字符串的后半部分。我没有对两个打印循环的逻辑做任何实质性的改变。我确实删除了全局变量;你也应该避免它们，虽然有时候全局变量是必要的（而gets()只是毒药而且永远不应该被使用）。

Answer 2

您的代码似乎有效。至少如此：

#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 ++，实际上......

C printf中的奇怪行为

2 个答案: