Помогитe с программой

[grek23rus]

Будьте так добры,помогите с программой,почему она не работает,заранее спасибо!

/* Задан целочисленный массив с N элементами. Разработать алгоритм определения количества пар соседних элементов массива,в который первый элемент больше второго,причём второй-нечётен. */

#include <iostream>

using namespace std;

int main()
{
   int n,s;
   cout << "input n: ";
   cin >> n;
   int a[n];

   cout << "input array:\n";
   for(int i=0; i<n; i++)
       cin >> a[i];

   for(int i=0; i<(n-1); i++)
   {
           if ((a[i-1]>>a[i+1])&&(a[i+1]%2==1))
           {
           s=+1;
           }
   }
       cout << s << " "; 
       
   system("PAUSE");
   return EXIT_SUCCESS;
}

[Stilet]

(a[i-1]>>a[i+1]

Это сдвиг? Может нужен один знак >?

int a[n];

Хм... ИМХО єто не каждому компилятору понравится...

почему она не работает

А в чем это проявляется?

[Mandrivnyk]

if ((a[i-1]>>a[i+1])&&(a[i+1]%2==1))

Одна ">" -- лишняя.
Да и по логике, сравнение должно быть

(a[i] > a[i+1])

сравнивать-то надо соседние элементы, а не через один...
Кроме того, счетчик s не инициализируется. Надо

s = 0;

Ну и совсем по-хорошему надо бы использовать динамический массив, если компилятор старый.

Update

Опять опоздал -)

[grek23rus]

Неверное решение показывает. Когда ввожу 5 4 3 2 1,должно 2 выводить,а он мне 1 показывает. В этом и вопрос.Почему?

[Mandrivnyk]

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

Неверное решение показывает. Когда ввожу 5 4 3 2 1,должно 2 выводить,а он мне 1 показывает. В этом и вопрос.Почему?

Внимательно почитай ответы выше.
Только что проверил код с указанными исправлениями -- все работает.

[An1ka]

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

он мне 1 показывает. В этом и вопрос.Почему?

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

s=+1;

Потому что это операция присвоения. Запись в s положительной единицы. Операция сложения и присвоения вот такая:

s+=1;

[Mandrivnyk]

s+=1;

Или вообще

s++;

[grek23rus]

Во блин!!! ТОЧНО! Заработало... Всем огромнейшее спасибо!!!!!!