Ttykuu

What is "gratricide"?

Is there hard evidence that the grant peer review system performs significantly better than random?

How come Sam didn't become Lord of Horn Hill?

How do I find out the mythology and history of my Fortress?

How to tell that you are a giant?

Converted a Scalar function to a TVF function for parallel execution-Still running in Serial mode

Central Vacuuming: Is it worth it, and how does it compare to normal vacuuming?

Significance of Cersei's obsession with elephants?

Is it possible for SQL statements to execute concurrently within a single session in SQL Server?

How were pictures turned from film to a big picture in a picture frame before digital scanning?

Why does it sometimes sound good to play a grace note as a lead in to a note in a melody?

SF book about people trapped in a series of worlds they imagine

A term for a woman complaining about things/begging in a cute/childish way

Trademark violation for app?

What do you call the main part of a joke?

Dating a Former Employee

An adverb for when you're not exaggerating

As a beginner, should I get a Squier Strat with a SSS config or a HSS?

How fail-safe is nr as stop bytes?

Why weren't discrete x86 CPUs ever used in game hardware?

How does light 'choose' between wave and particle behaviour?

How does the math work when buying airline miles?

Localisation of Category

How would a mousetrap for use in space work?

Ensure that a Sinatra hash always has a valid value

.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;

0

$begingroup$

I'm trying to ensure that a Sinatra hash always has a valid value.

Is there a more-concise way to write this?

params[:v] = if [:icons,:list].include? (params[:v] ||= :list).downcase.to_sym then (params[:v] ||= :list).downcase.to_sym else :list end

ruby sinatra

share|improve this question

asked Jun 26 '18 at 19:41

craigcraig

1085

$endgroup$

bumped to the homepage by Community♦ 16 mins ago

This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  I'm really unsure what your goal is here. Can you clarify?
  $endgroup$
  – thesecretmaster
  Jul 20 '18 at 2:59
  

  
  
  
  

add a comment | 

0

$begingroup$

I'm trying to ensure that a Sinatra hash always has a valid value.

Is there a more-concise way to write this?

params[:v] = if [:icons,:list].include? (params[:v] ||= :list).downcase.to_sym then (params[:v] ||= :list).downcase.to_sym else :list end

ruby sinatra

share|improve this question

asked Jun 26 '18 at 19:41

craigcraig

1085

$endgroup$

bumped to the homepage by Community♦ 16 mins ago

This question has answers that may be good or bad; the system has marked it active so that they can be reviewed.

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  I'm really unsure what your goal is here. Can you clarify?
  $endgroup$
  – thesecretmaster
  Jul 20 '18 at 2:59
  

  
  
  
  

add a comment | 

$begingroup$

I'm trying to ensure that a Sinatra hash always has a valid value.

Is there a more-concise way to write this?

params[:v] = if [:icons,:list].include? (params[:v] ||= :list).downcase.to_sym then (params[:v] ||= :list).downcase.to_sym else :list end

ruby sinatra

share|improve this question

asked Jun 26 '18 at 19:41

craigcraig

1085

$endgroup$

params[:v] = if [:icons,:list].include? (params[:v] ||= :list).downcase.to_sym then (params[:v] ||= :list).downcase.to_sym else :list end

share|improve this question

asked Jun 26 '18 at 19:41

craigcraig

1085

share|improve this question

asked Jun 26 '18 at 19:41

craigcraig

1085

asked Jun 26 '18 at 19:41

craigcraig

1085

asked Jun 26 '18 at 19:41

craigcraig

1085

1 Answer
1

active

oldest

votes

0

$begingroup$

I'd write this in multiple lines, so that you don't have to write out the complex conversion multiple times:

v = (params[:v] || :list).downcase.to_sym
params[:v] = if %i[icons list].include?(v) then v else :list end

share|improve this answer

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

$endgroup$

add a comment | 

Your Answer

StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");

StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "196"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');

var $window = $(window),
onScroll = function(e)
var $elem = $('.new-login-left'),
docViewTop = $window.scrollTop(),
docViewBottom = docViewTop + $window.height(),
elemTop = $elem.offset().top,
elemBottom = elemTop + $elem.height();
if ((docViewTop elemBottom))
StackExchange.using('gps', function() StackExchange.gps.track('embedded_signup_form.view', location: 'question_page' ); );
$window.unbind('scroll', onScroll);

;
$window.on('scroll', onScroll);

);

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f197306%2fensure-that-a-sinatra-hash-always-has-a-valid-value%23new-answer', 'question_page');

);

Post as a guest

Name

Email

Required, but never shown

0

$begingroup$

I'd write this in multiple lines, so that you don't have to write out the complex conversion multiple times:

v = (params[:v] || :list).downcase.to_sym
params[:v] = if %i[icons list].include?(v) then v else :list end

share|improve this answer

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

$endgroup$

add a comment | 

0

$begingroup$

I'd write this in multiple lines, so that you don't have to write out the complex conversion multiple times:

v = (params[:v] || :list).downcase.to_sym
params[:v] = if %i[icons list].include?(v) then v else :list end

share|improve this answer

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

$endgroup$

add a comment | 

$begingroup$

I'd write this in multiple lines, so that you don't have to write out the complex conversion multiple times:

v = (params[:v] || :list).downcase.to_sym
params[:v] = if %i[icons list].include?(v) then v else :list end

share|improve this answer

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

$endgroup$

v = (params[:v] || :list).downcase.to_sym
params[:v] = if %i[icons list].include?(v) then v else :list end

share|improve this answer

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011

answered Jul 22 '18 at 23:37

David MazeDavid Maze

1011