Stats

  • Locked Locked
  • Replies 8
  • Subscribers 145
  • Views 32492
  • Members are here 0
This discussion has been locked.
You can no longer post new replies to this discussion. If you have a question you can start a new discussion

SKILL if statement

nosaj

Can anyone point out why doesn't the if statement below return YES since argType is a string?

Code : if.il
;=============================
a="a"
argType=type(a)
printf("argType=(%s)\n" argType)

if( argType=="string" then
        printf("YES (%s)\n" argType)
else
        printf("NO   (%s)\n" argType)
)
;=============================


CIW Output :

load("if.il")
argType=(string)
NO   (string)
t

  • nosaj,

     

    Try this instead:

    if( argType=='string then

    printf("YES (%s)\n" argType)

    else

    printf("NO   (%s)\n" argType)

    )

  • Because type returns a symbol, not a string. There's an important difference between the two. A symbol is typically the name of something (in this case the name of the type, but could be the name of a variable, or the name of a function) rather than a string of characters.

    If you had:

    if(argType=='string then ...

    Then it would do what you want.

    Regards,

    Andrew

  • I have 2 questions about using the "if" statement

    1)  The variable that is being tested will have quotes and it needs to not have quotes for the if statement, is there an easy way to strip them off?   example :    "4"
    p, li { white-space: pr     

     2)   how do test for a range    Less 4 but greater than 1  (  <4 >1  )

     if( mtOption=="Manual" then
          if( numThreads = 8 then
               multiThread = "+mt=8")

  • 1. You can convert a string to a number in a couple ways:
    num = atof(var)
    num = evalstring(var)

    2. if(num > 1 && num < 4 then

    Derek

  • using atoi in the CIW worked but not in the code.   evalstring did work.  Any thoughts why the atoi didnt work?

       numThreads_opt=asiEnvGetVal(asiSession "turboOpts" "numThreads")

        numthreads = atoi(numThreads_opt)

        numthreads = evalstring(numThreads_opt)

  • Hi , i have written code to change metal layers  using if else statements , I am getting following error. not able to resolve ..

    *Error* if: too many arguments (missing then?) - ((((&~>lpp) == `&) then) setSGq(geGetSelSet() `("M2" "drawing") lpp) else if(((&~>lpp) == `&) then) setSGq(geGetSelSet() `("M3" "drawing") lpp) ... )




    hiSetBindKey( "Layout" "shift<Key>m" "metals()")
    procedure( metals()
                
            if(((( geGetSelSet()~>lpp) == `("M1" "drawing")) then)
                  geGetSelSet()~>lpp = `("M2" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("M2" "drawing")) then )  
                  geGetSelSet()~>lpp = `("M3" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("M3" "drawing")) then )  
                  geGetSelSet()~>lpp = `("M4" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("M4" "drawing")) then )  
                  geGetSelSet()~>lpp = `("M5" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("M5" "drawing")) then )  
                  geGetSelSet()~>lpp = `("M6" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("M6" "drawing")) then )  
                  geGetSelSet()~>lpp = `("FA" "drawing")
            else
                if(((geGetSelSet()~>lpp) == `("FA" "drawing")) then )  
                  geGetSelSet()~>lpp = `("3A" "drawing")

            else ( geGetSelSet()~>lpp = `("LB" "drawing")

    )
    )
    )

  • Your parentheses are all messed up. Why have you got a close parenthesis after the "then"?

    if() is a function call - it needs to be:

    if(expression then statement1 statement2 ... statementN else elseStatement1 ... elseStatementN)

    (can be over multiple lines).

    Rather than writing nested if-then-else statements, this might be better as a cond:

    cond(
      (geGetSelSet()~>lpp=='("M1" "drawing")  geGetSelSet()~>lpp='("M2" "drawing"))
      (geGetSelSet()~>lpp=='("M2" "drawing")  geGetSelSet()~>lpp='("M3" "drawing"))
    ...
      (t  geGetSelSet()~>lpp='("LB" "drawing"))
    )

    Personally I'd probably write it a bit differently - probably:

    geGetSelSet()~>layerName=
      case(car(geGetSelSet())~>layerName
        ("M1" "M2")
        ("M2" "M3")
        ("M3" "M4")
       ...
        (t "LB")
      )

    This is not checking that the purpose is drawing, but you could always add a when() or if() around it to check that the purpose is "drawing" first if that matters. As you can see, the above is very concise and clear. I'm only checking the first layer, but your code would only work with a single shape anyway - in my case it will work for multiple shapes, but only look at the layer of the first shape to determine the layer to change to.

    SKILL is a functional programming language - so case, cond, if etc all have a return value that you can do something useful with, rather than always have to think in terms of "statements". In this example, each branch of the case is returning the layer you want to change to - which avoids lots of duplication of the code which sets the layer (or lpp).

    Regards,

    Andrew.

Community Guidelines

The Cadence Design Communities support Cadence users and technologists interacting to exchange ideas, news, technical information, and best practices to solve problems and get the most from Cadence technology. The community is open to everyone, and to provide the most value, we require participants to follow our Community Guidelines that facilitate a quality exchange of ideas and information. By accessing, contributing, using or downloading any materials from the site, you agree to be bound by the full Community Guidelines.