C# Add two curves lists into one.

Replies to This Discussion

Permalink Reply by Petras Vestartas on August 30, 2015 at 3:03pm

Also, when I try to add 2 curves list into 1. I still get two separate sub lists.

for (int i = 0; i < wireframeJoined.Length; i++) {
                insideOutside.Add(Inside[i]);
                insideOutside.Add(Outside[i]);
            }

Passing this list to output:

             for (int i = 0; i < glassFrames.Count; i++)    {
                DA.SetData(i+1, glassFrames[i]);
            }

Is it possible to flatten the last for loop to have  only one list  DA.SetData(i, glassFrames[i]) to DA.SetData(1, glassFrames[i]) ?

Permalink Reply by Daniel González Abalde on August 30, 2015 at 3:05pm

List<Curve> curves = new List<Curve>();
curves.AddRange(wireframeJoined);
curves.AddRange(wireframeJoinedOffset);

Hello Petras,
in your code I see strange things, for example, why use an array (wireframeJoined) if you really just going to get an item? or why you duplicate the curve (wireframe)?
I enclose the script in gh that does what you want. 

Attachments:

• FrameGlass.gh, 7 KB

Permalink Reply by Petras Vestartas on August 30, 2015 at 3:59pm

Thank you.

One last question about the output of the function.

When I pass brep list to output I get the following error with empyt list:

1. Invalid cast: Brep » List`1

//OUTPUT:

protected override void RegisterOutputParams(GH_Component.GH_OutputParamManager pManager) {

pManager.AddBrepParameter("FramesOutisde", "F", "Glass frames", GH_ParamAccess.list);

 }

//INPUT

        protected override void SolveInstance(IGH_DataAccess DA) {

            List <Brep> glassSrf = new List <Brep>();
            double frameWidth = -0.3;

            if (!DA.GetData(0, ref glassSrf)) return;

            List<Brep> glassFrames = CreateGlassFrames(glassSrf, frameWidth);

            DA.SetData(0, glassFrames);

//I do not know how to write this line correctly. Before I was trying to use for loop, but it did not help.
        }

Attachments:

• question2.png, 85 KB

Permalink Reply by Daniel González Abalde on August 30, 2015 at 4:35pm

It is possible at register the input parameter, you gave access for items, rather than list? If this is the problem, the fault was mine, that I did the function working with lists, rather than individual items. It should work well if you change to GH_ParamAccess.list in the input register, but as this component not binding on in any way the different elements, it should use item access on the param and use the function as follows :

private void RunScript(Brep B, double F, ref object A)
{

A = GlassFrames(B, F);

}

// <Custom additional code>
public Brep GlassFrames(Brep GlassSrf, double W)
{
Brep Frame = new Brep();
List<Curve> edges = GlassSrf.GetWireframe(-1).ToList(); //Edges.
Curve joinEdges = Curve.JoinCurves(edges)[0]; //Joined edges.
Curve offset = joinEdges.Offset(Plane.WorldXY, W, 0.01, CurveOffsetCornerStyle.Sharp)[0]; //Offset joined edges.
List<Curve> curves = new List<Curve>(); //list of curves.
curves.AddRange(edges);
curves.Add(offset);
Frame = Brep.CreatePlanarBreps(curves)[0]; //Create brep.
return Frame;
}

(Make the function to a single item, but when you enter more than one item, the script is executed once for each element). But first try changing the GH_ParamAccess.list on register input with the code you already have, and if it works, you can leave it. Feel free to ask whatever you want.

Permalink Reply by Petras Vestartas on August 30, 2015 at 4:54pm

Magic... Works. Thanks a lot.

I changed all lists to items in main functions, inputs and outputs. No there are no for loops and they work.

I have question regarding syntax:

In this and similar lines, what does the [0] stand for?

Curve joinEdges = Curve.JoinCurves(edges)[0]

Also do you know any tutorials regarding C# and RhinoCommon?

Kind Regards,

Petras

Permalink Reply by Daniel González Abalde on August 30, 2015 at 5:37pm

Cool. Actually, I'm starting to learn C#, I know VB and like they are very similar, I take these consultations to learn to write in C, but no, I do not know tutorials about C# + RhinoCommon.

Curve.JoinCurves (edges) returns an array of curves, because you can join several curves and can not join all of them. But in this case we know that they are going to join, then we collect the first element (and the unique) of the array [0] to reduce statements. It would be like:

Curve() joinEdges = Curve.JoinCurves(edges);

Curve joinededge = joinEdges[0];

but in a one line :]