Using a switch with an array?

Question

So I think a switch might not be the most appropriate method of doing this, how else could I do it? Basically I want to break the string into single letters, then pass values for each letter to a new variable. What I have so far (with most letters taken out, so you can get the gist but I don't fill the screen):

$word = $_POST["word"];
$word_value = 0;
$word_array = str_split($word);

switch ($word_array){
    case "a":
    case "t":
        $word_value++;
        break;
    case "g":
    case "d":
        $word_value +2;
        break;
 }

Thanks in advance!

Use a letter value lookup table instead.

Ja͢ck
– Ja͢ck

2013-07-29 03:04:08 +00:00
Commented Jul 29, 2013 at 3:04 — Ja͢ck
– Ja͢ck, Commented Jul 29, 2013 at 3:04

Perocat · Accepted Answer · 2013-07-28 23:33:12Z

1

You insert the switch inside a foreach loop where you analyze every element of the array. $word_array is your array and $word is an element of the array.

$word = $_POST["word"];
$word_value = 0;
$word_array = str_split($word);

foreach($word_array as $letter){
    switch ($letter){
        case "a":
        case "t":
            $word_value++;
            break;
        case "g":
        case "d":
            $word_value+=2;
            break;
    }
}

edited Jul 28, 2013 at 23:33

answered Jul 28, 2013 at 23:27

Perocat

1,5217 gold badges25 silver badges50 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Neil Over a year ago

Shouldn't the $word in the switch case be $word_value? Right now, you'd just be increment, or adding to the letter itself.

Jeff N · Accepted Answer · 2013-07-28 23:45:36Z

1

Assuming that the values are constant, create a second array. This array holds the amount to increment for each letter. Now you don't need a switch statement, you simply index into the second array.

For example:

$letter_value["a"] = $letter_value["t"] = 1;
$letter_value["g"] = $letter_value["d"] = 2;

foreach ($word_array as $letter){
   $word_value += $letter_value[$letter];
}

answered Jul 28, 2013 at 23:45

Jeff N

3,4291 gold badge14 silver badges4 bronze badges

Comments

StackSlave · Accepted Answer · 2013-07-28 23:53:11Z

1

Try something like:

class Word{
  private $pn = 'word'; private $wv = 0;
  public function __construct($postName){
    $this->pn = $postName;
  }
  public function change_postName($postName){
    $this->pn = $postName;
  }
  public function value(){
    $wA = str_split($_POST[$this->pn]);
    foreach($wA as $w){
      switch($w){
        case 'a':
        case 't':
          $this->wv++;
          return $this->wv;
        case 'g':
        case 'd':
        $this->wv+=2;
          return $this->wv;
      }
    }
  }
}
$wrd = new Word('word'); $word_value1 = $wrd->value();

Now you can change where you get your information like:

$wrd->change_postName('other'); $word_value2 = $wrd->value();

and reset your word value to zero, like:

$nwrd = new Word('word');

edited Jul 28, 2013 at 23:53

answered Jul 28, 2013 at 23:31

StackSlave

10.6k2 gold badges20 silver badges35 bronze badges

Collectives™ on Stack Overflow

Using a switch with an array?

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related