author | Claus Gittinger <cg@exept.de> |
Mon, 19 May 2003 20:40:02 +0200 | |
changeset 7311 | 6878116b0a27 |
parent 7268 | 4a2b7acd6913 |
child 7598 | 27f37bd56675 |
permissions | -rw-r--r-- |
1 | 1 |
" |
5 | 2 |
COPYRIGHT (c) 1991 by Claus Gittinger |
159 | 3 |
All Rights Reserved |
1 | 4 |
|
5 |
This software is furnished under a license and may be used |
|
6 |
only in accordance with the terms of that license and with the |
|
7 |
inclusion of the above copyright notice. This software may not |
|
8 |
be provided or otherwise made available to, or used by, any |
|
9 |
other person. No title to or ownership of the software is |
|
10 |
hereby transferred. |
|
11 |
" |
|
12 |
||
5471
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
13 |
"{ Package: 'stx:libbasic' }" |
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
14 |
|
1 | 15 |
Collection subclass:#Set |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
16 |
instanceVariableNames:'tally keyArray' |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
17 |
classVariableNames:'DeletedEntry' |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
18 |
poolDictionaries:'' |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
19 |
category:'Collections-Unordered' |
1 | 20 |
! |
21 |
||
2272 | 22 |
Object subclass:#EmptySlot |
2259 | 23 |
instanceVariableNames:'' |
4017
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
24 |
classVariableNames:'TheOneAndOnlyInstance' |
2259 | 25 |
poolDictionaries:'' |
26 |
privateIn:Set |
|
27 |
! |
|
28 |
||
29 |
!Set class methodsFor:'documentation'! |
|
88 | 30 |
|
31 |
copyright |
|
32 |
" |
|
33 |
COPYRIGHT (c) 1991 by Claus Gittinger |
|
159 | 34 |
All Rights Reserved |
1 | 35 |
|
88 | 36 |
This software is furnished under a license and may be used |
37 |
only in accordance with the terms of that license and with the |
|
38 |
inclusion of the above copyright notice. This software may not |
|
39 |
be provided or otherwise made available to, or used by, any |
|
40 |
other person. No title to or ownership of the software is |
|
41 |
hereby transferred. |
|
42 |
" |
|
43 |
! |
|
1 | 44 |
|
88 | 45 |
documentation |
46 |
" |
|
47 |
a Set is a collection where each element occurs at most once. |
|
95 | 48 |
The inclusion test is done using = for comparison; |
49 |
see IdentitySet for sets using identity compare. |
|
6191 | 50 |
Keep in mind, that a regular Set therefore treats 3.0 and 3 as equal |
51 |
and therefore: |
|
6994 | 52 |
(Set with:3.0) includes:3 |
6191 | 53 |
will return true (since 3.0 and 3 are equal). |
54 |
In contrast, an IdentitySet will return false, because 3.0 and 3 are not |
|
55 |
identical. |
|
56 |
||
95 | 57 |
Sets use hashing for fast access, this access is considerably faster, |
58 |
if a good hash-number is returned by the elements. |
|
362 | 59 |
|
60 |
Notice that the default hash (Object>>hash) is not perfect; due to |
|
61 |
the implementation of hash-keys in ST/X, increased hash collisions |
|
62 |
are to be expected for large sets (say: > 20000 element). |
|
63 |
If your objects are heavyly used in sets or dictionaries, and you need |
|
64 |
big collections, your instances may provide a better hash values. |
|
95 | 65 |
|
345 | 66 |
Performance hints: |
657 | 67 |
If only symbols or smallIntegers are used as keys, |
362 | 68 |
use an instance of IdentitySet for slightly better performance, |
69 |
since both hashing and comparison is faster. |
|
345 | 70 |
|
71 |
If you have a rough idea how big the set is going to grow, |
|
72 |
create it using #new: instead of #new. Even if the size given is a |
|
73 |
poor guess (say half of the real size), there is some 20-30% performance |
|
74 |
win to expect, since many resizing operations of the set are avoided. |
|
75 |
||
95 | 76 |
Examples: |
77 |
||
6994 | 78 |
|s| |
79 |
s := Set new. |
|
80 |
s add:'hello'. |
|
81 |
s add:'world'. |
|
82 |
s add:#foo. |
|
83 |
s add:1.2345678. |
|
84 |
s add:'hello'. |
|
95 | 85 |
|
6994 | 86 |
s printCR. |
87 |
's size -> ' print. s size printCR. |
|
88 |
'(s includes:''hello'') -> ' print. (s includes:'hello') printCR. |
|
89 |
'(s includes:#foo) -> ' print. (s includes:#foo) printCR. |
|
90 |
'(s includes:''foo'') -> ' print. (s includes:'foo') printCR. |
|
91 |
'(s includes:#bar) -> ' print. (s includes:#bar) printCR. |
|
1290 | 92 |
|
93 |
[author:] |
|
6994 | 94 |
Claus Gittinger |
88 | 95 |
" |
96 |
! ! |
|
1 | 97 |
|
2259 | 98 |
!Set class methodsFor:'initialization'! |
41 | 99 |
|
100 |
initialize |
|
101 |
"initialize the Set class" |
|
102 |
||
103 |
DeletedEntry isNil ifTrue:[ |
|
6994 | 104 |
DeletedEntry := EmptySlot new |
41 | 105 |
]. |
106 |
||
107 |
"Set initialize" |
|
2259 | 108 |
|
109 |
"Modified: 24.1.1997 / 21:09:00 / cg" |
|
41 | 110 |
! ! |
111 |
||
2259 | 112 |
!Set class methodsFor:'instance creation'! |
1 | 113 |
|
6083 | 114 |
decodeFromLiteralArray:anArray |
115 |
"create & return a new instance from information encoded in anArray." |
|
116 |
||
6994 | 117 |
|col| |
6083 | 118 |
|
119 |
col := self new:anArray size - 1. |
|
120 |
anArray from:2 do:[:el | col add:el decodeAsLiteralArray]. |
|
121 |
^ col |
|
122 |
||
123 |
" |
|
124 |
(Set with:1234 |
|
6994 | 125 |
with:(1 @ 2) |
126 |
with:'hello' |
|
6083 | 127 |
) literalArrayEncoding decodeAsLiteralArray |
128 |
" |
|
129 |
! |
|
130 |
||
1 | 131 |
new |
132 |
"return a new empty Set" |
|
133 |
||
134 |
^ self new:7 |
|
135 |
! |
|
136 |
||
137 |
new:anInteger |
|
138 |
"return a new empty Set with space for anInteger elements" |
|
139 |
||
159 | 140 |
" |
141 |
make it somewhat bigger; hashing works better if fill grade is |
|
142 |
below 10% (make it 75% here ..) |
|
143 |
" |
|
144 |
^ self basicNew setTally:(anInteger * 4 // 3) |
|
145 |
! ! |
|
146 |
||
2259 | 147 |
!Set class methodsFor:'queries'! |
252 | 148 |
|
149 |
goodSizeFrom:arg |
|
150 |
"return a good array size for the given argument. |
|
151 |
Returns the next prime after arg, since prime sizes are good for hashing." |
|
152 |
||
302 | 153 |
|n| |
154 |
||
2461
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
155 |
"/ arg <= 11 ifTrue:[^ 11]. |
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
156 |
"/ n := arg * 3 // 2. |
252 | 157 |
|
2461
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
158 |
arg <= 7 ifTrue:[^ 7]. |
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
159 |
n := arg. |
302 | 160 |
|
252 | 161 |
" |
162 |
mhmh - this returns good numbers for collections with up-to about |
|
163 |
500k elements; if you have bigger ones, add some more primes here ... |
|
164 |
" |
|
302 | 165 |
n <= 524288 ifTrue:[ |
6994 | 166 |
"2 4 8 16 32 64 128 256 512 1024 2048 4096 8192 16384 32768 65536 131072 262144 524288" |
167 |
^ #(7 7 11 17 37 67 131 257 521 1031 2053 4099 8209 16411 32771 65537 131101 262147 524309) at:(n highBit) |
|
252 | 168 |
]. |
169 |
" |
|
170 |
make it odd - at least |
|
171 |
" |
|
302 | 172 |
^ n bitOr:1 |
2461
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
173 |
|
2d65e1845502
minimum goodSize reduced to 7.
Claus Gittinger <cg@exept.de>
parents:
2459
diff
changeset
|
174 |
"Modified: 19.3.1997 / 16:07:59 / cg" |
252 | 175 |
! ! |
176 |
||
3048 | 177 |
!Set class methodsFor:'utilities'! |
178 |
||
179 |
rehashAllSubInstances |
|
180 |
"rehash all sets & dictionaries. |
|
181 |
Useful utility if hash/identityHash method was changed |
|
182 |
of some object which is known to be kept in a set" |
|
183 |
||
184 |
self allSubInstances do:[:s | s rehash] |
|
185 |
||
186 |
" |
|
187 |
Set rehashAllSubInstances |
|
188 |
" |
|
189 |
||
190 |
"Created: / 24.10.1997 / 23:13:44 / cg" |
|
191 |
! ! |
|
192 |
||
7311
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
193 |
|
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
194 |
|
7268 | 195 |
!Set methodsFor:'Compatibility-ST80'! |
6991
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
196 |
|
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
197 |
initialIndexFor:hashKey boundedBy:length |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
198 |
"for ST-80 compatibility only; it is (currently) not used in this |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
199 |
implementation of sets. Therefore, in ST/X it does not make sense |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
200 |
to redefine it. (which may be a bad design decision, but slightly |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
201 |
improves performance, by avoiding an extra message send ...)" |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
202 |
|
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
203 |
|index "{ Class:SmallInteger }"| |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
204 |
|
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
205 |
index := hashKey bitAnd:16r3FFFFFFF. |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
206 |
index < 16r1FFFFFFF ifTrue:[ |
6994 | 207 |
index := index * 2 |
6991
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
208 |
]. |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
209 |
index := index \\ length + 1. |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
210 |
^ index. |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
211 |
|
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
212 |
"Modified: 1.3.1997 / 01:01:13 / cg" |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
213 |
! ! |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
214 |
|
7311
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
215 |
!Set methodsFor:'Compatibility-Squeak'! |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
216 |
|
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
217 |
copyWithout:anElement |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
218 |
^ self select:[:each | each ~= anElement] |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
219 |
|
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
220 |
" |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
221 |
#(1 2 3 4 5 6 7) asSet copyWithout:5 |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
222 |
" |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
223 |
! |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
224 |
|
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
225 |
copyWithoutAll:aCollection |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
226 |
^ self select:[:each | (aCollection includes:each)] |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
227 |
|
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
228 |
" |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
229 |
#(1 2 3 4 5 6 7) asSet copyWithoutAll:#(3 5 7 9) |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
230 |
" |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
231 |
! ! |
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
232 |
|
609 | 233 |
!Set methodsFor:'accessing'! |
234 |
||
235 |
at:index |
|
236 |
"report an error: at: is not allowed for Sets" |
|
237 |
||
238 |
^ self errorNotKeyed |
|
239 |
! |
|
240 |
||
241 |
at:index put:anObject |
|
242 |
"report an error: at:put: is not allowed for Sets" |
|
243 |
||
244 |
^ self errorNotKeyed |
|
2469
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
245 |
! |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
246 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
247 |
elementAt:anObject |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
248 |
"return the element, if contained in the set. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
249 |
If there is none, report an error. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
250 |
This may seem confusing at first - however, it is useful with |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
251 |
non-identitysets, to find an existing element, for a |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
252 |
given equal (but not identical) object. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
253 |
This is the same functionality as provided by the goodies/KeyedSet |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
254 |
goody." |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
255 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
256 |
^ self elementAt:anObject ifAbsent:[^ self errorNotFound]. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
257 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
258 |
"Modified: 20.3.1997 / 20:35:24 / cg" |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
259 |
! |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
260 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
261 |
elementAt:anObject ifAbsent:exceptionBlock |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
262 |
"return the element, if contained in the set. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
263 |
If there is none, return the result from evaluating exceptionBlock. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
264 |
This may seem confusing at first - however, it is useful with |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
265 |
non-identitysets, to find an existing element, for a |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
266 |
given equal (but not identical) object. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
267 |
This is the same functionality as provided by the goodies/KeyedSet |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
268 |
goody." |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
269 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
270 |
|index "{ Class: SmallInteger }" | |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
271 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
272 |
index := self find:anObject ifAbsent:0. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
273 |
index == 0 ifTrue:[ |
6994 | 274 |
^ exceptionBlock value |
2469
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
275 |
]. |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
276 |
^ keyArray basicAt:index |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
277 |
|
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
278 |
"Created: 20.3.1997 / 20:34:07 / cg" |
ecca86d50852
added #elementAt: / #elementAt:ifAbsent:
Claus Gittinger <cg@exept.de>
parents:
2461
diff
changeset
|
279 |
"Modified: 20.3.1997 / 20:35:49 / cg" |
609 | 280 |
! ! |
281 |
||
282 |
!Set methodsFor:'adding & removing'! |
|
283 |
||
284 |
add:anObject |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
285 |
"add the argument, anObject to the receiver. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
286 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
287 |
WARNING: do not add elements while iterating over the receiver. |
6994 | 288 |
Iterate over a copy to do this." |
609 | 289 |
|
2328
0fd1d715e5a9
oops - the last one was not good
Claus Gittinger <cg@exept.de>
parents:
2323
diff
changeset
|
290 |
|index "{ Class: SmallInteger }"| |
609 | 291 |
|
5335
fae7e0698241
added a warning: adding nil will soon be an error.
Claus Gittinger <cg@exept.de>
parents:
5298
diff
changeset
|
292 |
anObject isNil ifTrue:[ |
6994 | 293 |
^ self invalidElementError. |
5335
fae7e0698241
added a warning: adding nil will soon be an error.
Claus Gittinger <cg@exept.de>
parents:
5298
diff
changeset
|
294 |
]. |
609 | 295 |
|
5335
fae7e0698241
added a warning: adding nil will soon be an error.
Claus Gittinger <cg@exept.de>
parents:
5298
diff
changeset
|
296 |
index := self findKeyOrNil:anObject. |
fae7e0698241
added a warning: adding nil will soon be an error.
Claus Gittinger <cg@exept.de>
parents:
5298
diff
changeset
|
297 |
(keyArray basicAt:index) isNil ifTrue:[ |
6994 | 298 |
"/ not already there |
299 |
keyArray basicAt:index put:anObject. |
|
300 |
tally := tally + 1. |
|
5335
fae7e0698241
added a warning: adding nil will soon be an error.
Claus Gittinger <cg@exept.de>
parents:
5298
diff
changeset
|
301 |
|
6994 | 302 |
self fullCheck. |
609 | 303 |
]. |
304 |
^ anObject |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
305 |
|
2328
0fd1d715e5a9
oops - the last one was not good
Claus Gittinger <cg@exept.de>
parents:
2323
diff
changeset
|
306 |
"Modified: 30.1.1997 / 14:58:08 / cg" |
609 | 307 |
! |
308 |
||
309 |
remove:oldObject ifAbsent:exceptionBlock |
|
310 |
"remove oldObject from the collection and return it. |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
311 |
If it was not in the collection return the value of exceptionBlock. |
6191 | 312 |
Notice, that the returned object could be non-identical to the argument |
313 |
(although it will always be equal). |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
314 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
315 |
WARNING: do not remove elements while iterating over the receiver. |
6994 | 316 |
See #saveRemove: to do this." |
609 | 317 |
|
6191 | 318 |
|index next objectRemoved| |
609 | 319 |
|
3265
18d96c910653
handle attempt to remove nil.
Claus Gittinger <cg@exept.de>
parents:
3261
diff
changeset
|
320 |
oldObject isNil ifTrue:[^ exceptionBlock value]. |
18d96c910653
handle attempt to remove nil.
Claus Gittinger <cg@exept.de>
parents:
3261
diff
changeset
|
321 |
|
609 | 322 |
index := self find:oldObject ifAbsent:0. |
323 |
index == 0 ifTrue:[^ exceptionBlock value]. |
|
324 |
||
6191 | 325 |
objectRemoved := keyArray basicAt:index. |
609 | 326 |
keyArray basicAt:index put:nil. |
327 |
tally := tally - 1. |
|
328 |
tally == 0 ifTrue:[ |
|
6994 | 329 |
keyArray := self keyContainerOfSize:(self class goodSizeFrom:0). |
609 | 330 |
] ifFalse:[ |
6994 | 331 |
index == keyArray basicSize ifTrue:[ |
332 |
next := 1 |
|
333 |
] ifFalse:[ |
|
334 |
next := index + 1. |
|
335 |
]. |
|
336 |
(keyArray basicAt:next) notNil ifTrue:[ |
|
337 |
keyArray basicAt:index put:DeletedEntry. |
|
338 |
]. |
|
339 |
self emptyCheck |
|
609 | 340 |
]. |
6191 | 341 |
^ objectRemoved |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
342 |
|
6191 | 343 |
"Modified: / 16.11.2001 / 10:14:24 / cg" |
609 | 344 |
! |
345 |
||
346 |
removeAll |
|
1164 | 347 |
"remove all elements from the receiver. Returns the receiver." |
609 | 348 |
|
349 |
self setTally:7. |
|
1164 | 350 |
|
351 |
"Modified: 12.4.1996 / 13:35:06 / cg" |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
352 |
! |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
353 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
354 |
saveRemove:oldObject |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
355 |
"remove the element, oldObject from the collection. |
6191 | 356 |
Return the element |
357 |
(could be non-identical to oldObject, since I hash on equality, not on identity). |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
358 |
If it was not in the collection return nil. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
359 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
360 |
In contrast to #remove:, this does not resize the underlying collection |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
361 |
and therefore does NOT rehash & change the elements order. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
362 |
Therefor this can be used while enumerating the receiver, |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
363 |
which is not possible if #remove: is used. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
364 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
365 |
WARNING: since no resizing is done, the physical amount of memory used |
6994 | 366 |
by the container remains the same, although the logical size shrinks. |
367 |
You may want to manually resize the receiver using #emptyCheck. |
|
368 |
(after the loop)" |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
369 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
370 |
|index "{ Class:SmallInteger }" |
6191 | 371 |
next "{ Class:SmallInteger }" |
372 |
removedObject| |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
373 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
374 |
oldObject isNil ifTrue:[^ nil]. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
375 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
376 |
index := self find:oldObject ifAbsent:0. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
377 |
index == 0 ifTrue:[^ nil]. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
378 |
|
6191 | 379 |
removedObject := keyArray basicAt:index. |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
380 |
keyArray basicAt:index put:nil. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
381 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
382 |
tally := tally - 1. |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
383 |
tally ~~ 0 ifTrue:[ |
6994 | 384 |
index == keyArray basicSize ifTrue:[ |
385 |
next := 1 |
|
386 |
] ifFalse:[ |
|
387 |
next := index + 1. |
|
388 |
]. |
|
389 |
(keyArray basicAt:next) notNil ifTrue:[ |
|
390 |
keyArray basicAt:index put:DeletedEntry |
|
391 |
]. |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
392 |
]. |
6191 | 393 |
^ removedObject |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
394 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
395 |
"does NOT work: |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
396 |
|
6994 | 397 |
|s| |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
398 |
|
6994 | 399 |
s := Set new. |
400 |
s add:1. |
|
401 |
s add:2. |
|
402 |
s add:3. |
|
403 |
s add:4. |
|
404 |
s add:5. |
|
405 |
s add:6. |
|
406 |
s add:7. |
|
407 |
s add:8. |
|
408 |
s add:9. |
|
409 |
s do:[:v | |
|
410 |
v odd ifTrue:[ |
|
411 |
s remove:v |
|
412 |
] |
|
413 |
]. |
|
414 |
s inspect |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
415 |
" |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
416 |
|
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
417 |
"DOES work: |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
418 |
|
6994 | 419 |
|s| |
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
420 |
|
6994 | 421 |
s := Set new. |
422 |
s add:1. |
|
423 |
s add:2. |
|
424 |
s add:3. |
|
425 |
s add:4. |
|
426 |
s add:5. |
|
427 |
s add:6. |
|
428 |
s add:7. |
|
429 |
s add:8. |
|
430 |
s add:9. |
|
431 |
s do:[:v | |
|
432 |
v odd ifTrue:[ |
|
433 |
s saveRemove:v |
|
434 |
] |
|
435 |
]. |
|
436 |
s inspect |
|
1052
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
437 |
" |
7d43187f7e11
added saveRemove: & commentaries
Claus Gittinger <cg@exept.de>
parents:
969
diff
changeset
|
438 |
|
6191 | 439 |
"Created: / 1.3.1996 / 21:14:26 / cg" |
440 |
"Modified: / 16.11.2001 / 10:22:59 / cg" |
|
609 | 441 |
! ! |
442 |
||
443 |
!Set methodsFor:'binary storage'! |
|
444 |
||
445 |
readBinaryContentsFrom: stream manager: manager |
|
446 |
"must rehash after reload" |
|
447 |
||
448 |
super readBinaryContentsFrom: stream manager: manager. |
|
449 |
self rehash |
|
450 |
! ! |
|
451 |
||
5280
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
452 |
!Set methodsFor:'comparing'! |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
453 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
454 |
= aCollection |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
455 |
"return true, if the argument is a Set containing the same elements |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
456 |
as I do" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
457 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
458 |
aCollection species == self species ifFalse:[^ false]. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
459 |
aCollection size == self size ifFalse:[^ false]. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
460 |
"/ same number of elements; since I am a Set, all of |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
461 |
"/ of my elements must be in the other collection ... |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
462 |
self do:[:element | |
6994 | 463 |
(aCollection includes:element) ifFalse:[ |
464 |
^ false |
|
465 |
] |
|
5280
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
466 |
]. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
467 |
^ true |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
468 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
469 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
470 |
#(1 2 3 4 5) asSet = #(2 3 4 5 1) asSet |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
471 |
#(1 2 3 4 5) asSet = #(2 3 4 5 1.0) asSet |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
472 |
#(1 2 3 4 5) asSet = #(2 3 4 5 'one') asSet |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
473 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
474 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
475 |
|d1 d2| |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
476 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
477 |
d1 := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
478 |
d2 := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
479 |
d1 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
480 |
d1 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
481 |
d1 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
482 |
d1 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
483 |
d1 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
484 |
d1 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
485 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
486 |
d2 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
487 |
d2 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
488 |
d2 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
489 |
d2 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
490 |
d2 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
491 |
d2 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
492 |
d1 = d2 |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
493 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
494 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
495 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
496 |
|d1 d2| |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
497 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
498 |
d1 := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
499 |
d2 := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
500 |
d1 at:1 put:'uno'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
501 |
d1 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
502 |
d1 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
503 |
d1 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
504 |
d1 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
505 |
d1 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
506 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
507 |
d2 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
508 |
d2 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
509 |
d2 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
510 |
d2 at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
511 |
d2 at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
512 |
d2 at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
513 |
d1 = d2 |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
514 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
515 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
516 |
! |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
517 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
518 |
hash |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
519 |
"return a hash key for the receiver" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
520 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
521 |
"this hash is stupid - but for larger collections, the hashing |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
522 |
time can become much bigger than the time lost in added probing. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
523 |
Time will show ... |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
524 |
Notice & warning: |
6994 | 525 |
if the #= method is ever changed to compare non-dictionaries equal, |
526 |
the code below must be changed to assert that the same hash-value is |
|
527 |
still returned. |
|
528 |
(which may be hard to acomplish) |
|
5280
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
529 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
530 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
531 |
|mySize| |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
532 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
533 |
mySize := self size. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
534 |
mySize == 0 ifTrue:[^ 0]. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
535 |
^ ((self first) hash times:mySize) bitAnd:16r3FFFFFFF |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
536 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
537 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
538 |
|d| |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
539 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
540 |
d := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
541 |
d at:1 put:'one'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
542 |
d at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
543 |
d at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
544 |
d at:'two' put:2. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
545 |
d hash |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
546 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
547 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
548 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
549 |
|d| |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
550 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
551 |
d := Dictionary new. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
552 |
d at:1 put:'uno'. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
553 |
d at:'one' put:1. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
554 |
d at:2 put:#two. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
555 |
d at:'two' put:2. |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
556 |
d hash |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
557 |
" |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
558 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
559 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
560 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
561 |
|
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
562 |
! ! |
ad806c9b992e
added #= & #hash; EXPERIMENTAL
Claus Gittinger <cg@exept.de>
parents:
5048
diff
changeset
|
563 |
|
6083 | 564 |
!Set methodsFor:'converting'! |
565 |
||
566 |
literalArrayEncoding |
|
567 |
"encode myself as an array literal, from which a copy of the receiver |
|
568 |
can be reconstructed with #decodeAsLiteralArray." |
|
569 |
||
570 |
|encoding| |
|
571 |
||
572 |
encoding := OrderedCollection with:(self class name). |
|
573 |
self do:[:el| encoding add:(el literalArrayEncoding) ]. |
|
574 |
^ encoding asArray |
|
575 |
||
576 |
||
577 |
" |
|
578 |
(Set with:1234 |
|
6994 | 579 |
with:(1 @ 2) |
580 |
with:'hello' |
|
6083 | 581 |
) literalArrayEncoding decodeAsLiteralArray |
582 |
" |
|
583 |
||
584 |
"Modified: 22.4.1996 / 13:00:56 / cg" |
|
585 |
! ! |
|
586 |
||
159 | 587 |
!Set methodsFor:'copying'! |
588 |
||
589 |
postCopy |
|
590 |
"have to copy the keyArray too" |
|
591 |
||
592 |
keyArray := keyArray shallowCopy |
|
1 | 593 |
! ! |
594 |
||
5471
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
595 |
!Set methodsFor:'encoding'! |
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
596 |
|
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
597 |
encodeOn:anEncoder with:aParameter |
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
598 |
|
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
599 |
anEncoder encodeSet:self with:aParameter |
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
600 |
|
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
601 |
|
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
602 |
! ! |
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
603 |
|
609 | 604 |
!Set methodsFor:'enumerating'! |
1 | 605 |
|
609 | 606 |
do:aBlock |
1056 | 607 |
"perform the block for all members in the collection. |
608 |
||
609 |
WARNING: do not add/remove elements while iterating over the receiver. |
|
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
610 |
Iterate over a copy to do this." |
13 | 611 |
|
609 | 612 |
|sz "{ Class: SmallInteger }" |
613 |
element| |
|
13 | 614 |
|
609 | 615 |
sz := keyArray size. |
616 |
1 to:sz do:[:index | |
|
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
617 |
element := keyArray at:index. |
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
618 |
(element notNil and:[element ~~ DeletedEntry]) ifTrue:[ |
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
619 |
aBlock value:element |
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
620 |
] |
609 | 621 |
] |
1056 | 622 |
|
623 |
"Modified: 1.3.1996 / 21:41:13 / cg" |
|
609 | 624 |
! ! |
61 | 625 |
|
969 | 626 |
!Set methodsFor:'inspecting'! |
627 |
||
628 |
inspectorClass |
|
629 |
"redefined to use SetInspector |
|
630 |
(instead of the default Inspector)." |
|
631 |
||
632 |
^ SetInspectorView |
|
633 |
! ! |
|
634 |
||
7169 | 635 |
!Set methodsFor:'obsolete set operations'! |
636 |
||
637 |
+ aCollection |
|
638 |
"Kept for backward compatibility. Use #union: instead, to isolate |
|
639 |
arithmethic and set operations" |
|
640 |
||
641 |
<resource: #obsolete> |
|
642 |
||
643 |
^ self union:aCollection. |
|
644 |
! |
|
645 |
||
646 |
- aCollection |
|
647 |
"Kept for backward compatibility. Use #union: instead, to isolate |
|
648 |
arithmethic and set operations" |
|
649 |
||
650 |
<resource: #obsolete> |
|
651 |
||
652 |
^ self \ aCollection |
|
653 |
! ! |
|
654 |
||
609 | 655 |
!Set methodsFor:'private'! |
2 | 656 |
|
1 | 657 |
find:key ifAbsent:aBlock |
658 |
"Look for the key in the receiver. If it is found, return |
|
659 |
the index of the slot containing the key, otherwise |
|
660 |
return the value of evaluating aBlock." |
|
661 |
||
61 | 662 |
|index "{ Class:SmallInteger }" |
663 |
length "{ Class:SmallInteger }" |
|
664 |
startIndex probe| |
|
1 | 665 |
|
13 | 666 |
length := keyArray basicSize. |
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
667 |
|
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
668 |
"/ |
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
669 |
"/ length < 10 ifTrue:[ |
362 | 670 |
"/ "assuming, that for small collections the overhead of hashing |
671 |
"/ is larger ... maybe that proves wrong |
|
672 |
"/ (if overhead of comparing is higher)" |
|
673 |
"/ ^ keyArray indexOf:key ifAbsent:aBlock |
|
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
674 |
"/ ]. |
13 | 675 |
|
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
676 |
startIndex := index := self initialIndexForKey:key. |
1 | 677 |
|
678 |
[true] whileTrue:[ |
|
6994 | 679 |
probe := (keyArray basicAt:index). |
680 |
key = probe ifTrue:[^ index]. |
|
681 |
(self slotIsEmpty:probe) ifTrue:[^ aBlock value]. |
|
1 | 682 |
|
6994 | 683 |
index == length ifTrue:[ |
684 |
index := 1 |
|
685 |
] ifFalse:[ |
|
686 |
index := index + 1 |
|
687 |
]. |
|
688 |
index == startIndex ifTrue:[^ aBlock value]. |
|
1 | 689 |
] |
690 |
! |
|
691 |
||
13 | 692 |
findKeyOrNil:key |
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
693 |
"Look for the key in the receiver. |
3082 | 694 |
If it is found, return the index of the first unused slot. |
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
695 |
Grow the receiver, if key was not found, and no unused slots were present" |
1 | 696 |
|
61 | 697 |
|index "{ Class:SmallInteger }" |
698 |
length "{ Class:SmallInteger }" |
|
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
699 |
startIndex probe |
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
700 |
delIndex| |
1 | 701 |
|
13 | 702 |
length := keyArray basicSize. |
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
703 |
startIndex := index := self initialIndexForKey:key. |
1 | 704 |
|
705 |
[true] whileTrue:[ |
|
6994 | 706 |
probe := keyArray basicAt:index. |
707 |
key = probe ifTrue:[^ index]. |
|
708 |
(self slotIsEmpty:probe) ifTrue:[ |
|
709 |
delIndex isNil ifTrue:[^ index]. |
|
710 |
keyArray basicAt:delIndex put:nil. |
|
711 |
^ delIndex |
|
712 |
]. |
|
1126
497de696dff0
OOPS - could add elements twice after a remove (shame on me: how could that go unnoticed for so long ...)
Claus Gittinger <cg@exept.de>
parents:
1056
diff
changeset
|
713 |
|
6994 | 714 |
probe == DeletedEntry ifTrue:[ |
715 |
delIndex isNil ifTrue:[ |
|
716 |
delIndex := index |
|
717 |
] |
|
718 |
]. |
|
1 | 719 |
|
6994 | 720 |
index == length ifTrue:[ |
721 |
index := 1 |
|
722 |
] ifFalse:[ |
|
723 |
index := index + 1 |
|
724 |
]. |
|
725 |
index == startIndex ifTrue:[ |
|
726 |
delIndex notNil ifTrue:[ |
|
727 |
keyArray basicAt:delIndex put:nil. |
|
728 |
^ delIndex |
|
729 |
]. |
|
730 |
^ self grow findKeyOrNil:key |
|
731 |
]. |
|
1 | 732 |
] |
1450
18c2a1da8be2
first look for nil-entry in findKeyOrNil: - saves a compare in the good case, when adding new elements
Claus Gittinger <cg@exept.de>
parents:
1319
diff
changeset
|
733 |
|
3082 | 734 |
"Modified: / 30.10.1997 / 16:01:57 / cg" |
1 | 735 |
! |
736 |
||
737 |
findNil:key |
|
2475 | 738 |
"Look for the next slot usable for key. |
739 |
WARNING: |
|
6994 | 740 |
This method assumes that key is not already in the receiver |
741 |
AND that keyArray does not have previously removed entries |
|
742 |
AND that there is an empty slot. |
|
2475 | 743 |
To be used ONLY while growing/rehashing to enter elements into a fresh |
744 |
collection - if any of the above conditions is not met, the method |
|
745 |
loops forever." |
|
1 | 746 |
|
2475 | 747 |
|index "{ Class:SmallInteger }" |
61 | 748 |
length "{ Class:SmallInteger }"| |
1 | 749 |
|
13 | 750 |
length := keyArray basicSize. |
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
751 |
index := self initialIndexForKey:key. |
1 | 752 |
|
5772 | 753 |
"/ index := keyArray identityIndexOf:nil startingAt:index. |
754 |
"/ index == 0 ifTrue:[ |
|
755 |
"/ index := keyArray identityIndexOf:nil startingAt:1 endingAt:index. |
|
756 |
"/ ]. |
|
757 |
"/ that code is completely inlined by stc ... |
|
758 |
[(keyArray basicAt:index) notNil] whileTrue:[ |
|
6994 | 759 |
index == length ifTrue:[ |
760 |
index := 1 |
|
761 |
] ifFalse:[ |
|
762 |
index := index + 1 |
|
763 |
]. |
|
764 |
"notice: no check for no nil found - we must find one since |
|
765 |
this is only called after growing" |
|
1 | 766 |
]. |
767 |
^ index |
|
1055 | 768 |
|
2475 | 769 |
"Modified: 21.3.1997 / 10:32:58 / cg" |
1 | 770 |
! |
771 |
||
2459
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
772 |
hashFor:aKey |
6991
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
773 |
"return the arguments hash value. |
e95313d7c780
comment and category change only
Claus Gittinger <cg@exept.de>
parents:
6917
diff
changeset
|
774 |
Redefined in subclasses, which use a different comparison (i.e. identity compare)" |
2459
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
775 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
776 |
^ aKey hash |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
777 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
778 |
"Created: 19.3.1997 / 15:03:03 / cg" |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
779 |
! |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
780 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
781 |
initialIndexForKey:aKey |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
782 |
"return an initial index given a key." |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
783 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
784 |
|index "{ Class:SmallInteger }" |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
785 |
length "{ Class:SmallInteger }"| |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
786 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
787 |
length := keyArray basicSize. |
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
788 |
index := (self hashFor:aKey) bitAnd:16r3FFFFFFF. |
2459
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
789 |
index < 16r1FFFFFFF ifTrue:[ |
6994 | 790 |
index := index * 2 |
2459
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
791 |
]. |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
792 |
index := index \\ length + 1. |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
793 |
^ index. |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
794 |
|
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
795 |
"Modified: 1.3.1997 / 01:01:13 / cg" |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
796 |
"Created: 19.3.1997 / 15:02:41 / cg" |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
797 |
! |
0a38e8602aac
findNil: no need to check for wraparound;
Claus Gittinger <cg@exept.de>
parents:
2427
diff
changeset
|
798 |
|
5845
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
799 |
invalidElementError |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
800 |
"this is send when a nil is used with a Set. |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
801 |
For now this is just an info-Message. Will become an error sometimes." |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
802 |
|
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
803 |
"/ self elementBoundError. |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
804 |
|
6540
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
805 |
|con| |
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
806 |
|
5845
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
807 |
'Set [warning]: nil is not a valid element - will be an error in later versions' errorPrintCR. |
6540
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
808 |
con := thisContext sender. |
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
809 |
(' from ' , con printString) errorPrintCR. |
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
810 |
con := con sender. |
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
811 |
con notNil ifTrue:[ |
6994 | 812 |
(' from ' , con printString) errorPrintCR. |
813 |
con := con sender. |
|
814 |
con notNil ifTrue:[ |
|
815 |
(' from ' , con printString) errorPrintCR. |
|
816 |
] |
|
6540
cf1a6a1a8645
better invalidElementError message
Claus Gittinger <cg@exept.de>
parents:
6519
diff
changeset
|
817 |
]. |
5845
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
818 |
! |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
819 |
|
3261
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
820 |
keyArray |
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
821 |
^ keyArray |
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
822 |
|
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
823 |
"Created: / 2.2.1998 / 14:50:55 / cg" |
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
824 |
! |
7e4e57c041df
added access to my (private) keyArray instVar (sorry)
Claus Gittinger <cg@exept.de>
parents:
3082
diff
changeset
|
825 |
|
609 | 826 |
keyContainerOfSize:n |
827 |
"return a container for keys of size n. |
|
828 |
Extracted to make life of weak subclasses easier ..." |
|
829 |
||
830 |
^ Array basicNew:n |
|
831 |
! |
|
832 |
||
1 | 833 |
rehash |
61 | 834 |
"rehash is done by re-adding all elements to a new empty set. |
835 |
Rehash is needed after a binaryRead, for example." |
|
1 | 836 |
|
3082 | 837 |
|element oldKeyArray newKeyArray newIdx |
95 | 838 |
n "{ Class:SmallInteger }"| |
1 | 839 |
|
95 | 840 |
oldKeyArray := keyArray. |
841 |
n := oldKeyArray size. |
|
842 |
keyArray := newKeyArray := self keyContainerOfSize:n. |
|
843 |
||
2 | 844 |
1 to:n do:[:index | |
6994 | 845 |
element := oldKeyArray at:index. |
846 |
(element notNil and:[element ~~ DeletedEntry]) ifTrue:[ |
|
847 |
"cannot be already there" |
|
848 |
newIdx := self findNil:element. |
|
849 |
newKeyArray basicAt:newIdx put:element |
|
850 |
]. |
|
1 | 851 |
] |
3082 | 852 |
|
853 |
"Modified: / 30.10.1997 / 16:04:46 / cg" |
|
1 | 854 |
! |
855 |
||
856 |
rehashFrom:startIndex |
|
61 | 857 |
"rehash elements starting at index - after a remove. |
858 |
Notice: due to the new implementation of remove, |
|
159 | 859 |
this is no longer needed" |
1 | 860 |
|
3 | 861 |
|element i "{ Class:SmallInteger }" |
862 |
length |
|
1 | 863 |
index "{ Class:SmallInteger }" | |
864 |
||
13 | 865 |
length := keyArray basicSize. |
1 | 866 |
index := startIndex. |
13 | 867 |
element := keyArray basicAt:index. |
1 | 868 |
[element notNil] whileTrue:[ |
159 | 869 |
i := self findNil:element. |
870 |
i == index ifTrue:[ |
|
871 |
^ self |
|
872 |
]. |
|
873 |
keyArray basicAt:i put:element. |
|
874 |
keyArray basicAt:index put:nil. |
|
1 | 875 |
|
159 | 876 |
index == length ifTrue:[ |
877 |
index := 1 |
|
878 |
] ifFalse:[ |
|
879 |
index := index + 1. |
|
880 |
]. |
|
881 |
element := keyArray basicAt:index. |
|
1 | 882 |
] |
883 |
! |
|
884 |
||
609 | 885 |
setTally:count |
886 |
"initialize the contents array (for at least count slots) |
|
887 |
and set tally to zero. |
|
888 |
The size is increased to the next prime for better hashing behavior." |
|
1 | 889 |
|
3726
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
890 |
|n| |
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
891 |
|
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
892 |
n := self class goodSizeFrom:count. |
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
893 |
n == keyArray size ifTrue:[ |
6994 | 894 |
keyArray atAllPut:nil. |
3726
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
895 |
] ifFalse:[ |
6994 | 896 |
keyArray := self keyContainerOfSize:n. |
3726
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
897 |
]. |
609 | 898 |
tally := 0 |
3726
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
899 |
|
cf3eafa152a4
try to avoid allocation of a new keyArray in #setTally:
Claus Gittinger <cg@exept.de>
parents:
3265
diff
changeset
|
900 |
"Modified: / 4.8.1998 / 22:07:25 / cg" |
6233 | 901 |
! |
902 |
||
903 |
slotIsEmpty:aSlotValue |
|
904 |
"only redefined in weak subclasses, since they treat a 0-value |
|
905 |
as being empty" |
|
906 |
||
907 |
^ aSlotValue isNil |
|
1 | 908 |
! ! |
909 |
||
7260 | 910 |
!Set methodsFor:'private-grow & shrink'! |
3082 | 911 |
|
6127
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
912 |
changeCapacityTo:newSize |
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
913 |
newSize > self capacity ifTrue:[ |
6994 | 914 |
self grow:newSize |
6127
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
915 |
]. |
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
916 |
|
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
917 |
"Created: / 30.10.2001 / 17:56:50 / cg" |
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
918 |
! |
0c4d7890cf6c
+ changeCapacityTo: (VW compatibility)
Claus Gittinger <cg@exept.de>
parents:
6083
diff
changeset
|
919 |
|
3082 | 920 |
emptyCheck |
921 |
"check if the receiver has become too empty (after a remove) |
|
922 |
and shrink if it makes sense. |
|
923 |
Definition of 'too empty' is 'filled less than 12.5% (i.e. 1/8th)'" |
|
924 |
||
925 |
|sz "{Class: SmallInteger}" |
|
926 |
newSize "{Class: SmallInteger}" | |
|
927 |
||
928 |
sz := keyArray basicSize. |
|
929 |
sz > 56 ifTrue:[ |
|
6994 | 930 |
" |
931 |
shrink if too empty |
|
932 |
" |
|
933 |
tally < (sz // 8) ifTrue:[ |
|
934 |
newSize := sz // 4. |
|
935 |
self grow:newSize |
|
936 |
] |
|
3082 | 937 |
] |
938 |
||
939 |
"Modified: 19.3.1997 / 16:02:55 / cg" |
|
940 |
! |
|
941 |
||
942 |
fullCheck |
|
943 |
"check if collection is full (after an add); grow if so. |
|
944 |
Definition of 'full' is currently: 'filled more than 75% (i.e. 3/4th)'" |
|
945 |
||
946 |
|sz "{Class: SmallInteger}" | |
|
947 |
||
948 |
" |
|
949 |
grow if filled more than 75% |
|
950 |
" |
|
951 |
sz := keyArray basicSize. |
|
952 |
tally > (sz * 3 // 4) ifTrue:[ |
|
953 |
self grow |
|
954 |
] |
|
955 |
! |
|
956 |
||
957 |
grow |
|
958 |
"change the number of element slots of the collection to a useful |
|
959 |
new size" |
|
960 |
||
961 |
self grow:(keyArray basicSize * 2) |
|
962 |
! |
|
963 |
||
964 |
grow:newSize |
|
965 |
"change the number of element slots of the collection - to do this, |
|
966 |
we have to rehash (which is done by re-adding all elements to a new |
|
967 |
empty set)." |
|
968 |
||
969 |
|elem oldKeyArray newKeyArray deletedEntry |
|
970 |
containerSize oldSize "{ Class:SmallInteger }"| |
|
971 |
||
972 |
oldKeyArray := keyArray. |
|
973 |
oldSize := oldKeyArray size. |
|
974 |
containerSize := (self class goodSizeFrom:newSize). |
|
975 |
containerSize == oldSize ifTrue:[^ self]. |
|
976 |
||
977 |
keyArray := newKeyArray := self keyContainerOfSize:containerSize. |
|
978 |
||
979 |
deletedEntry := DeletedEntry. |
|
980 |
1 to:oldSize do:[:srcIndex | |
|
981 |
elem := oldKeyArray basicAt:srcIndex. |
|
982 |
(elem notNil and:[elem ~~ deletedEntry]) ifTrue:[ |
|
983 |
"cannot be already there" |
|
984 |
newKeyArray basicAt:(self findNil:elem) put:elem |
|
985 |
]. |
|
986 |
]. |
|
987 |
! ! |
|
988 |
||
5557 | 989 |
!Set methodsFor:'queries'! |
990 |
||
991 |
size |
|
992 |
"return the number of set elements" |
|
993 |
||
994 |
^ tally |
|
995 |
! ! |
|
5471
a57eeb01c5ab
General encoding method (#encodeOn:with:)
Stefan Vogel <sv@exept.de>
parents:
5335
diff
changeset
|
996 |
|
1 | 997 |
!Set methodsFor:'testing'! |
998 |
||
362 | 999 |
capacity |
1000 |
"return the number of elements, that the receiver is |
|
1001 |
prepared to take. |
|
1002 |
Not used by the system; added for ST-80 compatibility." |
|
1003 |
||
1004 |
^ keyArray size |
|
1005 |
! |
|
1006 |
||
1 | 1007 |
includes:anObject |
1008 |
"return true if the argument anObject is in the receiver" |
|
1009 |
||
5568
826390eef742
includes:nil - allowed, but send out a warning
Claus Gittinger <cg@exept.de>
parents:
5557
diff
changeset
|
1010 |
anObject isNil ifTrue:[ |
6994 | 1011 |
"even if adding nil to a set is forbidden, asking for it should be ok!!" |
1012 |
"self invalidElementError." |
|
1013 |
^ false |
|
5568
826390eef742
includes:nil - allowed, but send out a warning
Claus Gittinger <cg@exept.de>
parents:
5557
diff
changeset
|
1014 |
]. |
362 | 1015 |
^ (self find:anObject ifAbsent:0) ~~ 0 |
1 | 1016 |
! |
1017 |
||
1018 |
isEmpty |
|
1019 |
"return true if the receiver is empty" |
|
1020 |
||
1021 |
^ tally == 0 |
|
1022 |
! |
|
1023 |
||
609 | 1024 |
isFixedSize |
1025 |
"return true if the receiver cannot grow - this will vanish once |
|
1026 |
Arrays and Strings learn how to grow ..." |
|
1027 |
||
1028 |
^ false |
|
1029 |
! |
|
1030 |
||
2383 | 1031 |
notEmpty |
1032 |
"return true if the receiver is not empty" |
|
1033 |
||
1034 |
^ tally ~~ 0 |
|
1035 |
||
1036 |
"Created: 12.2.1997 / 12:39:02 / cg" |
|
1037 |
! |
|
1038 |
||
1 | 1039 |
occurrencesOf:anObject |
5048 | 1040 |
"return the number of occurrences of anObject in the receiver. |
6191 | 1041 |
As I am a Set, this can only return 0 or 1." |
1 | 1042 |
|
5845
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
1043 |
anObject isNil ifTrue:[ |
6994 | 1044 |
"even if adding nil to a set is forbidden, asking for it should be ok!!" |
1045 |
"self invalidElementError." |
|
1046 |
^ 0. |
|
5845
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
1047 |
]. |
64a385f14970
Use #invalidElementError to warn about illegal use of nil in Sets
Stefan Vogel <sv@exept.de>
parents:
5834
diff
changeset
|
1048 |
|
362 | 1049 |
(self find:anObject ifAbsent:0) == 0 ifTrue:[^ 0]. |
1 | 1050 |
^ 1 |
6191 | 1051 |
|
1052 |
"Modified: / 16.11.2001 / 10:30:14 / cg" |
|
1 | 1053 |
! |
1054 |
||
4795 | 1055 |
sameContentsAs:aCollection |
1056 |
"answer true, if all the elements in self and aCollection |
|
1057 |
are common. This is not defined as #=, since we cannot redefine #hash |
|
1058 |
for aCollection." |
|
1059 |
||
1060 |
aCollection size ~~ self size ifTrue:[ |
|
6994 | 1061 |
^ false |
4795 | 1062 |
]. |
1063 |
||
1064 |
aCollection do:[:e| |
|
6994 | 1065 |
(self includes:e) ifFalse:[ |
1066 |
^ false. |
|
1067 |
] |
|
4795 | 1068 |
]. |
1069 |
||
1070 |
^ true. |
|
1071 |
||
1072 |
||
1073 |
" |
|
1074 |
#(1 2 3) asSet sameContentsAs: #(1 2 3) |
|
1075 |
#(1 2 3 4) asSet sameContentsAs: #(1 2 3) |
|
1076 |
#(1 2 3) asSet sameContentsAs: #(1 2 3 3) |
|
1077 |
#(1 2 3 'aa') asSet sameContentsAs: #(1 2 3 'aa') |
|
1078 |
#(1 2 3 'aa') asIdentitySet sameContentsAs: #(1 2 3 'aa') |
|
1079 |
#(1 2 3 #aa) asIdentitySet sameContentsAs: #(1 2 3 #aa) |
|
1080 |
" |
|
1081 |
||
1082 |
||
1083 |
||
1084 |
||
1085 |
||
1 | 1086 |
! ! |
1087 |
||
4017
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1088 |
!Set::EmptySlot class methodsFor:'instance creation'! |
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1089 |
|
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1090 |
basicNew |
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1091 |
TheOneAndOnlyInstance isNil ifTrue:[ |
6994 | 1092 |
TheOneAndOnlyInstance := super basicNew |
4017
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1093 |
]. |
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1094 |
^ TheOneAndOnlyInstance |
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1095 |
|
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1096 |
! ! |
cdf103ce2e30
there can be only one EmptySlot
Claus Gittinger <cg@exept.de>
parents:
3726
diff
changeset
|
1097 |
|
2259 | 1098 |
!Set class methodsFor:'documentation'! |
635 | 1099 |
|
1100 |
version |
|
7311
6878116b0a27
+copyWithout/copyWithoutAll:
Claus Gittinger <cg@exept.de>
parents:
7268
diff
changeset
|
1101 |
^ '$Header: /cvs/stx/stx/libbasic/Set.st,v 1.86 2003-05-19 18:40:02 cg Exp $' |
635 | 1102 |
! ! |
6917
979bfb9267e3
refactored to invoke #hash/#identityHash only from one place
Claus Gittinger <cg@exept.de>
parents:
6577
diff
changeset
|
1103 |
|
609 | 1104 |
Set initialize! |