One-liners

noelwarr

@dan rathbun said:

The each iterator returns the receiver (upon which it acts,) not the result of the block.

Try using one of the iterators from the Enumerable mixin module (which is mixed into most of the API collection classes.)

I might as well give it to you:

find_faces_of_selected_entities = Sketchup.active_model.selection.find_all {|ent| ent.is_a?(Sketchup::Face)}

I was unaware of the find_all method. You live and learn. The one-liner you propose, however, does nothing. It would be useful in the context of more code. I think the problem is the variable name I chose. It is misleading. Here is the original code I posted

find_faces_of_selected_entities = Sketchup::active_model.selection.each{|ent| ent.find_faces if ent.is_a?(Sketchup::Edge)}

I say find_faces_of_selected_entities because the Edge class has the find_faces() method. I think both should be renamed to reconstruct_faces or something. In the hope everything is more clear we'll get rid of the variable alltogether. I'm changing my first post to...

#Reconstruct all possible faces from selection
Sketchup::active_model.selection.each{|ent| ent.find_faces if ent.is_a?(Sketchup::Edge)}

Try it out. Draw something, delete a few faces, select everything and run the one-liner.

Dan Rathbun

@noelwarr said:

Try it out. Draw something, delete a few faces, select everything and run the one-liner.

YOUR CODE DOES NOT WORK ! .. as I had said.

It returns nil

noelwarr

What it returns is irrelevant. It's what it does that matters, Dan. Please see attatched video for proof (Sorry its in wmv). There was a moment of fear when the code did nothing. Then I realised I had nothing selected

I know it is bothersome when people post code that doesn't work. I'm sory if I didn't explain myself correctly giving the impression that that was the case. It is also, however bothersome when people say
@dan rathbun said:

YOUR CODE DOES NOT WORK !
without considering the possibility that it might.

Rest assured I shall be checking the oneliners people post before I include it in my first post.

oneliner.wmv

TIG

It does work, in that it adds faces to selected edges.
For me it works, then returns the selection, not the new faces or nil.
I think the initial confusion was that the code had something like "add_faces_to_selected_lines=..." to show what it did, and of course add_faces_to_selected_lines is set to be the selection, I assume this was used because of the arbitrary rule that you can't use a ';' - which makes it impossible to do a one-liner with its intent shown unless you add '#' and then the description at the end of the code line...

noelwarr

You got it TIG. Could you suggest a way to rephrase the whole "challenge" so it is readily apparent?

Dan Rathbun

Yes I was confused by the original wording you chose.

daiku

OBFUSCATORS!

Your goal should be to write the most readable code possible. This is the opposite.

noelwarr

大工ですか。すごい。何年前日本に住んだけど。大阪に。それに、私も大工ですよ。面白い、なぁ。
Anyhow, I'm not advocating for this type of code, though I think it's cool that ruby allows for it. All I wanted was a list of cool lines that one might copy and paste into the console to do something useful. Instead it would seem I've started a flame war or something

daiku

Sorry, but I don't speak a lick of Japanese. I can tell from the first two characters that this was addressed to me, but the rest is lost on me. As a professional timber framer, the Daiku are my heroes. We had a group come to one of our conferences, and the level of skill and craftsmanship was amazing.

And of course I was only trying to be clever with my first post. Nothing personal

noelwarr

No offence was taken! The japanese just says I'm a carpenter too and lived (a short period of time) in Osaka a few years back. I was also not left unmoved by Japanese craftmanship when it comes to woodworking. We even ended up selling one of their products a few years back, here in Europe. http://mitaka.eu/
It's still the traditional side that gets to the heart

thomthom

Why no semi-colons? Seems that we're doing one-statements instead of one-liners then.

jeff hammond

@thomthom said:

Why no semi-colons?

meh..

view = Sketchup.active_model.active_view ; view = view.zoom 1.052

parallel projection -> zoom extents -> use the code for true zoom extents…
i use it sometimes for printing.
(also part of an applescript i use for easy printing to scale from sketchup)

noelwarr

Why not...

Sketchup.active_model.active_view.zoom(1.052)

Or to include the zoom extents...

Sketchup.active_model.active_view.zoom_extents.zoom(1.052)

Don't know how to activate the parallel projection though. Anyone?

Dan Rathbun

Activate Parallel Projection for the current View's Camera:

@noelwarr said:

Don't know how to activate the parallel projection though. Anyone?

Sketchup.active_model.active_view.camera.perspective = false

thomthom

@unknownuser said:

view.zoom 1.052

Where does this magic number come from?

Dan Rathbun

@thomthom said:

@unknownuser said:

view.zoom 1.052

Where does this magic number come from?

.. and it seems to work in Parallel Projection mode, but not in Perspective mode.

jeff hammond

@noelwarr said:

Why not...

Sketchup.active_model.active_view.zoom(1.052)

Because I don't know ruby
I just found something that worked for what I needed.
but your version doesn't use a semicolon so I'll make that my thread entry

jeff hammond

@thomthom said:

@unknownuser said:

view.zoom 1.052

Where does this magic number come from?

by trying .05... then .06... then .055.. then .054.. etc

[edit-- hmm.. just checked my AppleScript and I'm actually using .053 in it.. that's the one I tested with actual paper/printer]

noelwarr

I've actually gone and changed my first post so that semi colons are allowed, and have modified your magic formula so the 3 steps are all in one line. I wanted no semicolons to avoid people going crazy with them (you could essentially write a whole program with them in one line of text). How about 3 semi colons maximum?