Debugging : Application defined or object defined error

[|)iGiTaL]

Hi my code groups areas with same value i.e. it reads a cell that is not yet assigned to a area and find all cells which are connected to it ( similar to tracing the tree from its root to its leaves except that the root is the first cell that has not been assigned to any area previously) have the same value as it has to form an area.

This is the code which I have written to for the above task:

When I run the code I get the following error:

"Application defined or object defined error"

Please Login or Register  to view this content.

I hope someone could help me in debugging this error as excel debug is not very helpful.

Thanks in advance

[skywriter]

When I run the code I get the following error:

"Application defined or object defined error"

Although it's helpful to describe the error, it even more helpful to also state at which line you are getting the error.

[mikerickson]

This line at the start of FindTrustZone will error if row is = 1

Please Login or Register  to view this content.

A fix would be

Please Login or Register  to view this content.

In the failing code (when row=1) the line tries to evaluate whether Cells(0, col) = x (trying to look at row 0 is the error)

In the second code, if row = 1, it doesn't try to evaluate Cells(row -1, col)

[|)iGiTaL]

@skywriter

The error occurs in every executable line in function FindTrustZones.

[skywriter]

I don't know about using a keyword like row as a variable. Every executable line?
How do you get it to go to the other lines after it stops on the first executable row, which it should do if it causes an error?

[shg]

I think you have a few logic errors.

This ...

Please Login or Register  to view this content.

... turns this

Row\Col	A	B	C	D	E	F	G	H	I	J
1	1	1	1	2	2	3	4	6	7	7
2	1	3	3	3	4	6	7	7	2	4
3	1	2	2	3	3	3	3	3	5	5
4	1	3	5	6	6	1	1	3	4	5
5	1	1	3	4	5	6	6	1	1	2
6	1	2	4	4	6	6	7	7	1	1
7	1	2	2	3	3	5	6	1	2	3
8	1	2	4	4	6	6	1	3	3	5
9	1	1	3	4	5	6	7	7	2	3
10	1	2	3	5	6	1	1	2	2	4
11	1	3	3	3	5	5	5	5	6	7
12	1	2	3	4	5	7	2	2	3	5
13	1	3	3	4	5	5	6	7	2	4
14	1	2	2	3	4	5	5	5	5	7
15	1	1	3	4	4	5	6	7	7	2
16	1	3	3	4	4	5	6	6	6	1

... into this:

Row\Col	A	B	C	D	E	F	G	H	I	J
1	TZ-1	TZ-1	TZ-1	TZ-2	TZ-2	TZ-3	TZ-4	TZ-5	TZ-6	TZ-6
2	TZ-1	TZ-7	TZ-7	TZ-7	TZ-8	TZ-9	TZ-10	TZ-10	TZ-11	TZ-12
3	TZ-1	TZ-13	TZ-13	TZ-7	TZ-7	TZ-7	TZ-7	TZ-7	TZ-14	TZ-14
4	TZ-1	TZ-15	TZ-16	TZ-17	TZ-17	TZ-18	TZ-18	TZ-7	TZ-19	TZ-14
5	TZ-1	TZ-1	TZ-20	TZ-21	TZ-22	TZ-23	TZ-23	TZ-24	TZ-24	TZ-25
6	TZ-1	TZ-26	TZ-21	TZ-21	TZ-23	TZ-23	TZ-27	TZ-27	TZ-24	TZ-24
7	TZ-1	TZ-26	TZ-26	TZ-28	TZ-28	TZ-29	TZ-30	TZ-31	TZ-32	TZ-33
8	TZ-1	TZ-26	TZ-34	TZ-34	TZ-35	TZ-35	TZ-36	TZ-37	TZ-37	TZ-38
9	TZ-1	TZ-1	TZ-39	TZ-34	TZ-40	TZ-35	TZ-41	TZ-41	TZ-42	TZ-43
10	TZ-1	TZ-44	TZ-39	TZ-45	TZ-46	TZ-47	TZ-47	TZ-42	TZ-42	TZ-48
11	TZ-1	TZ-39	TZ-39	TZ-39	TZ-49	TZ-49	TZ-49	TZ-49	TZ-50	TZ-51
12	TZ-1	TZ-52	TZ-39	TZ-53	TZ-49	TZ-54	TZ-55	TZ-55	TZ-56	TZ-57
13	TZ-1	TZ-39	TZ-39	TZ-53	TZ-49	TZ-49	TZ-58	TZ-59	TZ-60	TZ-61
14	TZ-1	TZ-62	TZ-62	TZ-63	TZ-64	TZ-49	TZ-49	TZ-49	TZ-49	TZ-65
15	TZ-1	TZ-1	TZ-66	TZ-64	TZ-64	TZ-49	TZ-67	TZ-68	TZ-68	TZ-69
16	TZ-1	TZ-66	TZ-66	TZ-64	TZ-64	TZ-49	TZ-67	TZ-67	TZ-67	TZ-70

[|)iGiTaL]

@skywriter: I was able to debug the error. Thanx for your help

@shg: No I dont have logic error. My logic is correct. That I have checked.

@mikerickson : Thanx for helping me debug the error.

This is the first time I am coding in VBA, so might ask really silly doubts too pls bear with me.

After correcting code as mentioned by mikerickson, I encountered a new error "Run time error 28 out of stack space". Any help on that ??

[shg]

@shg: No I dont have logic error. My logic is correct. That I have checked.

This moves right (which I would call East), not left, and you should be checking that you're not in the rightmost column:

Please Login or Register  to view this content.

[skywriter]

After correcting code as mentioned by mikerickson, I encountered a new error "Run time error 28 out of stack space". Any help on that ??

Recursive calls?
Out of Stack Space.

[|)iGiTaL]

oh ok i get it.

[|)iGiTaL]

@skywriter: yes Im calling the function recursively.
I cant get ride of the recursive calls as I need it.