Форум программистов, компьютерный форум CyberForum.ru

Нужно сделать переменную статической - C++

Восстановить пароль Регистрация
 
ВиталикБВГ
11 / 10 / 3
Регистрация: 29.11.2009
Сообщений: 186
25.03.2010, 19:16     Нужно сделать переменную статической #1
Доброго времени суток всем. У меня вот какая проблема: создан класс множество. Перегружен оператор + чтоб добавлять элементы. Но вот при добавлении элемента оно добавляет элемент не в конец множества,а вместо последнего добавленного, т е получается множество является не статической переменной,а обновляется каждый раз при добавлении. со счётчиком ,который есть для того чтоб указывать в какое место вставлять новый элемент всё нормально - он растёт.
Помогите разобраться в чем сдесь дело. Заранее благодарю.
Вот текст проги(это из Шилдта):
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
#include <iostream> 
#include<conio.h>
using namespace std;
 
const int MaxSize = 100;
int g=1;
 
class Set {
    
    char members[100];
 
    int Set::find(char ch);
 
public:
    int len;
    Set(){len=0;}
 
    int getLength(){return len;}
 
    void showset();
    
    Set operator +(char ch);
    
};
 
 
int Set::find(char ch){
    int i;
 
    for(i=0;i<len;i++)
        if(members[i]==ch)return i;
 
    return -1;
}
 
void Set::showset(){
    cout<<"{";
    for(int i=0;i<len; i++)
    cout<<members[i]<< " ";
 
    cout<<"}\n";
}
 
 
 
 
Set Set::operator +(char ch){
    Set newset;
 
    if(len==MaxSize){
        cout<<"Множество полно\n";
        return *this;
    }
 
        if(find(ch)==-1){
            newset.members[newset.len]=ch;
            newset.len=newset.len+g;
            g++;
        }
        return newset;
    
}
 
    
 
    
 
 
    int main(){
        setlocale( LC_ALL,"Russian" );
    Set s1;
 
    s1=s1+ 'A';
    cout<<"s1 после добавления А  ";
    s1.showset();
    
 
    s1=s1+ 'B';
    cout<<"s1 после добавления А и В  ";
    s1.showset();
    
 
    s1=s1+ 'C';
    cout<<"s1 после добавления А и В и С ";
    s1.showset();
    
    s1=s1+'D';
    cout<<"s1 после добавления А и В ,C и D ";
    s1.showset();
 
    cout<<s1.len;
    
 
    cout<<"\n";
 
    getch();
    }
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
breate
 Аватар для breate
56 / 56 / 2
Регистрация: 23.10.2009
Сообщений: 250
26.03.2010, 03:18     Нужно сделать переменную статической #2
итак код отвратителен!!! просто ужасен!
поехали: рассмотрим Set Set::operator +(char ch)
первой строкой мы видим Set newset; что эквивалентно Set newset(); или конструктору по умолчанию Set(){len=0;} следовательно вопрос чему равен newset.len ?
ответ
правильно 0

опустившись ниже видим строку newset.members[newset.len]=ch;
вопрос в какую позицию мы пишем символ
ответ
в первую

newset.len=newset.len+g; что даёт эта строка?
ответ
она записывает новое значение длинны в newset.len и оно становится равно глобальной переменной и длинне которой якобы должно быть равно множество

g++; эта строка инкримирует глобальную переменную

делай выводы..
Выводы

1. программа не работает потому что каждый раз происходит запись в 1-ю позицию.
2. программа не может правильно работать потому что не происходит копирования из this в newset.
3. программа выводит мусор потому что существует только первый элемент а длинна увеличивается
4. код ужасен потому что не соблюдены правила ООП (сокрытие данных), используются глобальные переменные (это очень плохо), и вообще он не аккуратен и не несет смысловой нагрузки.

совет: читай
Роберт Седжвик фундаментальные алгоритмы 1-4 часть или
Структуры данных в C++. Уильям Форд, Уильям Топп
PointsEqual
ниначмуроФ
 Аватар для PointsEqual
832 / 516 / 33
Регистрация: 12.10.2009
Сообщений: 1,915
26.03.2010, 09:08     Нужно сделать переменную статической #3
статическая переменная:
C++
1
static int x=0;
ВиталикБВГ
11 / 10 / 3
Регистрация: 29.11.2009
Сообщений: 186
26.03.2010, 10:39  [ТС]     Нужно сделать переменную статической #4
Благодарю за ответы - теперь понятно в каком направлении двигаться :-)
breate
 Аватар для breate
56 / 56 / 2
Регистрация: 23.10.2009
Сообщений: 250
26.03.2010, 11:58     Нужно сделать переменную статической #5
Цитата Сообщение от breate Посмотреть сообщение
что эквивалентно Set newset(); или конструктору по умолчанию Set(){len=0;
наверно я не совсем корректно выразился - тут происходит создание нововй локальной переменной, а в момент создания новой переменной происходит вызов конструктора.
Yandex
Объявления
26.03.2010, 11:58     Нужно сделать переменную статической
Ответ Создать тему
Опции темы

Текущее время: 09:20. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru