If condition failure

Your code will always say either "apple is absent" or "orange is absent" because for each window, it will check first that it is not "apple" - if so, it will print that, otherwise if it's not "orange" it will print that. Put another way, if the window contained "apple", it would tell you that orange was absent. If it contained "orange", it would tell you that apple is absent. That's not really a specific SKILL issue, it's a programming logic issue.

I suspect what  you really want to do is find out for a particular lib/cell/view whether it's in a window already. You could do:

procedure(CCFfindExistingWindow(libName cellName viewName)
  let((svw)
    car(exists(win hiGetWindowList()
     {
       svw=win~>cellView
       svw~>libName==libName && svw~>cellName==cellName svw~>viewName==viewName
     }
    ))
  )
)

If you then do CCFfindExistingWindow("lp_amr" "apple" "layout") it would return the window id of the window containing it, or nil if not found.

Alternatively, you could use these built-in functions instead:

procedure(CCFfindExistingWindow(libName cellName viewName)
  let((cv)
    cv=dbFindOpenCellView(ddGetObj(libName) cellName viewName)
    cv && geGetEditCellViewWindow(cv)
  )
)

That's probably a bit more direct.

Regards,

Andrew.

Hi Laksmiprahanth,

I think that the issue is that perhaps you are expecting the "!" to cover combined "&&" statements, however it has a higher precedence than the other operators and so only applies to the first item of the statement. You can rewrite the statement using the function rather than the operator, it should correct the problem and may offer more clarity:

if( null(and((svw~>libName =="lp_amr")  (svw~>cellName == "apple") (svw~>viewName == "layout"))) then ...

Hope this helps!

Best regards,

Lawrence.

Hi Lawrence,

Our responses crossed. However, I don't think you're right. The three && conditions are surrounded by parentheses, and so the ! is doing the not of all of them, not just the first. Put another way, it's the same as your code.

To show this, I used sstatus(printinfix nil) and pp'd the original code:

(if
  (null
    (and
      (equal
        (getSGq svw libName) "lp_amr"
      )
      (equal
        (getSGq svw cellName) "apple"
      )
      (equal
        (getSGq svw viewName) "layout"
      )
   )
 ) then
 (printf "apple is absent\n")
)

Regards,

Andrew.