Области видимости

@Ника)) · Регистрация: 08.12.2010

Author24 — интернет-сервис помощи студентам

Если я запихиваю любые операции (в данном случае задание и заполнение массива);
То далее не видит, не то что не заполненный массив, а даже то что я его объявлял...
Подскажите как правильно описать все это ?

C++ (Qt)

#include <iostream>
#include <math.h>
#include <locale>
using namespace std;
 
class mas
{
private:
    int k1,k2,vremennai,per,predel,rez,ryz,*a;
    int b[];
 
public :
    mas( int n1); //êîíñòðóêòîð mas:mas
    void vvod(int n1);
    ~mas();
};
 
mas::mas(int n1){int *a=new int[n1];}
 
void mas::vvod(int n1){for(int j=0;j<n1;j++){a[j]=k1;k1++;} 
                        }
mas::~mas(){ delete a; delete b;}
 
int main(){
    setlocale(LC_ALL,"russian");
    int k1,k2;
    cout<<"(ó÷òèòå, åñëè ïðàâîå çíà÷åíèå ìåíüøå ëåâîãî , îíè ïîìåíÿþòñÿ ìåñòàìè)"<<endl;
    cout<< "Ââåäèòå äèàïîçîí ìèí. çíà÷åíèå, ìàêñ. çíà÷åíèå: "<< endl;
    cin >> k1 >> k2;
    int n=k2-k1+1;
    if(k2<k1){ int vremennai=k1; k1=k2;k2=vremennai; n=k2-k1+1;}
//int *a=new int[n]; // ЕСЛИ ТАК ТО ВСЕ НОРМ
    mas O(n);
    O.vvod(n);
    
    //ÂÎÒ ÅÑËÈ ÄÅËÀÒÜ ÝÒÎ ÂÑÅ ÒÓÒ , ÒÎ ÂÑÅ ÕÎÐÎØÎ
//  for(int j=0;j<n;j++){a[j]=k1;k1++;}  // çàïîëíåíèå ìàññèâà  // k1-çàìåíèòü êàêèì íèòü ïðåîáðàçîâàíèåì
        
    cout<<"Âñå  ÷èñëà â óêàçàííîì ìàññèâå : ";
    for(int t=0;t<n;t++){cout<<a[t]<<" ";}
    cout<<endl;
    
    
 
 
//  
 
    int per=0;
    for(int h=0;h<n;h++){ 
    int predel=a[h]-1;
    
 
//  cout<<predel<<"+"<<endl;
 
    // ïðîâåðêà íà ïðîñòîå ÷èñëî
        for(int g=2;g<predel;g++){
            if(a[h]%g==0){goto bu;
            }   
        }   
        a[per]=a[h]; per++;     
        bu:
        ;
    }
    cout<<"Âñå ïðîñòûå ÷èñëà â óêàçàííîì ìàññèâå : ";
    for(int h=0;h<per;h++){cout<<a[h]<<" ";}
    cout<<endl;
    
    int b[n];
    int rez=0;int ryz=0;
    //ïðîâåðêà íà áëèçíåöîâ
    for (int i=1;i<per;i++){
        if(a[i]-a[i-1]==2){b[rez]=a[i],a[ryz]=a[i-1];rez++;ryz++;
        }
    }
                                
    for(int h=0;h<ryz;h++){cout<<a[h]<<" áëèçíåö "<<b[h]<<endl;}
    cin.get();
    return 0;
}

@nmcf · 14.02.2015, 14:45

C++

mas::mas(int n1)
{
    a = new int[n1];
}
mas::~mas()
{
    delete[] a;
}

Что в десятой строке и где устанавливаются k1, k2?

@zss · 14.02.2015, 14:48

C++

#include <iostream>
#include <math.h>
#include <locale>
using namespace std;
 
class mas
{
private:
    //int k1,k2,vremennai,per,predel,rez,ryz; // временные переменные не храним
    int n; // размер массива a
    int *a;
 
public :
    mas( int n1);
    void vvod();
    void print();
    void CheckSimple(); 
    ~mas();
};
 
mas::mas(int n1)
{
    a=new int[n1];
    n=n1;
    for(int i=0;i<n;i++)
        a[i]=0;
}
 
void mas::vvod()
{
    for(int j=0;j<n;j++)
       a[j]=j;
}
void mas::print()
{
    for(int j=0;j<n;j++)
       cout<<a[j]<<' ';
    cout<<endl;
}
mas::~mas(){ delete[] a;}
 
bool IsSimple(int n)
{
    int i=2;
    for(;i*i<=n;i++)
        if(n%i==0)
            break;
    return (i*i>n);
}
 
void mas::CheckSimple()    // проверка на простые числа
{
    cout<<"Все простые числа в указанном массиве : ";
    for(int i=0;i<n;i++)
      if( IsSimple(a[i]) )
        cout<<a[i]<<' ';
    cout<<endl;
}
 
 
int main()
{
    setlocale(LC_ALL,"russian");
    int k1,k2;
    cout<< "Введите диапозон мин. значение, макс. значение: "<< endl;
    cin >> k1 >> k2;
    int n=abs(k2-k1)+1;
   
    mas O(n);
    O.vvod();
    cout<<"Все  числа в указанном массиве : ";
    O.print();
    O.CheckSimple();
    // остальное аналогично делаем в функциях - членах класса mas    
    system("pause");
    return 0;
}

@Ника)) · 14.02.2015, 14:59 **[ТС]**

Я просто не вижу разницы почему у вас работает в конструкторе , у меня нет....
//////
В 10-ой строке объявление второго массива, а задаются к1 и к2 в 29 строке

@zss · 14.02.2015, 15:05

Вы задаете переменные, которые не являются членами класса.
Переносите код в метод класса и повторно их не объявляйте.

@Ника)) · 14.02.2015, 16:08 **[ТС]**

Разрешите еще поинтересоваться какие именно это переменные, не циклов же?

Добавлено через 49 минут
Вот написал ,работает но у разрешите задать не сколько вопросов:
1) В конструкторе в различных функциях у каждой своя область видимости?
2) Почему пока не написать в 51 строке переобозначение одной и той же переменной он её не видит?
3) То что в main ни каких массивов , переменных используемых в конструкторе не видно , это нормально?

C++

#include <iostream>
#include <math.h>
#include <locale>
using namespace std;
 
class mas
{
private:
//  int k1,k2,vremennai,per,predel,rez,ryz,*a;
   int n;
    int *b;
    int *a;
 
public :
    mas( int n1,int k11); //конструктор mas:mas
    void proverka(int n1);
    void masivb(int n1);
    ~mas();
};
 
mas::mas(int n1,int k11){a=new int[n1];n=n1;
  for(int j=0;j<n1;j++){a[j]=k11;k11++;} 
    cout<<"Все  числа в указанном массиве : ";
        for(int t=0;t<n;t++){cout<<a[t]<<" ";}
                    cout<<endl;
                        }
 
void mas::proverka(int n1){
    int per=0;
    n=n1;
    for(int h=0;h<n;h++){ 
    int predel=a[h]-1;
    // проверка на простое число
        for(int g=2;g<predel;g++){
            if(a[h]%g==0){goto bu;
            }   
        }   
        a[per]=a[h];
        cout<<"число простое :"<<a[per]<<endl; // если сдесь убратть то перестаает работать проверка на близнецов
        per++; 
        bu:
        ;
                        }
        cout<<"Все простые числа в указанном массиве : ";
                for(int h=0;h<per;h++){cout<<a[h]<<" ";}
                                cout<<endl;                         
    }
                        
void mas::masivb(int n1){
    n=n1;
    int per=per;
    b=new int[n];
    int rez=0;int ryz=0;
    //проверка на близнецов
    for (int i=1;i<per;i++){
        if(a[i]-a[i-1]==2){b[rez]=a[i],a[ryz]=a[i-1];rez++;ryz++;
        }
    }                               
    for(int h=0;h<ryz;h++){cout<<a[h]<<" близнец "<<b[h]<<endl;}
}                   
                        
mas::~mas(){ delete a; delete b;}
 
int main(){
    setlocale(LC_ALL,"russian");
    int k1,k2;
    cout<<"(учтите, если правое значение меньше левого , они поменяются местами)"<<endl;
    cout<< "Введите диапозон мин. значение, макс. значение: "<< endl;
    cin >> k1 >> k2;
    int n=k2-k1+1; 
    if(k2<k1){ int vremennai=k1; k1=k2;k2=vremennai; n=k2-k1+1;}
    mas O(n,k1);
    O.proverka(n);
    O.masivb(n);
    cin.get();
    return 0;
}

@zss · 14.02.2015, 16:43

1. Конструктор срабатывает при создании КАЖДОЙ переменной типа mas.
Он должен инициализировать ВСЕ данные класса. В том числе и массив b
(а Вы этого не сделали).
2. Переменная per не является членом класса.
Поэтому ее надо объявлять и присваивать значение.
Надо

C++

1	int per=n;

Если ее надо передать из метода proverka, то сделайте так

C++

int mas::proverka(int n1)
{
...
   return per;
}

А в main будет так

C++

1 2	int per=O.proverka(n); O.masivb(n,per);

3. Это не только нормально, но и является целью создания класса -
скрыть его внутренние данные.

@Ника)) · 14.02.2015, 17:03 **[ТС]**

Вот на return у меня ругается:In member function 'void mas:

roverka(int)': На сколько я понял ранее, return не используется с void-ом

Благодарю за помощь !!

@zss · 14.02.2015, 20:31

А кому я написал ЭТО:

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

int mas:

roverka(int n1)

@Ника)) 43 / 43 / 20 Регистрация: 08.12.2010 Сообщений: 342
	14.02.2015, 14:59 [ТС]	4
	Я просто не вижу разницы почему у вас работает в конструкторе , у меня нет.... ////// В 10-ой строке объявление второго массива, а задаются к1 и к2 в 29 строке 0

@zss Модератор 13498 / 10752 / 6407 Регистрация: 18.12.2011 Сообщений: 28,692
	14.02.2015, 15:05	5
	Вы задаете переменные, которые не являются членами класса. Переносите код в метод класса и повторно их не объявляйте. 1

@Ника)) 43 / 43 / 20 Регистрация: 08.12.2010 Сообщений: 342
	14.02.2015, 17:03 [ТС]	8
	Вот на return у меня ругается:In member function 'void mas:roverka(int)': На сколько я понял ранее, return не используется с void-ом Благодарю за помощь !! 0

@zss Модератор 13498 / 10752 / 6407 Регистрация: 18.12.2011 Сообщений: 28,692
	14.02.2015, 20:31	9
	А кому я написал ЭТО: Сообщение от zss int mas:roverka(int n1) 1

Опции темы

Области видимости

Решение