FeatureScript: opFunction with boolean for optional parameters

jelte_steur814

Hi,

I'm working on a featurescript using opFitSpline functionality.

I always use the endDerivative and startDerivative, but have added a boolean option to check/uncheck in the UI for the intermediary derivatives.

my code now looks like this:

   // create a spline using the arrays
    
    if (definition.derivativeEndOnly)
    {
    opFitSpline(context, id + "fitSpline1", {
                "points" : pathPoints,
                "startDerivative" : startDerivative,
                "endDerivative" : endDerivative,
                "derivatives" : derivatives
            });
    }
    else
    {
              opFitSpline(context, id + "fitSpline1", {
                "points" : pathPoints,
                "startDerivative" : startDerivative,
                "endDerivative" : endDerivative,
            });  
    }

is there a cleaner way to disable the "derivatives" in one opFitSpline without having two different opFitSpline functions?

leaving the derivatives map empty hasn't worked so far.

MichaelPascoe

See how the definition of the opFitSpline is a map { }. You can create that map beforehand then add only what you need to it, then feed it into the opFitSpline:

var splineDef = {};
splineDef.points = pathPoints;
splineDef.startDerivative = startDerivative;
splineDef.endDerivative = endDerivative;

if (definition.derivativeEndOnly) splineDef.derivatives = derivatives;

opFitSpline(context, id + "fitSpline1", splineDef);

MichaelPascoe

See how the definition of the opFitSpline is a map { }. You can create that map beforehand then add only what you need to it, then feed it into the opFitSpline:

var splineDef = {};
splineDef.points = pathPoints;
splineDef.startDerivative = startDerivative;
splineDef.endDerivative = endDerivative;

if (definition.derivativeEndOnly) splineDef.derivatives = derivatives;

opFitSpline(context, id + "fitSpline1", splineDef);

jelte_steur814

Got it!

thanks Micheal!

var fitSplineMap = {

        "points" : pathPoints,

        "startDerivative" : startDerivative,

        "endDerivative" : endDerivative,

    };

    if (!definition.derivativeEndsOnly)

    {

        fitSplineMap = mergeMaps(fitSplineMap, { "derivatives" : derivatives });

    }

    opFitSpline(context, id + "fitSpline1", fitSplineMap);

jelte_steur814

ah yes,

splineDef.derivatives = derivatives

is cleaner than mergeMaps, but the idea is the same indeed