Не получается перевернуть наоборот строку символьного массива.

[Zaza7]

#include <iostream>
#include <string>
using namespace std;
int main( )
{
    char S[10]={'O', 'p', 'p', 'a', 'P', 'o', 'p', 'a', '\0'};
    int i;
    char p;
    while (i<10/2)
    {
          p=S[i];
          S[i]=S[9-i];
          S[9-i]=p;
          i++;
          }
          i=0;
          for (i=0; S[i]!='\0'; i++)
          {
              printf("%c ", S[i]);
              }
              printf("%s ", S);
    return 0;
}

в результате получаю строку написанную 2 раза, а не перевернутую. Что не так делаю?

[pproger]

2Zaza7

переменную i используешь без инициализации.

так лучше:

#include <cstdio>
#include <cstring>

int main( )
{
	char str[] = "hello, world";

	for (int i = 0, l = strlen(str); i < l / 2; i++) {
		char c = str[i];
		str[i] = str[l - i - 1];
		str[l - i - 1] = c;
	}

	printf("%s\n", str);
	return 0;
}

[three_cats]

Да зачем?

Вот:

#include <conio.h>
#include <iostream>
#include <string>
using namespace std;
int main( )
{
    char S[10]={'O', 'p', 'p', 'a', 'P', 'o', 'p', 'a', '\0'};
    int i;
    char p;
          for (i=9; i>=0; i--)
              printf("%c ", S[i]);
              _getch();
    return 0;
}

Если, конечно, нужно просто его вывести.

А так, да, инициализация i плюс неверное условие в цикле вывода.

#include <iostream>
#include <string>
#include <conio.h>
using namespace std;
int main( )
{
    char S[10]={'O', 'p', 'p', 'a', 'P', 'o', 'p', 'a', '\0'};
    int i = 0;
    char p;
    while (i<10/2)
    {
          p=S[i];
          S[i]=S[9-i];
          S[9-i]=p;
          i++;
          }
          i=0;
          for (i=0; i<10; i++)
          {
              printf("%c ", S[i]);
              }
              printf("%s ", S);
          _getch();
    return 0;
}

[Zaza7]

Спасибо! Все получилось! Но не совмем понятно, почему это работает для числового массива, а для символьного нет.

[three_cats]

Здесь и есть символьный массив. И все работает.

[KuPuJIJI]

Вот код если я правильно понял тебя ток на языке С


#include <stdio.h>
#include <conio.h>
void main()
{
const int n=4;
int a[n];
puts("Vvedite element massiva:");
for(int i=0;i<n;i++)
scanf("%d",&a[i]);
for(int i=n-1;i>=0;i--)
printf("%5d",a[i]);
getch();
}
проверял вроде работает

[Zaza7]

я имела в виду, то, что я написала работает для числового, а для символьного не работает

[Murashov]

for(int i=0, j=SIZE-1;i<SIZE;i++, j--) mas2[i]=mas[j];
не важно какой массив символьный или целочисленный, такой вариант должен работать

если без использования второго массива, то я бы сделал бы счётчик от нуля до SIZE/2 и от SIZE до SIZE/2 и менял бы местами элементы через сумму (a=a+b ; b=a-b ; a=a-b ; ). ну или в Вашем случае через temp (temp=mas[i]; mas[i]=mas[j]; mas[j]=temp; ) (хотя символьный массив тоже можно через сумму: суммировать номер символа по таблице.. экономичнее выйдет по памяти. номер символа берём printf("%d", 'f');например)

[Nuklon]

Сообщение от Zaza7

я имела в виду, то, что я написала работает для числового, а для символьного не работает

Zaza7, не надо мучить червячка, надо использовать шаблончики.

#include <iostream>
#include <algorithm>

template<typename T> void t_reverse(T* a, T* b) {
    T t;
    for(b -= 1; a < b; ++a, --b) {
        t  = *a;
       *a = *b;
       *b = t;
   }
}


int main(){
   char s[] = "123";
   puts(s);  // исходная строка
   t_reverse(s, s + strlen(s));
   puts(s);  // вывод реверс-строки
	
   double arr[] = { 10.99, 34.66, 3.14567, 1.7 };
   int size  = sizeof(arr)/sizeof(arr[0]);

   // выведем исходный массив
   std::copy(arr, arr + size, std::ostream_iterator<double>(std::cout, "\t"));
   t_reverse(arr, arr + size);
   std::cout << std::endl;
   // выведем реверс массив
   std::copy(arr, arr + size, std::ostream_iterator<double>(std::cout, "\t"));
   return 0;
}

Данный шаблон любой тип реверснёт.