не сравниваются элементы массива

[Faton 11]

Добрый вечер! Программа в общих чертах выглядит так:

int mas[200];//объявляем массив;

loop:
совершаем какое-то действие,значение которого возвращаем в х
i=0;
mas[i]=x;//
i=i++;
goto loop;

void Sravnenie(); /*чтобы сравнить все элементы массива между собой в   отдельной функции*/


void Sravnenie()
{
for(int k=0;k<1000;++k)
{
for(int j=0;j<1000;++j)
   {

	if (mas[j]==mas[k])
	{
		cout<<"bla-bla"<<endl;
	}
     }
  }
}

но никакого сравнения между элементами не происходит. Скажите пожалуйста, в чем принципиальная ошибка?

[Murashov]

не понял задания немного.. но например так можно решить:

const int SIZE=5;
int mas[SIZE]={8,8,8,8,8};
int f=1,i=0;
for(i=1;i<SIZE;i++)
    if(mas[i]==mas[i-1]) f++;
if(SIZE==f) printf("Элементы одинаковы");

о господи что это?

i=i++;

ни разу такого не видел)))

[Murashov]

аа кажется понял, Вы хотите каждый элемент сравнить с каждым?

const int SIZE=5;
int mas[SIZE]={8,8,8,8,8};
for(int i=0;i<SIZE;i++)
    for(int j=0;j<SIZE;j++)
        if((mas[i]==mas[j])&&(i!=j)) printf("mas[%d]=mas[%d]",i,j);

а так ничего не понял из кода что вообще делает эта часть? вернее должна делать?
int mas[200];//объявляем массив;

loop:
совершаем какое-то действие,значение которого возвращаем в х
i=0;
mas[i]=x;//
i=i++;
goto loop;

чему равен x? зачем прибавлять i и записывать его в i? и потом что за гоуту? зачем оно?) комментарий не соотвествует коду вообще... ничего мы тут в x не возвращаем, а зацикливаем loop, где каждый раз прибавляем и тут же обнуляем i. и что за метод инкремента?) почему i=i++?
и почему внешний и внутренние циклы идут до 100, а не до 200? у нас же размер массива 200

[Faton 11]

массив задается не вручную, а представляет из себя все значения некоторой переменной. И сравнить между собой нужно все элементы, а не два последних.

[Murashov]

я же выше написал прогу, какой вопрос-то?) почему два последних? где я сравниваю два последних?) ввод с клавы - в чём проблема?

[Faton 11]

int mas[200];//объявляем массив;

int i=0;
int x;

loop:
  mas[i]=x;
  i=i++; 
  x=x++;// например пусть такое действие
goto loop;

void Sravnenie(); /*чтобы сравнить все элементы массива между собой в   отдельной функции*/


void Sravnenie()
{
for(int k=0;k<200;++k)/*тут ,кстати, даже не сто было, а 1000, но это не  принципиально*/
{
for(int j=0;j<200;++j)
   {
	if (mas[j]==mas[k])
	{
		cout<<"bla-bla"<<endl;
	}
     }
  }
}

Само сравнение делается правильно, но на практике не сравнивает ничего. Почему?

[BDA]

Да уж, странный код.
Ваш loop зацикливается, скорее всего.
Да и принято передавать указатель на массив и размер массива в функцию, а не объявлять глобальные переменные.

Еще проверьте, что функция Sravnenie() вообще вызывается (сделайте вывод какой-либо надписи в начале функции).

[Faton 11]

Хотелось написать покороче и попонятнее, т.к. сама программа длинная, бестолковая и достала меня. Вот она:

//"Метод Серединных Квадратов"

#include <stdafx.h>
#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <conio.h>
#include <time.h>
#include <string>
#include <sstream>
#include <new>

using namespace std;
void Period();

string podstr(string x);//прототип

int sub_intArray[1000];
 int main ()
 {
	int x = 2152;
	int sqr;
	int i=0;
	int j=i+1;
	//int sub_int;
	string sub;
    sqr = x*x;
	

loop1:
    //число-->строка
    ostringstream buffer;
    buffer << sqr;
    string subc = buffer.str();

    //"вырезаем" подстроку 
    sub=podstr(subc);

    //строка-->число
    std::istringstream ss(sub);
    int sub_int;
    ss >> sub_int;

	sub_intArray[i]=sub_int;

		i=i++;
        sqr = sub_int*sub_int;
		if (sub_intArray[200])
		{
		}
		else
        goto loop1;

		for(int k=0;k<1000;++k)
{
   for(int j=0;j<1000;++j)
   {

	if (sub_intArray[j]==sub_intArray[k])
	{
		cout<<"virojdenie"<<endl;
		system ("pause");
	}
   }
  }	
 } 

string podstr(string x)
	 {
		string sub = x.substr(2,4);
        cout << "stroka " << x << endl;
        cout << "podstroka " << sub << endl;
	    return sub;
	 }

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

cout<<"virojdenie"<<endl;

. Почему?

[Murashov]

наверное это индус с промта пишет.. уже 2 готовые проги кинули и ответили на 10 вопросов вперёд, а он всё зациклился на своём коде, который без ста грамм не разберёшь

[BDA]

Murashov, Вы не правы.
Faton 11, программа выводит cout<<"virojdenie"<<endl;, потому что 0-й элемент массива равен 0-му элементу массива, как это ни странно

Update
Murashov, а может и правы
Faton 11, добавьте в проверку сравнение i и k, как в коде Murashovа.

Update2
Вывел для интереса массив на экран, там половина заполнена нулями