Solved

Javascript switch

Posted on 2009-07-12
2
326 Views
Last Modified: 2012-05-07
I have a callback function (onContentLoad) that is parsing some json.  I have a bunch of if/else statements that I'd like to turn into a swtich.
onContentLoad: function (json) {
    //Parse the returned JSON response
    if (json.result == 'success' && json.addremove == 'add') {
 
}
    else if (json.result == 'success' && json.addremove == 'remove') {
 
}
    else if (json.result == 'fail' && json.message == 'action-failed') {
 
}
    else if (json.result == 'fail' && json.message == 'number-exceeded') {
 
}
    else if (json.result == 'fail' && json.message == 'not-logged-in') {
 
}
}

Open in new window

0
Comment
Question by:aznprncritic
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
2 Comments
 
LVL 36

Accepted Solution

by:
Zyloch earned 250 total points
ID: 24835544
You cannot turn a condition in which you are testing two equalities into a single switch. What you can do is:

//Parse the returned JSON response
if (json.result == 'success')
{
    switch (json.addremove)
    {
    case 'add':
        // Code
        break;
    case 'remove':
        // Code
        break;
    }
}
else if (json.result == 'fail')
{
    switch (json.message)
    {
    case 'action-failed':
        // Code
        break;
    case 'number-exceeded':
        // Code
        break;
    case 'not-logged-in':
        // Code
        break;
    }
}

Open in new window

0
 
LVL 41

Expert Comment

by:HonorGod
ID: 24835560

if ( json. result == 'success' ) {
  if ( json.addremove == 'add' ) {
//  #1 - json.result == 'success' && json.addremove == 'add' 
  } else {
//  #2 - json.result == 'success' && json.addremove == 'remove' 
  }
} else {
  switch ( json.message ) {
    case 'action-failed' :
//    #3 json.result == 'fail' && json.message == 'action-failed'
      break;
    case  'number-exceeded' :
//    #4 json.result == 'fail' && json.message == 'number-exceeded'
      break;
    case 'not-logged-in' :
//    #5 json.result == 'fail' && json.message ==    'not-logged-in'
      break;
    default :
      alert( 'oops' );
  }
}

Open in new window

0

Featured Post

Is Your Team Achieving Their Full Potential?

74% of employees feel they are not achieving their full potential. With Linux Academy, not only will you strengthen your team's core competencies but also their knowledge of of the newest IT topics.

With new material every week, we'll make sure that you stay ahead of the game.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

There are a couple ways to attach a JavaScript function to dynamically created elements. You can make a new script for each element as it’s created or you can use delegation. Delegation allows a single script that is added at page creation to mat…
Introduction Chart.js, used properly, can visually add a difference to your charting applications. It engages your visitors and allows them to interact with data they otherwise wouldn't be able to without expensive and complicated systems. For this…
The viewer will learn how to dynamically set the form action using jQuery.
The viewer will learn the basics of jQuery including how to code hide show and toggles. Reference your jQuery libraries: (CODE) Include your new external js/jQuery file: (CODE) Write your first lines of code to setup your site for jQuery…

705 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question