can somebody tell me were this is wrong :D

GoToLoop

Re: can somebody tell me were this is wrong :D

1 month ago

I think you should use put() method before get().
That is, you gotta store something before accessing what's stored, right?

Even for the inner HashMap's instantiation, you gotta put() that into the outer HashMap!
After that, you gotta put() keys/values inside the inner HashMap.
Only then you can start using get()!

In a previous post of yours, my example clearly demonstrates that:
http://forum.processing.org/topic/can-i-do-hashmap-integer-hashmap-integet-boolean-and-more

I still think you don't need such a complex structure!

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

ty

quarks

Re: can somebody tell me were this is wrong :D

1 month ago

The code logic is a indecipherable and the purpose of the code is unknown because you don't explain it - so how do you expect a decent answer?

One error is line 22 because it will always throws a Null Pointer Eception because the only way to get here is if the identical statement in line 18 throws an exception, so delete line 22.

Please note that this was pointed out in reply to your previous post.

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

ty alot :D

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

after all this the code still gives error:

This should do the follow:

take sub from edjesMap.get(va)

see is sub have any key vb

if not sub.put (vb,true)

and edjes.put(va,sub);

and take sub1 from edjesMap.get(vb);

and sub1.put(va,true);

and edjesMap.put(vb,vb);

import java.util.Map;
Map<Integer, Map<Integer, Boolean>> edjesMap = new HashMap();
Map<Integer, Boolean> sub = new HashMap();
int vb = 0;
int va = 0;
Boolean help;
// AAAAA-------------------
sub = edjesMap.get(va);
// BBBBB-------------------
help = sub.get(vb);
if(help == null) {
sub.put(vb,true);
edjesMap.put(va,sub);
Map<Integer,Boolean> sub1 = new HashMap();
sub1 = edjesMap.get(vb);
sub1.put(va,true);
edjesMap.put(vb,sub1);

}can u plz tell me a solution but plz test it first

quarks

Re: can somebody tell me were this is wrong :D

1 month ago

This should do the follow:

take sub from edjesMap.get(va)
see is sub have any key vb
if not sub.put (vb,true)
and edjes.put(va,sub);
and take sub1 from edjesMap.get(vb);
and sub1.put(va,true);
and edjesMap.put(vb,sub1);

Some questions for you

In line 1 the get will return null if edjesMap does not contain the key va - what should happen in that case because sub == null?
In line 5 the get will return null if edjesMap does not contain the key vb - what should happen in that case because sub1 == null?
I have change line 7 is that correct?

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

ok i got u point

and i made the repair an it work

import java.util.Map;
Map<Integer, Map<Integer, Boolean>> edjesMap = new HashMap();
Map<Integer, Boolean> sub = new HashMap();
int vb = 0;
int va = 0;
Boolean help;
// AAAAA-------------------
sub = edjesMap.get(va);
if(sub == null) {
sub = new HashMap();
}
// BBBBB-------------------
help = sub.get(vb);
if(help == null) {
sub.put(vb,true);
edjesMap.put(va,sub);
Map<Integer,Boolean> sub1 = new HashMap();
sub1 = edjesMap.get(vb);
sub1.put(va,true);
edjesMap.put(vb,sub1);
}

but when i made the reapir in my semi-full code, it still wrong.

i cant show u the full code because is too bigg

but do u know a way that that could go wrong

note:the error is in that part of the code.

i can show the page of the code, but will not compile ,because , all the other classes are in diferente pages

watch:

(ps : that is the class object, resposeble for holding all the hashmap and arraylists that hold vertex , surfaces,and edjes

ps : the error is in the function addSurf, this one should create the surface and check if the edjes of that surface already exist, if not it will create them)

ArrayList objs = new ArrayList();
// CLASS -----------------------------------------------------------------------------------------------------------------
class obj {
obj() {
}
obj(float x ,float y , float z) {
posx = x;
posy = y;
posz = z;
}
obj(float x ,float y , float z,float rx ,float ry , float rz) {
posx = x;
posy = y;
posz = z;
rotx = rx;
roty = ry;
rotz = rz;
}
// VARIABLES -----------------------------------------------------------------------------------------------------------------
boolean selected = false;
ArrayList surfs = new ArrayList();
ArrayList edjes = new ArrayList();
Map <Integer,vert> verts = new HashMap<Integer,vert>();
Map<Integer, Map<Integer,Boolean>> edjesMap = new HashMap();
int[] selectVerts =new int [0];
int[] selectFaces =new int [0];
int[] selectEdjes =new int [0];
float posx=0;
float posy=0;
float posz=0;
int sPosx;
int sPosy;
float rotx = 0;
float roty = 0;
float rotz = 0;
boolean visible = false;
int cont = -1;
int []delVerts = new int[0];
//EDJE_CONTROLL------------------------------------------------------------------------------------------------------------------------
void edjesMove() {
for(int i = 0; i < edjes.size();i++) {
edje e = (edje) edjes.get(i);
vert v1 = (vert) verts.get(e.v1);
vert v2 = (vert) verts.get(e.v2);
e.cPosx = (v1.posx + v2.posx)*0.5;
e.cPosy = (v1.posy + v2.posy)*0.5;
e.cPosz = (v1.posz + v2.posz)*0.5;
}
}
// CLASS FUNCTIONS -----------------------------------------------------------------------------------------------------------------
void facesMove() {
for(int i = 0; i < surfs.size();i++) {
surf s =(surf) surfs.get(i);
vert v1 = (vert) verts.get(s.p1);
vert v2 = (vert) verts.get(s.p2);
vert v3 = (vert) verts.get(s.p3);
s.cPosx = (v1.posx + v2.posx + v3.posx)*0.333;
s.cPosy = (v1.posy + v2.posy + v3.posy)*0.333;
s.cPosz = (v1.posz + v2.posz + v3.posz)*0.333;
}
}
// CREATE VERTEX -----------------------------------------------------
int createVert(float x, float y , float z) {
if(delVerts.length >0){
int i = delVerts[delVerts.length -1];
delVerts = shorten(delVerts);
verts.put(i,new vert(x,y,z,i));
return i;
}
else{
cont++;
verts.put(cont,new vert(x,y,z,cont));
return cont;
}
}
// DELETE VERTEX -----------------------------------------------------
void delVert(int i){
delVerts = append(delVerts,i);
verts.remove(i);
}
// ADDING SURFS -----------------------------------------
void addSurf(int a , int b, int c) {
int va = 0;
int vb = 0;
int vc = 0;
surfs.add(new surf(a,b,c));
for(int i= 0; i<3 ; i++) {
if(i == 0) {va =a;vb = b; vc = c;}
else if(i == 1) {va =c;vb = a; vc = b;}
else if(i == 2) {va =b;vb = a; vc = c;}
Map<Integer,Boolean> sub = new HashMap();
boolean help;
// AAAAA-------------------
sub = edjesMap.get(va);
if(sub == null){
sub = new HashMap();
}
// BBBBB-------------------
help = sub.get(vb);
if(help == null) {
sub.put(vb,true);
edjesMap.put(va,sub);
Map<Integer,Boolean> sub1 = new HashMap();
sub1 = edjesMap.get(vb);
if(sub1 == null) {
sub1 = new HashMap();
}
sub1.put(va,true);
edjesMap.put(vb,sub1);
edjes.add(new edje(va,vb));
}
//-----
// CCCCC-------------------
/*try{
help = sub.get(vc);
}
catch(Exception e ){
e.printStackTrace();
sub.put(vc,true);
HashMap<Integer,Boolean> sub1 = new HashMap();
try{
sub1 = edjesMap.get(vc);
sub1.put(va,true);
}
catch(Exception w ){
w.printStackTrace();
sub1.put(va,true);
}
edjesMap.put(vc,sub1);
edjes.add(new edje(va,vc));
}*/
}
}
}

if it is any way form e send u the full project´

plz say.

:D

ty for u ilemited patience.

:D

just one more thing is u are interested in elpping me in this project i will be very grateffull(let me know)

note:

this project is a software toedit and create geometry

then will slice the geometric in 2d data to be later printer in a 3d printer that im doing

if u were interested let me know and i will tell u more of wht is done and wht is stilll needed to do

:D

cya

quarks

Re: can somebody tell me were this is wrong :D

1 month ago

If you look in the fist block of code you pasted - in section BBB lines 27 - 30 are

27 Map<Integer,Boolean> sub1 = new HashMap();
28 sub1 = edjesMap.get(vb);
29 sub1.put(va,true);
30 edjesMap.put(vb,sub1);

In line 27 you create a new HashMap called sub1 but then the statement in line 28 replaces it with one from edjesMap but if it doesn't exist then sub1 becomes null. Then you get an error on line 29 . Replace lines 27-30 with

Map<Integer,Boolean> sub1 = edjesMap.get(vb);
if(sub1 == null)
sub1 = new HashMap();
sub1.put(va,true);
edjesMap.put(vb,sub1);

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

i replace and still an error

watch blue part:

ArrayList objs = new ArrayList();
// CLASS -----------------------------------------------------------------------------------------------------------------
class obj {
obj() {
}
obj(float x ,float y , float z) {
posx = x;
posy = y;
posz = z;
}
obj(float x ,float y , float z,float rx ,float ry , float rz) {
posx = x;
posy = y;
posz = z;
rotx = rx;
roty = ry;
rotz = rz;
}
// VARIABLES -----------------------------------------------------------------------------------------------------------------
boolean selected = false;
ArrayList surfs = new ArrayList();
ArrayList edjes = new ArrayList();
Map <Integer,vert> verts = new HashMap<Integer,vert>();
Map<Integer, Map<Integer,Boolean>> edjesMap = new HashMap();
int[] selectVerts =new int [0];
int[] selectFaces =new int [0];
int[] selectEdjes =new int [0];
float posx=0;
float posy=0;
float posz=0;
int sPosx;
int sPosy;
float rotx = 0;
float roty = 0;
float rotz = 0;
boolean visible = false;
int cont = -1;
int []delVerts = new int[0];
//EDJE_CONTROLL------------------------------------------------------------------------------------------------------------------------
void edjesMove() {
for(int i = 0; i < edjes.size();i++) {
edje e = (edje) edjes.get(i);
vert v1 = (vert) verts.get(e.v1);
vert v2 = (vert) verts.get(e.v2);
e.cPosx = (v1.posx + v2.posx)*0.5;
e.cPosy = (v1.posy + v2.posy)*0.5;
e.cPosz = (v1.posz + v2.posz)*0.5;
}
}
// CLASS FUNCTIONS -----------------------------------------------------------------------------------------------------------------
void facesMove() {
for(int i = 0; i < surfs.size();i++) {
surf s =(surf) surfs.get(i);
vert v1 = (vert) verts.get(s.p1);
vert v2 = (vert) verts.get(s.p2);
vert v3 = (vert) verts.get(s.p3);
s.cPosx = (v1.posx + v2.posx + v3.posx)*0.333;
s.cPosy = (v1.posy + v2.posy + v3.posy)*0.333;
s.cPosz = (v1.posz + v2.posz + v3.posz)*0.333;
}
}
// CREATE VERTEX -----------------------------------------------------
int createVert(float x, float y , float z) {
if(delVerts.length >0){
int i = delVerts[delVerts.length -1];
delVerts = shorten(delVerts);
verts.put(i,new vert(x,y,z,i));
return i;
}
else{
cont++;
verts.put(cont,new vert(x,y,z,cont));
return cont;
}
}
// DELETE VERTEX -----------------------------------------------------
void delVert(int i){
delVerts = append(delVerts,i);
verts.remove(i);
}
// ADDING SURFS -----------------------------------------
void addSurf(int a , int b, int c) {
int va = 0;
int vb = 0;
int vc = 0;
surfs.add(new surf(a,b,c));
for(int i= 0; i<3 ; i++) {
if(i == 0) {va =a;vb = b; vc = c;}
else if(i == 1) {va =c;vb = a; vc = b;}
else if(i == 2) {va =b;vb = a; vc = c;}
Map<Integer,Boolean> sub = new HashMap();
boolean help;
// AAAAA-------------------
sub = edjesMap.get(va);
if(sub == null){
sub = new HashMap();
}
// BBBBB-------------------
help = sub.get(vb);
if(help == null) { // the operator == is undifined for the argument type(s) boolean , null
sub.put(vb,true);
edjesMap.put(va,sub);
Map<Integer,Boolean> sub1 = new HashMap();
sub1 = edjesMap.get(vb);
if(sub1 == null) {
sub1 = new HashMap();
}
sub1.put(va,true);
edjesMap.put(vb,sub1);
edjes.add(new edje(va,vb));
}
}
}
}
// STARTING OBJECTS -------------------------------
void plane(float x, float y ,float z,float s){
objs.add(new obj(x,y,z));
obj o = (obj) objs.get(objs.size() - 1);
int a = o.createVert((-1 *s),(-1*s),0f);
int w = o.createVert((-1*s),(1*s),0f);
int d = o.createVert((1*s),(1*s),0f);
int f = o.createVert((1*s),(-1*s),0f);
o.surfs.add(new surf(a,w,d));
o.surfs.add(new surf(d,f,a));
o.facesMove();
o.edjesMove();
}
void square(float x, float y ,float z,float s){
objs.add(new obj(x,y,z));
obj o = (obj) objs.get(objs.size() - 1);
int a = o.createVert((-1 *s),(-1*s),(-1*s));
int w = o.createVert((-1*s),(1*s),(-1*s));
int d = o.createVert((1*s),(1*s),(-1*s));
int f = o.createVert((1*s),(-1*s),(-1*s));
int g = o.createVert((-1 *s),(-1*s),(1*s));
int h = o.createVert((-1*s),(1*s),(1*s));
int j = o.createVert((1*s),(1*s),(1*s));
int k = o.createVert((1*s),(-1*s),(1*s));
// botton Face ------------------------
o.addSurf(a,w,d);
o.addSurf(d,f,a);
// Top Face --------------------------
o.addSurf(g,h,j);
o.addSurf(j,k,g);
// Left Face ------------------------
o.addSurf(a,g,k);
o.addSurf(a,k,f);
// Right Face --------------------------
o.addSurf(w,d,j);
o.addSurf(j,h,w);
//forwar Face -------------------------
o.addSurf(k,j,d);
o.addSurf(d,f,k);
// Back Face --------------------------
o.addSurf(a,g,h);
o.addSurf(h,w,a);
o.facesMove();
o.edjesMove();
}
Still gives an error at the red part
:(

PhiLho

Re: can somebody tell me were this is wrong :D

1 month ago

You have to learn the difference between Boolean (object) and boolean (primitive type).

GoToLoop

Re: can somebody tell me were this is wrong :D

1 month ago

Java's 8 primitive data-types can't be assigned and compared to null.
Only object/reference variable types can have null.

mmm.fuinha

Re: Re: can somebody tell me were this is wrong :D

1 month ago

ty all for helping me

::D

TTTTTTYYYYYYY AAAAAALLLLLLLL!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!