We Keep Coding

Function with two switch cases will only log one string

Im new to coding and javascript, atm im doing som tests in school.
I have this function with two different switch cases with strings which i want to add together and return. But it only returns one of the strings. If i use switch(card.value) on the first and switch(card.suit) on the second it only returns the first one. But if i take it away on the frist one:
switch(value) and switch(card.suit) it return string from the lower switch-case. Why is that? And how do i get it to return A♥? Here is the code. Sorry for my messy description.
const prettyCard = function (card) {
let suit, value
switch (card.value) {
case 1:
return 'A';
break;
case 10:
return 'T';
break;
case 11:
return 'J';
break;
case 12:
return 'Q';
break;
case 13:
return 'K';
break;
case 2:
return '2';
break;
case 3:
return '3';
break;
case 4:
return '4';
break;
case 5:
return '5';
break;
case 6:
return '6';
break;
case 7:
return '7';
break;
case 8:
return '8';
break;
case 9:
return '9';
break;
}
switch (card.suit) {
case 'HEARTS':
return '♥';
break;
case 'SPADES':
return '♠';
break;
case 'DIAMONDS':
return '♦';
break;
case 'CLUBS':
return '♣';
break;
}
return value + suit
}
console.log(prettyCard({ suit: 'HEARTS', value: 1 }))

In addition to the fix suggested in comments, the code can shrink considerably by looking-up suits and values in an object, rather than a long switch.
const prettyCard = card => {
const values = { 1: 'A', 11: 'J', 12: 'Q', 13: 'K' }
const suits = { 'HEARTS': '♥', 'SPADES': '♠', 'DIAMONDS': '♦', 'CLUBS': '♣' };
const suit = suits[card.suit];
const value = card.value > 1 && card.value < 11 ? `${card.value}` : values[card.value];
return { suit, value };
}
console.log(prettyCard({ value: 12, suit: 'HEARTS' }))

How do i make switch and if statement in one function

function texas(val) {
var answer = "";
switch(val) {
case 1:
case 2:
case 3:
var answer = "low";
case 4:
case 5:
case 6:
var answer = "medium";
break;
} else if(val => 7) {
var answer = "Huge"
}
return answer;
}
it says error Declaration or statement expected. ts(1128) [13, 7]
and it poits at the else if statement

You can use the "default" keyword, but you should probably update your code in order to handle the cases in which the value of the parameter is not positive or not a number:
function texas(val) {
if (val <= 0 || isNan(val)) {
throw new InvalidOperationException("val should be a positive number");
}
switch(val) {
case 1:
case 2:
case 3:
return "low";
case 4:
case 5:
case 6:
return "medium";
default:
return "Huge"
}
}

It's >= and the elsehas to be deleted. The varfor answer is unnecesary, just declare it once with let. You forgot the break in case 3:.
function texas(val) {
let answer = "";
switch(val) {
case 1:
case 2:
case 3:
answer = "low";
break;
case 4:
case 5:
case 6:
answer = "medium";
break;
}
if(val >= 7) {
answer = "Huge"
}
return answer;
}
console.log(texas(2));
console.log(texas(8));

You just need to return in the switch
function texas(val) {
var answer = "";
switch(val) {
case 1:
case 2:
case 3:
var answer = "low";
return answer;
case 4:
case 5:
case 6:
var answer = "medium";
return answer;
}
if(val => 7) {
var answer = "Huge"
}
return answer;
}

The syntax does not allow to put an else after a switch. else only makes sense in combination with an if statemen. But switch has a default: case which most closely matches your intention (hopefully):
function texas(val) {
var answer = "";
switch(val) {
case 1:
case 2:
case 3:
var answer = "low";
break;
case 4:
case 5:
case 6:
answer = "medium";
break;
default:
if(val >= 7) {
answer = "Huge"
}
// decide what should happen if val is 0, -1 or not even a number (e.g. texas('gotcha!')
break;
}
return answer;
}
Don't forget to put break in your cases, otherwise execution will "fall through" and execute the next cases. You would never end up with "low"

You can't use an if statement within a switch block.
You do have the default option tho -
function texas(val) {
var answer = "";
switch(val) {
case 1:
case 2:
case 3:
answer = "low";
case 4:
case 5:
case 6:
answer = "medium";
break;
default:
answer = val >= 7 ? "Huge" : "Invalid";
break;
}
return answer;
Note that if you have a minus / negative answer, it'll also fall into this clause, but you can the the value of answer with an inline ?: if statement...

You can't put the else after the switch block as people have stated above. switch statement is better for multi way branching and fixed data values. On the other side, if statement is better for boolean values. You can do something like this. It might not be the shortest line of codes, but just so you that there's another approach:
function texas(val) {
let answer = "";
switch (true) {
case (val == 1 || val == 2 || val == 3):
answer = "low";
break;
case (val == 4 || val == 5 || val == 6):
answer = "medium";
break;
case (val >= 7):
answer = "huge";
break;
}
return answer;
}

Why can I not view an updated count for a variable with console.log in Javascript?

I currently have this code below, it is supposed to emulate a card counting system, in which different characters increment/decrement the count when passed. The code below runs successfully, but when I attempt console.log(cc(2,3,4,5,6); it returns 1 Bet, when I know the code runs 5 Bet, as those characters should increment the count, yet console.log does not return the accurate count, I assume this is due to scope? But I would like if someone could explain why count isn't accurately returned.
var count = 0;
function cc(card) {
switch(card) {
case 2:
case 3:
case 4:
case 5:
case 6:
count += 1;
break;
case 7:
case 8:
case 9:
count += 0;
break;
case 10:
case 'J':
case 'Q':
case 'K':
case 'A':
count -= 1;
break;
}
if (count > 0) {
return count + " Bet";
}
else {
return count + " Hold";
}
}
cc(2,3,4,5,6); \\ returns 5 Bet
console.log(cc(2,3,4,5,6)); \\ returns 2 Bet

var count = 0;
const bet = card => {
switch (card) {
case 2:
case 3:
case 4:
case 5:
case 6:
count += 1;
break;
case 7:
case 8:
case 9:
count += 0;
break;
case 10:
case "J":
case "Q":
case "K":
case "A":
count -= 1;
break;
}
};
function cc(...moves) {
moves.forEach(bet);
if (count > 0) {
return count + " Bet";
} else {
return count + " Hold";
}
}
console.log(cc(2, 3, 4, 5, 6)); // returns 5 Bet
console.log(cc(2, 3, 4, 5, 6)); // returns 10 Bet

Use arguments or splat operator to get all arguments passed and then loop it:
function cc(...cards) {
var count = 0;
var card;
for (var i = 0; i < cards.length; i++) {
card = cards[i];
switch (card) {
case 2:
case 3:
case 4:
case 5:
case 6:
count += 1;
break;
case 7:
case 8:
case 9:
count += 0;
break;
case 10:
case 'J':
case 'Q':
case 'K':
case 'A':
count -= 1;
break;
}
}
if (count > 0) {
return count + " Bet";
} else {
return count + " Hold";
}
}
cc(2, 3, 4, 5, 6); // returns 5 Bet
console.log(cc(2, 3, 4, 5, 6)); // returns 2 Bet

You have two major problems.
First, your count need to start at zero every time you call cc(). Because you have declared count outside the function, its values is preserved between calls to cc(). By declaring it inside the function, it's initialized to zero before it starts counting the card values.
(Unless you want to continue to the previous count, in which case you should keep it declared outside the function)
Second, your function only accepts one argument. You need to make it accept a list of arguments. This can be done simply with the spread operator .... Then you need to loop through each card value and do the count.
function cc(...cards) { // accept a list of card values as arguments.
let count = 0
for (const card of cards) { // loop through all card values.
switch (card) {
case 2:
case 3:
case 4:
case 5:
case 6:
count += 1
break
case 7:
case 8:
case 9:
count += 0
break
case 10:
case 'J':
case 'Q':
case 'K':
case 'A':
count -= 1
break
}
}
if (count > 0) {
return `${count} Bet`
}
return `${count} Hold`
}
cc(2, 3, 4, 5, 6) // returns 5 Bet
console.log(cc(2, 3, 4, 5, 6)) // returns 5 Bet
console.log(cc(2, 3, 4, 5, 6)) // returns 5 Bet

switch case of greater than or less than variable [duplicate]

This question already has answers here:
Expression inside switch case statement
(8 answers)
Closed 3 years ago.
I use this code to enter a number and compare it using less and greater than within switch case how should I do to get the correct result, only default can works
var sum=prompt("enter sum:");
// sum=50;
switch(sum)
{
case sum=0:
alert("sucess");
break;
case sum>50:
alert("also sucess");
break;
case sum<50:
alert("failed");
default:
alert("there is errorrrr");
}

You can use switch (true):
switch (true) {
case sum === 0:
alert('success');
break;
case sum < 50:
alert('also success');
break;
case sum > 50:
alert('failed');
break;
default:
alert('there is an error.')
break;
}
Note that in your code, the first case is actually an assignment and modify sum to set it to 0.

It actually doesn't work, as you expect, the switch statement is compared against all cases, such as :
switch (something)
{
case 1: // something == 1 ?
// ....
}
Actually, what you have write was interpreted such as
var sum = 42;
switch(sum)
{
case sum < 50: // sum == sum < 50 ? -> 42 == 42 < 50 ? -> 42 == true ? false !
// ...
Instead, you can use a switch true statement.
// v------- convert the prompt to a numeric value
let sum = + prompt("enter sum:");
switch(true)
{
// VV----- notice the double equal
case sum == 0: // true == sum == 0 ?
alert("sucess");
break;
case sum > 50:
alert("also sucess");
break;
case sum < 50:
alert("failed");
break; // <---- You forgot a break; there
default:
alert("there is errorrrr");
break;
}

ur idea works fine
sum=20;
switch(true)
{
case 50:
alert("sucess");
break;
case (sum>50):
alert("also sucess");
break;
case sum<50:
alert("failed");
break;
default:
alert("there is errorrrr");
}

Counting Cards Code Explanation

I am doing the FreeCodeCamp courses and am confused as to why my code doesn't work. It seems to be a messier version of the correct answer. Below is the correct answer and underneath that is mine. I just want to understand.
Here is the correct answer:
var count = 0;
function cc(card) {
{
case 2:
case 3:
case 4:
case 5:
case 6:
return count++;
case 7:
case 8:
case 9:
return count;
case 10:
case 'J':
case 'Q':
case 'K':
case 'A':
return count--;
}
}
Here is my answer:
var count = 0;
function cc(card) {
// Only change code below this line
switch (card) {
case 2:
return count + 1;
case 3:
return count + 1;
case 4:
return count + 1;
case 5:
return count + 1;
case 6:
return count + 1;
case 7:
return count + 0;
case 8:
return count + 0;
case 9:
return count + 0;
case 10:
return count - 1;
case 'J':
return count - 1;
case 'Q':
return count - 1;
case 'K':
return count - 1;
case 'A':
return count - 1;
}
if (count <= 0){
return count + " hold";
} else {
return count + " bet";
}

Two differences; what you get back and what is modified
count++ and count-- give you back the value of count, then modify count
count + 1 and count - 1 give you back a calculated value based on count and do not modify count
Consider
let x = 1;
x++; // 1
x; // 2
vs
let x = 1;
x + 1; // 2
x; // 1
It is also worth noting that the unary increment and decrement operators can be used prefix, which performs the modification before you get back the value, i.e.
let x = 1;
++x; // 2
x; // 2
Furthermore, please notice that it is impossible to reach your condition if (count <= 0) ... assuming card in [2-9JQKA]|10 as an earlier return will always have been called.
Finally, as I can't see the input I can not be sure on this, however it looks like this code is mixing Number and String. It may be wise to consider choosing just one type so you do not get unexpected behaviour.
Depending on the behaviour you require for A, I would suggest either using Number and mapping {J: 11, Q: 12, K: 13, A: 14}, or doing everything in String.

var count = 0;
function cc(card) {
switch(card) {
case 2:
case 3:
case 4:
case 5:
case 6:
count += 1;
break;
case 7:
case 8:
case 9:
count += 0;
break;
case 10:
case 'J':
case 'Q':
case 'K':
case 'A':
count -= 1;
break;
}
return count + (count > 0 ? " Bet" : " Hold");
}