配列とループの組み合わせについて

Question

（↓）の配列があります

// 市区町村の数
let area = [39, 33, 54];

let city =[
'千代田区OK中央区OK港区OK新宿区OK文京区NG・・・',
'横浜市NG川崎市OK横須賀市OK・・・',
'千葉市OK銚子市NG市川市NG・・・'
];

cityの文字列があらかじめ配列であるとしてOKかNGかを検索して一致するかどうかをifで判定させ、それを上記の市区町村の数だけループさせるとした場合どのように組めばいいでしょうか？

自分なりに組んでみたこと。

let z = 0;
for (m = 0; m < city; m++) {
		if (city[m].indexOf(area[z]) !== -1) {

		} else {

		}
}

もしわかる方がいれば教えて頂けると恐縮です。
宜しくお願い致します。

質問への追記・修正

• areaは市区町村の数
• areaとcityは1：1で対応している
• cityは省略していますが要素1なら東京都の市区町村名とOKもしくはNGの文字列が格納されている

得たいプログラムの記述

cityの配列の要素1の場合,areaの要素1の数値分だけループ
→文字列のはじめから順番に検索していきOKと一致するとtrue、NGの場合はfalseをif文で記述。2回目は1回目に一致したその直後から再度検索していく
cityの配列の要素2の場合,areaの要素2の数値分だけループ
→文字列のはじめから順番に検索していきOKと一致するとtrue、NGの場合はfalseをif文で記述。2回目は1回目に一致したその直後から再度検索していく
cityの配列の要素3の場合,areaの要素3の数値分だけループ
→文字列のはじめから順番に検索していきOKと一致するとtrue、NGの場合はfalseをif文で記述。2回目は1回目に一致したその直後から再度検索していく

Comments

[Lhankor_Mhy]

let area = [39, 33, 54];
↑これはなにを表わしていますか？

[kei344]

cityの例をより実際に近い形にしてください。今の状態では「OK」で分割することになりますが、そういうデータではないですよね。

[kakeru99]

Lhankor_Mhyさん
let area = [39, 33, 54];
は市区町村の数になります。

[kakeru99]

kei344さん
スクレイピングして取得した配列がこのような配列になります。
そのためOKで分割で問題ないんです。
実際では東京の電車遅延があるかどうかの判定で
OK→平常運転
NG→列車遅延
という形のスクリプトを作りたいと思っています。
わかりにくくてすみませんm(_ _)m

[Lhankor_Mhy]

「OKかNGかを検索して一致するかどうか」とは、何と一致するかどうかを見るのですか？

[kei344]

let area = [39, 33, 54]; は「千代田区…」「横浜市…」「千葉市…」と1：1で対応している物ですか？1レコード目は東京で39市区町村、ということでしょうか。

[kakeru99]

cityの配列にある文字列をはじめから順番に検索していきOKと一致するとtrue、2回目は1回目に一致したその直後から再度検索していくといった流れのプラグラムです。
areaの配列はループの回数として利用する想定でした。

[kakeru99]

kei344さん
＞ let area = [39, 33, 54]; は「千代田区…」「横浜市…」「千葉市…」と1：1で対応している物ですか？1＞ レコード目は東京で39市区町村、ということでしょうか。
その通りです。

[Lhankor_Mhy]

NGと市町村名はすっとばしていい、ということですね？

[kakeru99]

OKとNGを条件分岐、市区町村名はすっとばして大丈夫です。

[kei344]

（質問文は編集できます）この「質問への追記・修正の依頼」の部分はデフォルトで表示されませんので、データの説明と得たいデータの最終形態（どういう配列か）を質問本文に追記することをお勧めします。

[kakeru99]

質問を修正してみました。わかりますでしょうか。

[kei344]

こちらの環境では変更／修正が確認できませんでした。すみませんが、もう一度確認してもらえませんか？最近teratailで修正が保存されない事がたまにあるようなので･･･。

[kakeru99]

kei344さん
すみません。きちんと送信できていなかったようで・・・。
次はきちんと更新できました。

[kei344]

編集ありがとうございます。例えば、「東京の電車遅延があるかどうかの判定」であれば1個でも「NG」が含まれていたら「遅延あり」になりますが、それだけの情報でよいのでしょうか。

[miyabi-sun]

> cityの配列の要素1の場合,areaの要素1の数値分だけループ
これは命令です。
これでは「お前は黙って俺の言うことに従っていればいいんだ」と怒鳴り散らす無能な上司です。
目的を書いてください。

Answer 1

難しいデータを難しいまま扱おうとして挫折するのは初心者によくある光景です。
簡単でわかりやすく、扱いやすいデータにしましょう。

let city =[
  '千代田区OK中央区OK港区OK新宿区OK文京区NG',
  '横浜市NG川崎市OK横須賀市OK',
  '千葉市OK銚子市NG市川市NG'
];

// 調査
for (let citiesName of city) {
  console.log(citiesName.split(/(OK|NG)/));
  // ["千代田区", "OK", "中央区", "OK", "港区", "OK", "新宿区", "OK", "文京区", "NG", ""]
  // ["横浜市", "NG", "川崎市", "OK", "横須賀市", "OK", ""]
  // ["千葉市", "OK", "銚子市", "NG", "市川市", "NG", ""]

  // 最後の空文字を削除する
  console.log(citiesName.split(/(OK|NG)/).slice(0, -1));
  // ["千代田区", "OK", "中央区", "OK", "港区", "OK", "新宿区", "OK", "文京区", "NG"]
  // ["横浜市", "NG", "川崎市", "OK", "横須賀市", "OK"]
  // ["千葉市", "OK", "銚子市", "NG", "市川市", "NG"]
}

const cityList = [];
for (let citiesName of city) {
  const result = {};
  const nameAndCondition = citiesName.split(/(OK|NG)/).slice(0, -1);
  for (let i = 0; i < nameAndCondition.length; i += 2) {
    const name = nameAndCondition[i];
    const condition = nameAndCondition[i + 1] === 'OK';
    // もしNGの都市を捨てるならこの行のコメントアウトを削る
    // if (!condition) continue;
    result[name] = condition;
  }
  cityList.push(result);
}
console.log(cityList);
// [
//   {千代田区: true, 中央区: true, 港区: true, 新宿区: true, 文京区: false},
//   {横浜市: false, 川崎市: true, 横須賀市: true},
//   {千葉市: true, 銚子市: false, 市川市: false}
// ]

このままではまだ質問者さんの使いたいデータに届いてないかも知れませんね。
ですが、最初のクソみたいな文字列からすれば大きな前進が出来たと思います。

このように人間が扱いやすいデータに作り変えてから使用する癖を付けましょう。

腕の良いエンジニアは難しい計算を難しい計算のまま行うのではなく、
子供でもわかる簡単な計算に作り変えてしまうから不具合が少ないのです。
この問題をいかに簡単に出来るのかが腕の見せどころなので、色んなデータを作り変えるというのを試してみて下さい。

Answer 2

元のコードを生かしてみました。

  let city =[
  '千代田区OK中央区OK港区OK新宿区OK文京区NG・・・',
  '横浜市NG川崎市OK横須賀市OK・・・',
  '千葉市OK銚子市NG市川市NG・・・'
  ];

  for (m = 0; m < city.length; m++) {
    var target = city[m].slice(0);
    while (true){
      var n;

      n = target.indexOf('OK');
      if ( n !== -1) {
        target = target.slice(n+2);
        console.log("OK", target);
        continue;
      }

      n = target.indexOf('NG');
      if ( n !== -1) {
        target = target.slice(n+2);
        console.log("NG", target);
        continue;
      }
      
      break;
    }
  }  

Answer 3

let city =[
'千代田区OK中央区OK港区OK新宿区OK文京区NG',
'横浜市NG川崎市OK横須賀市OK',
'千葉市OK銚子市NG市川市NG'
];
const oks = city
  .map(e=>e.split(/(OK|NG)/))
  .flatMap(e=> e.reduce( (ac,c,i,ar)=> i % 2 === 1 && c === "OK" ?  ac.concat(ar[i-1]) : ac,[] ))
console.log(oks.includes("中央区"));
console.log(oks.includes("文京区"));

---

「得たいプログラムの記述」をコード化しました。

ループのやりかたは書いてありましたが、どう結果を得たいのかわからなかったので、stackというのに判定したtrueだか、falseだかを入れてます。

let area = [39, 33, 54];

let city =[
'千代田区OK中央区OK港区OK新宿区OK文京区NG・・・',
'横浜市NG川崎市OK横須賀市OK・・・',
'千葉市OK銚子市NG市川市NG・・・'
];

let stack = [];
for(let areaIdx = 0; areaIdx < area.length; areaIdx++) {
  let boolarrayForArea = [];
  stack.push(boolarrayForArea);
  const str = city[areaIdx];
  let fromIdx = 0;
  let noPos = -1;
  let okPos = -1;
  for(let cityIter = 0; cityIter < area[areaIdx]; cityIter++) {
    ngPos = str.indexOf("NG", fromIdx) === -1 ? Number.MAX_VALUE : str.indexOf("NG", fromIdx) ;
    okPos = str.indexOf("OK", fromIdx) === -1 ? Number.MAX_VALUE : str.indexOf("OK", fromIdx) ;
    const nextPos = Math.min(ngPos, okPos);
    const nextWord = str.substr(nextPos,2);
    if (nextWord === "OK") { boolarrayForArea.push(true); }
    else if (nextWord === "NG") { boolarrayForArea.push(false); }
    else { boolarrayForArea.push(undefined); }
    fromIdx = nextPos + 2;
  }
}
console.log(stack);

Answer 4

回答ありがとうございました。
その後、miyabi-sunのソースを元に試行錯誤して下記のように動かすことができました。
質問するのも難しいですが勉強になりました。

<div class="area">
<ul>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
</ul>

<ul>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
</ul>

<ul>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
	<li>◎</li>
</ul>
</div>

let city = [
	'千代田区OK中央区OK港区OK新宿区OK文京区NG',
	'横浜市NG川崎市OK横須賀市OK',
	'千葉市OK銚子市NG市川市NG'
];

const cityList = [];
for (let citiesName of city) {
	let u = city.indexOf(citiesName);
	let u_num = ':nth-of-type(' + (u + 1) + ')';
	const nameAndCondition = citiesName.split(/(OK|NG)/).slice(0, -1);
	for (let i = 0; i < nameAndCondition.length; i += 2) {
		const name = nameAndCondition[i];
		const condition = nameAndCondition[i + 1];
		let num_box = ':nth-of-type(' + (i / 2 + 1) + ')';
		if (condition === 'OK') {} else {
			//OKと一致しない場合の処理
			$('.area ul' + u_num + ' li' + num_box).text("×");
		}
	}
}

Comments

[papinianus]

低評価の理由:質問者自身が指示した「得たいプログラム」でない