2013-08-18 4 views
1

У меня есть следующий фрагмент кода.Странное поведение в C printf

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"); 

Для ввода и строки, например, «Я ENIL SIHTHSIREBBIG S», он должен просто напечатать «ЭТА ЛИНИЯ тарабарщина». Но он печатает только содержимое второго цикла, но, если я вставляю простой «\ n» в середине двух циклов, содержимое, printf внутри первого цикла работает. Что происходит здесь, здесь идет «рабочий» код (по крайней мере, я не хочу, что «\ п» в середине ')

#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; 
} 
+3

О, после того, как вы получите ответ (это хороший вопрос, в конце концов), можете ли вы попробовать это без 'gets'? Это ужасная функция, которая, как известно, является серьезной дырой в безопасности. И кроме того, [он был удален с C по C11] (http://en.wikipedia.org/wiki/Gets() #gets). –

+0

Я знаю, что это уродливо, но я не знаю другого способа прочитать полную строку с пустым пространством в строке С – aajjbb

+0

Ошибка, вероятно, находится где-то в другом месте, и она не попадает до тех пор, пока не будут выполнены другие части кода. Это не редкость видеть, что «printf» творит чудеса. –

ответ

2

В вашей программе есть 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() - это просто яд и никогда не должен использоваться).

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. Никакой другой подсказки, извините!

Редактировать: Oh! вы делаете C++, на самом деле ...