for文の中でカウンタ変数以外の変数を動かしていて悩んでしまっています

for文の中での別変数の扱い

PHPを勉強中の初心者ですが、for文の中でカウンタ変数とは別の変数を動かすことがうまくできず、無理やり解決させたのですが、これでいいのか悩んでおります。

課題としては、7枚のくじを4人で引いていって、外れくじ（=x)を引いたらその人はいなくなるというルールで、くじを最後まで順番にひいたあと、残る人の番号をだすという内容です。最初はくじを引く順番で増える変数$iを人数で割った余りで誰が引いたかを見れると思っていたのですが、unsetでデータを削除したらそれができないことに気づきまして別の変数$jを入れることにしました。悩んだ結果、以下のようなソースになり、一応結果としては望む内容になったのですが、、、これは無駄が多いのでは、と感じております。

コードまで書いていただこうとは思ってないのですが、こういう場合にどう考えるのがよいか、教えていただけると助かります。

※実際に取り組んでいる課題としては全く別の内容です。こちらに掲載するために詰まった箇所に絞って内容を変えました。

該当のソースコード

PHP
1$members= [1,2,3,4];
2$cards = ["a","a","x","a","a","a","x"];
3$num_of_members = 4;
4$num_of_card = 7;
5
6$j=0;
7for($i=0; $i<$num_of_card; $i++){
8    if($cards[$i] == 'x'){
9        unset($members[$j]);
10        $members = array_values($members);
11        $num_of_members -= 1;
12        if($j == $num_of_members){
13            $j = 0;
14        }
15    
16    }else{
17            $j += 1;
18            if($j == $num_of_members){
19                $j = 0;
20            }
21        }
22}
23foreach ($members as $member){
24    echo $member."\n";
25}

試したこと

unsetで要素を削除して、array_valuesでキーを振りなおしたのがダメだったのか？と思って、やり方を変えてみたのですがうまくいきませんでした。

退会済みユーザー

2019/10/15 09:56

雰囲気が paiza なのですが、もし paiza であれば回答者にも迷惑がかかることを理解していますか？その他の競技プログラムであっても利用規約次第で回答者に迷惑がかかります。出典元を明らかにしてください。

yambejp

2019/10/15 10:02 編集

考え方： member:1=a・・・OK member:2=a・・・OK member:3=x・・・NG→$membersを["1","2","4"]とする？ member:4=a・・・OK 先頭にもどって member:1=a・・・OK member:2=a・・・OK member:4=x・・・NG→$membersを["1","2"]とする？ということでよいのでしょうか？結果としてほしいのはなんですか？

shun_nosuke

2019/10/15 10:06

ご指摘ありがとうございます。おっしゃる通り、課題のもとはPaizaさんです。悩んだ内容のもととなったのはこちらの問題です。 https://paiza.jp/works/mondai/skillcheck_archive/word_chain?language_uid=php こちらの問題ですが、レベルアップ問題集にあったもので、「レベルアップ問題集にある問題については、ユーザー同士で解答を教え合ったり、コードを公開したり、授業や研修で使用することは何の問題もありません。是非、ご活用ください。」とあったので、問題ないかと思っておりました。https://paiza.jp/works/mondai ご面倒をおかけして申し訳ありませんが、ご確認いただけますと幸いです。また、もしこの内容で問題があった場合は、お手数ですが削除頂けますと幸いです。「このくらい変えていれば大丈夫では？」と素人判断をしてしまったのですが、Paiza様だけでなく回答者様にも迷惑がかかることを理解できておりませんでした。今後はしっかり気を付けたいと思います。この度はご指摘いただきまして、有難うございます。

shun_nosuke

2019/10/15 10:13

te2ji様申し訳ありません。初めての投稿だったため、運営の方からのご連絡だと思ってしまっておりました。 yambejp様ご回答ありがとうございます。先の方にご指摘を頂いたのですが、この問題を掲載していいのか判断がつかないので、一旦解決とさせて頂こうと思います。

yambejp

2019/10/15 10:16

丸写しじゃなくてロジックを確認したいというだけならそんなに気にすることもないような気もしますが、なにかモヤモヤするなら、質問を閉じてしまってください

shun_nosuke

2019/10/15 10:19

yambejp様コメント有難うございます。初めての投稿でまだルールがわかっていなかったのと、他の方にご迷惑をおかけしたくないので、閉じさせていただきました。改めまして、ご回答ありがとうございました。

退会済みユーザー

2019/10/15 12:20

paiza のスキルチェックの問題であれば paiza 規約違反ですがそうでないのであれば質問に「問題のないこと」を記述して質問すると良いです。回答者が戸惑うことはなくなります。

行動規範の内容に同意します

回答3件

paiza の問題を解いてました。
で、回答をもとに質問用に加工してみました。

php
1<?php
2$members= [1,2,3,4];
3$cards = ["a","a","x","a","a","a","x"];
4
5$j=0;
6$x_falg = TRUE;
7foreach($cards as $k => $v){
8  if($x_falg === FALSE){
9    $j += 1;
10  }
11  if($j >= count($members)){
12    $j = 0;
13  }
14  if($x_falg === TRUE || $v !== 'x'){
15    $x_falg = FALSE;
16  } else {
17    array_splice($members,$j,1);
18    $x_falg = TRUE;
19  }
20}
21echo count($members).PHP_EOL;
22foreach($members as $v){
23  echo $v.PHP_EOL;
24}

$j の取り扱いがちょっと厄介かもしれません。
member を配列から削除したときはカウントを進めず、人数を超過したときは初期値に戻す操作が必要です。

投稿2019/10/15 12:42

退会済みユーザー

総合スコア0

shun_nosuke

2019/10/15 13:02

te2jiさん質問する際の注意点を教えてくれた上に、問題を確認していただき、しかも回答をアレンジまでしてくれて。。本当に有難うございます。驚いて声出ました。感動しました。頂いた書き方が、私の想定と全然違うので、じっくり読んで理解をしようと思います。心より感謝いたします。もし機会がまたありましたら、ご指導いただけますと嬉しいです。有難うございました！

退会済みユーザー

2019/10/15 13:11

tacsheaven さんのキューを使う方法はもっとシンプルに記述できそうです。チャレンジしてみると良いです。

行動規範の内容に同意します