One-liners
-
@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.
-
@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
-
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.
-
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... -
You got it TIG. Could you suggest a way to rephrase the whole "challenge" so it is readily apparent?
-
Yes I was confused by the original wording you chose.
-
OBFUSCATORS!
Your goal should be to write the most readable code possible. This is the opposite.
-
ๅคงๅทฅใงใใใใใใใไฝๅนดๅๆฅๆฌใซไฝใใ ใใฉใๅคง้ชใซใใใใซใ็งใๅคงๅทฅใงใใใ้ข็ฝใใใชใใ
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 -
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
-
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 -
Why no semi-colons? Seems that we're doing one-statements instead of one-liners then.
-
@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) -
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?
-
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 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.
-
@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 -
@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]
-
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?
Advertisement