Open GoogleCodeExporter opened 8 years ago
Yes, the file requests are asynchronous and the responses may come in arbitrary
order.
I guess you are using a group of loader objects to request STL models from
multiple sources, with the loader.onload callbacks overridden for adding them
into one scene when done. In this case, since JavaScript allows adding
arbitrary properties to an existing object, we can just assign an unique ID to
a new mesh object as it has been loaded:
...
var loader = ...;
loader.onload = function(scene) {
if (scene.getChildren().length > 0) {
// an STL model contains only one mesh object
var mesh = scene.getChildren()[0];
// we assign an self-defined ID to the newly loaded mesh object
mesh.userId = anUniqueIdString;
// now add this mesh into the scene for display
...
}
};
loader.loadFromUrl(url);
...
Now each mesh has a self-defined new property 'userId', we just utilize them to
identify mesh objects in our application.
Original comment by Humu2...@gmail.com
on 5 Oct 2014 at 2:29
I tried to insert the suggestions on my code. There are some errors that I do
not know eliminate. Bellow my code.
1- at row: loader.onload = function(scene,idx_file){}, in function idx_file is
lost
2- setobject() not function
Thank
----------------------
var canvas = document.getElementById('cv');
viewer = new JSC3D.Viewer(canvas);
theScene = new JSC3D.Scene;
var numOfLoaded = 0;
viewer.setParameter('InitRotationX', 0);
viewer.setParameter('InitRotationY', 0);
viewer.setParameter('InitRotationZ', 0);
viewer.setParameter('ModelColor', '#FF0000');
viewer.setParameter('BackgroundImageUrl', 'images/bg_3D_viewer.jpg');
viewer.setParameter('Definition', 'standard');
viewer.setParameter('RenderMode', 'flat');
viewer.setParameter('Renderer', 'webgl');
viewer.setParameter('ProgressBar', 'on');
viewer.init();
// table components
tbl_components[0]=[id1,'obj_desc_1','file1.stl'];
tbl_components[1]=[id2,'obj_desc_2','file2.stl'];
tbl_components[2]=[id3,'obj_desc_3','file3.stl'];
tbl_components[3]=[id4,'obj_desc_4','file4.stl'];
tbl_components[4]=[id5,'obj_desc_5','file5.stl'];
var num_comp=tbl_components.length-1;
for (var jr=0; jr<num_comp; jr++) {
var loader = new JSC3D.StlLoader;
idx_file=tbl_components[jr,0];
loader.onload = function(scene,idx_file) {
if (scene.getChildren().length > 0){
var mesh = scene.getChildren()[0];
// set object_id linked to tbl_components
mesh.userId = idx_file;
theScene.addChild(mesh);
mesh.setMaterial(new JSC3D.Material('material', 0, components_color[idx_file]));
}
if (++numOfLoaded ==components.length ) {
viewer.replaceScene(theScene);
for(obj in viewer.scene.children) {
idx_file=obj.userId
obj_desc=tbl_components[idx_file,1];
// display checkbox (show/hide) - obj_desc
shtml+="<input type='checkbox' value='" + obj + "' checked onclick='setobject(this);'/>" + obj_desc + "<br />";
};
div_objects_list.innerHTML = shtml;
}
};
loader.loadFromUrl(tbl_components[jr,2]);
}
viewer.update();
////
function setobject(self) {
viewer.scene.children[self.value].visible = self.checked;
}
------------------------------------------
Original comment by 27vii...@gmail.com
on 5 Oct 2014 at 8:01
The callback loader.onload only takes a single parameter. You cannot pass in
your IDs in that way. Try to move your implementation of the onload function to
a closure instead:
for (var jr=0; jr<num_comp; jr++) {
...
var idx_file=tbl_components[jr][0];
loader.onload = (function(userId) {
return function onload(scene) {
if (...) {
...
// set object_id linked to tbl_components
mesh.userId = userId;
console.info('ID for this mesh is ' + mesh.userId);
...
}
...
};
})(idx_file);
}
...
This makes use of closures to bind mesh objects to correct user-defined IDs.
You can refer to this article
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Closures if you
are not familiar with either the syntax or the usage of JavaScript closure
feature.
Besides, maybe you have to check your codes a second time. I guess it contains
more than one mistakes.
Original comment by Humu2...@gmail.com
on 5 Oct 2014 at 10:03
Original issue reported on code.google.com by
27vii...@gmail.com
on 4 Oct 2014 at 10:41