seems like a handle class nesting bug

1 view (last 30 days)
Stuart MacGregor
Stuart MacGregor on 21 May 2023
Commented: Stuart MacGregor on 21 May 2023
classdef IqReader < handle
properties
fid=0
end
...
end
classdef InReader < handle
properties
reader=IqFileReader()
end
...
end
classdef T1Reader < InReader
...
end
classdef T2Reader < InReader
...
end
t1=T1Reader();
t2=T2Reader();
assert(t1.reader ~= t2.reader);
In the example above (when split into files as matlab requires), t1 and t2 get the same handle class object as a reader. e.g. changing t1.reader.fid changes t2.reader.fid
This does not seem right to me.
  2 Comments
Stuart MacGregor
Stuart MacGregor on 21 May 2023
cut and paste bug in example: first class should be IqFileReader
Image Analyst
Image Analyst on 21 May 2023
Then edit it to correct the first post rather than post the correction as a comment.

Sign in to comment.

Answers (1)

Steven Lord
Steven Lord on 21 May 2023
Not a bug. See the "Handle Objects as Default Property Values" section on this documentation page for an expanation and an alternate approach to give each instance of the class a separate handle object as its property.
  2 Comments
Stuart MacGregor
Stuart MacGregor on 21 May 2023
The doc is unclear re my example however. It says different instances get the same handle-derived property: ok, but my example did not have different instances of the same class, it had two different derived classes, each with a single instance.
This is not good, and not documented. Call it an undocumented feature rather than a bug possibly.
Stuart MacGregor
Stuart MacGregor on 21 May 2023
However, it does seem good practice to initialise handle derived properties in the constructor - the behaviour if you don't is very unlikely to be desired, or expected.

Sign in to comment.

Products


Release

R2022b

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!