回答編集履歴

質問文での仕様変更に伴う修正

2022/01/03 15:10

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -10,290 +10,200 @@
 ・１行読みとる
-・A列に空白でないものが現れた場合は、その値を項目１として保存
+・A列に空白でないものが現れた場合は、その値を項目１として保存。ブロックの開始行をstartという変数に保存
-・配列（下記コードでは「block」という変数）に、読み取った行データを蓄積していく
 ・次にA列に空白でないものが現れたとき
-->項目１の値と一緒に蓄積したblockを別関数に渡し、その別関数の中でステータス最大値を計算し、結果を返す
+->項目１の値と一緒にstartから現在行までのblockを別関数に渡し、その別関数の中でステータス最大値を計算し、結果を返す
-　その後項目１を切り替えて、blockを初期化、ループ最初に戻って次の行を読み取る
+　その後項目１を切り替えて、start行を更新、ループ最初に戻って次の行を読み取る
 という流れになります。
+※なお、下記のコードが正常に動く前提として
+・各行の確認日はすべて処理当日以前の日付であること
+・各項目において、同じ担当者で同じ確認日のデータは存在しない
-下記を実行すると、
+とします。
+```js
+// ステータス値を表すオブジェクト
+const status = { 'a': 1, 'b': 1, 'c': 3, 'd': 4, 'e': 5, 'f': 6 }
+const sheet = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
+// ステータスの最小値
+const MIN_STATUS_VALUE = Object.values(status).sort((a, b) => a - b)[0];
-各項目１のグループごとに、条件に当てはまるステータス最大値が
+// ステータスの最大値
+const MAX_STATUS_VALUE = Object.values(status).sort((a, b) => a - b).slice(-1)[0];
+function startFunc() {
+  // F列の最終行を取得
+  const lastRow = sheet.getRange(sheet.getMaxRows(), 6).getNextDataCell(SpreadsheetApp.Direction.UP).getRow();
+  // F列（連携日）のデータを取得
+  const renkeibiList = sheet.getRange(2, 6, lastRow - 1, 4).getValues();
+  // I列（項目１）をキー、Ｆ列（連携日）を値とする連想配列を作成。
+  const renkeibiTable = renkeibiList.reduce((acc, cur) => { acc[cur[3]] = cur[0]; return acc }, {})
+  // ステータス最大値を持つレコードを取得
+  const records = getMaxStatusRecords(renkeibiTable);
+  // 各項目１ごとのステータスの結果を書き込む。ステータス最大値の場合は、成約額を書き込む。
+  const writeValues = renkeibiList.map(e =>
+    [
+      records[e[3]]?.[2] ?? '',   // J列
+      status[records[e[3]]?.[2]] === MAX_STATUS_VALUE ? records[e[3]][4] ?? '' : ''  // K列
+    ]
+  );
+  sheet.getRange(2, 10, writeValues.length, writeValues[0].length).setValues(writeValues);
+}
+function getMaxStatusRecords(table) {
+  // 列Bの最終行を取得
+  const lastRow = sheet.getRange(sheet.getMaxRows(), 2).getNextDataCell(SpreadsheetApp.Direction.UP).getRow();
+  // シートからデータ部分を取得する。（最後の行の判定のため１行余分に取得）
+  const data = sheet.getRange(2, 1, lastRow, 5).getValues()
+  //　処理中のブロックの「項目１」を保存する変数
+  let currentKoumoku1 = '';
+  // 結果を格納する配列
+  const result = {};
+  // 各ブロックの最初の行
+  let start = 0;
+  for (let row = 0; row < data.length; row++) {
+    // 現在行のA列（項目１）を読み取る
+    const koumoku1 = data[row][0];
+    // 取得した行の「項目１」が空欄かどうか判定
+    if (koumoku1 !== '') {
+      // 現在保存中の「項目１」が空欄かどうか判定
+      if (currentKoumoku1 !== '') {
+        /* 取得した行の「項目１」が空欄ではない＆保存中の「項目１」が空欄ではない
+         ＝最初の行ではない＆項目１が切り替わったタイミング　ということなので、
+         ブロックを処理。*/
+        const statusValue = checkStatusValues3(data.slice(start, row), table[currentKoumoku1])
+        // 結果用連想配列に項目１をキーとしてステータス値を格納
+        result[currentKoumoku1] = statusValue;
+      }
+      //　切替後の新しい「項目１」をcurrentKoumoku1に代入し、ブロックのstartを現在行に更新する。
+      currentKoumoku1 = koumoku1;
+      start = row;
+    }
+  }
+  //　ループを抜けたので残っているブロックを処理する
+  const statusValue = checkStatusValues3(data.slice(start, lastRow - 1), table[currentKoumoku1]);
+  // 結果用連想配列に項目１をキーとしてステータス値を格納
+  result[currentKoumoku1] = statusValue;
+  return result;
+}
+function checkStatusValues3(block, renkeibi) {
+  // 確認日降順に並び替える。
+  return block.sort((a, b) => b[3].getTime() - a[3].getTime())
+    // 確認日が連携日より後のものを抽出
+    .filter(e => e[3] > renkeibi)
+    // 同じ担当者であれば確認日が処理当日に直近のものを抽出
+    .reduce((acc, cur) => acc.some(e => e[1] === cur[1]) ? acc : [...acc, cur], [])
+    // ステータスが最大のものを抽出
+    .reduce((a, b) => status[a[2]] > status[b[2]] ? a : b, MIN_STATUS_VALUE - 1);
+}
 ```
-[ { currentKoumoku1: 'A', statusValue: 1 },
-  { currentKoumoku1: 'B', statusValue: 1 },
-  { currentKoumoku1: 'C', statusValue: 1 } ]
-```
-のような形式でログに表示されます。
-（currentKoumoku1が各グループの項目１、statusValueがステータスの最大値）
-※なお、下記のコードが正常に動く前提として
-・各行の確認日はすべて処理当日以前の日付であること
-・ステータスに設定される値の最小値は−1より大きいこと
-とします。
-また、抽出条件は、質問文記載のものではなく、コメント後半での回答に記載されている文から読み取れる限りのものであることとします。
-すなわち
-「②確認日＞連携日である
- ③確認日が当日に一番近い
- ①ステータスが同じ項目内で最大である
-②->③->① の順番でふるいわけを行い、
-最終的に複数の異なる担当者が残った場合は、任意の１人を選べばよく、
-最終的に答えとして出すのはステータスだけでよく、誰のいつのステータスかまでは不要」
-ということとしています。
-・statusValueに−1が返ってくるときは条件にあてはまる行がないという意味です。
-・質問者さんの理解のしやすさを優先して、mapやfilter、reduceはあえて使用していません。
-```js
-// ステータス値を表すオブジェクト
-const status = { 'a': 1, 'b': 1, 'c': 3, 'd': 4, 'e': 5 }
-function main() {
-  const sheet = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
-  // シートからデータ部分を取得する。
-  const data = sheet.getDataRange().getValues();
-  // 連携日
-  const renkeibi = sheet.getRange("F2").getValue();
-  // ブロック（項目A,B,C...ごとに行データをまとめる箱）
-  let block = [];
-  //　処理中ブロックの「項目１」を保存する変数
-  let currentKoumoku1 = '';
-  // 結果を格納する配列
-  const result = [];
-  // row:行番号（2行目から読み取るので初期値は1）;
-  for (let row = 1; row < data.length; row++) {
-    // 現在の行データを１行読み取る
-    // A列（項目１）
-    const koumoku1 = data[row][0];
-    // 取得した行の「項目１」が空欄かどうか判定
-    if (koumoku1 !== '') {
-      // 現在保存中の「項目１」が空欄かどうか判定
-      if (currentKoumoku1 !== '') {
-        // 取得した行の「項目１」が空欄ではない＆保存中の「項目１」が空欄ではない
-        // ＝最初の行ではない＆項目１が切り替わったタイミング　ということなので、
-        // 蓄積したブロックを処理。
-        const statusValue = checkStatusValues(block, renkeibi)
-        // 結果用配列に項目１とステータス値を格納
-        result.push({ currentKoumoku1, statusValue });
-      }
-      //　切替後の新しい「項目１」をcurrentKoumoku1に代入し、ブロックを初期化する。
-      currentKoumoku1 = koumoku1;
-      block = [];
-    }
-    // ブロックに、現在の行データを追加。
-    block.push([currentKoumoku1, data[row][1], data[row][2], data[row][3]]);
-  }
-  //　ループを抜けたので残っているブロックを処理する
-  const statusValue = checkStatusValues(block, renkeibi);
-  // 結果用配列に項目１とステータス値を格納
-  result.push({ currentKoumoku1, statusValue });
-  //　結果を表示
-  console.log(result)
-}
-/**
- *  2次元配列から、条件にあてはまる行のステータス最大値を取り出す関数
- *  block : ２次元配列
- *  renkeibi　: 連携日
- *  戻り値：
- *    ステータスの最大値
- * 　 条件に当てはまるものがない場合：−1
- * **/
-function checkStatusValues(block, renkeibi) {
-  /**** 第１段階：確認日が連携日より後のものを抽出 ****/
-  const block1 = [];
-  for (const row of block) {
-    // 取り出した行の確認日が連携日より後であれば追加。
-    if (row[3] > renkeibi)
-      block1.push(row);
-  }
-  /****　第２段階：block1から、確認日が処理当日に一番近いものを抽出　****/
-  let block2 = [];
-  // 処理日当日
-  const now = new Date();
-  for (const row of block1) {
-    // 最初の１個目は配列に追加するだけ
-    if (block2.length === 0) {
-      block2.push(row);
-      continue;
-    }
-    // 取り出した要素の確認日と当日の差を、現時点の暫定最小値と比較する。
-    // 取り出した要素の確認日と当日の差(=A)
-    const a = now.getTime() - row[3].getTime();
-    // 現在のblock2の確認日と当日の差（暫定最小値）(=B)
-    const b = now.getTime() - block2[0][3].getTime();
-    if (a === b) {
-      // => A=Bであれば、block2に取り出した行を追加する
-      block2.push(row);
-    } else if (a < b) {
-      // => A<Bであれば、取り出した行をblock2とする。
-      block2 = [row];
-    }
-    // => A>Bであれば、何もしない。
-  }
-  /****　第３段階:block2 からステータスが最大のものを抽出　****/
-  let maxStatus = -1;
-  for (const row of block2) {
-    if (maxStatus < status[row[2]]) maxStatus = status[row[2]];
-  }
-  // 結果（ステータス最大値）を返す
-  return maxStatus;
-}
-```

2022/01/03 15:10

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -8,13 +8,17 @@
 詳細は下記のコードのmain()を参照いただきたいのですが、下記のように、
+・１行読みとる
-・１行ずつ読みとり、A列に空白でないものが現れたときに、その値を項目１として保存
+・A列に空白でないものが現れた場合は、その値を項目１として保存
-・A列が空白の間は、配列（下記では「block」）にデータを蓄積
+・配列（下記コードでは「block」という変数）に、読み取った行データを蓄積していく
-・次にA列に空白でないものが現れたときに、項目１の値と一緒に蓄積したblockを別関数に渡す。
+・次にA列に空白でないものが現れたとき
-・その別関数の中でステータス最大値を計算し、結果を返す
+->項目１の値と一緒に蓄積したblockを別関数に渡し、その別関数の中でステータス最大値を計算し、結果を返す
+　その後項目１を切り替えて、blockを初期化、ループ最初に戻って次の行を読み取る
 という流れになります。
@@ -202,7 +206,7 @@
  * **/
-function checkStatusValues2(block, renkeibi) {
+function checkStatusValues(block, renkeibi) {
   /**** 第１段階：確認日が連携日より後のものを抽出 ****/

修正

2021/12/31 05:59

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -1,63 +1,295 @@
+(コメントより）
+> 画像内の「項目1」のAとBをそれぞれ異なるものとして扱い集計をかけたい
+>　項目1がAの場合のステータス比較と、項目1がBの場合のステータス比較は、それぞれ独立しているものとして扱いたいのですが、項目1がAから次のBにいくまでに、画像のA列が空白になっているので、区別して扱うにはどのようにすれば良いか
+詳細は下記のコードのmain()を参照いただきたいのですが、下記のように、
+・１行ずつ読みとり、A列に空白でないものが現れたときに、その値を項目１として保存
-★の基本的な考え方の所ですが、下記のようにやれば効率的だと思います。
+・A列が空白の間は、配列（下記では「block」）にデータを蓄積
+・次にA列に空白でないものが現れたときに、項目１の値と一緒に蓄積したblockを別関数に渡す。
+・その別関数の中でステータス最大値を計算し、結果を返す
-以下の2段階の操作を行います。
+という流れになります。
+下記を実行すると、
-（※前提として、各項目ごとのデータについて条件に当てはまるものを求めるのですから、
+各項目１のグループごとに、条件に当てはまるステータス最大値が
-**各項目ごとのデータについて**、以下の操作を行う必要があることに注意してください。）
-```text
-０．　最初の１行を配列に格納する。これを以下「仮MAXの配列」と称する。
-１．　次の行があるか調べ、なければ終了。あれば次の１行を取り出す。
-２．　取り出した行の確認日が連携日より後かどうか比較。
-３．　２がyes（確認日＞連携日）であれば、４に行く。noであれば１に戻る。
-４．　取り出した行のステータス値を現在の仮MAXのステータス値と比較する。
-　　　　=>ステータス値が仮MAX値と同じであれば、仮MAXの配列に現在行を追加する
-　　　　=>ステータス値が仮MAX値より大きければ、現在の行を仮MAXとする。
-　　　　=>ステータス値が仮MAX値より小さければ、何もしない。
-５.　１に戻る
 ```
+[ { currentKoumoku1: 'A', statusValue: 1 },
+  { currentKoumoku1: 'B', statusValue: 1 },
+  { currentKoumoku1: 'C', statusValue: 1 } ]
+```
+のような形式でログに表示されます。
+（currentKoumoku1が各グループの項目１、statusValueがステータスの最大値）
+※なお、下記のコードが正常に動く前提として
+・各行の確認日はすべて処理当日以前の日付であること
+・ステータスに設定される値の最小値は−1より大きいこと
+とします。
+また、抽出条件は、質問文記載のものではなく、コメント後半での回答に記載されている文から読み取れる限りのものであることとします。
+すなわち
+「②確認日＞連携日である
+ ③確認日が当日に一番近い
+ ①ステータスが同じ項目内で最大である
+②->③->① の順番でふるいわけを行い、
-これにより最終的に残った「仮MAXの配列」に対して、以下の操作を行います。
+最終的に複数の異なる担当者が残った場合は、任意の１人を選べばよく、
+最終的に答えとして出すのはステータスだけでよく、誰のいつのステータスかまでは不要」
+ということとしています。
+・statusValueに−1が返ってくるときは条件にあてはまる行がないという意味です。
+・質問者さんの理解のしやすさを優先して、mapやfilter、reduceはあえて使用していません。
+```js
+// ステータス値を表すオブジェクト
+const status = { 'a': 1, 'b': 1, 'c': 3, 'd': 4, 'e': 5 }
+function main() {
+  const sheet = SpreadsheetApp.getActiveSpreadsheet().getActiveSheet();
+  // シートからデータ部分を取得する。
+  const data = sheet.getDataRange().getValues();
+  // 連携日
+  const renkeibi = sheet.getRange("F2").getValue();
+  // ブロック（項目A,B,C...ごとに行データをまとめる箱）
+  let block = [];
+  //　処理中ブロックの「項目１」を保存する変数
+  let currentKoumoku1 = '';
+  // 結果を格納する配列
+  const result = [];
+  // row:行番号（2行目から読み取るので初期値は1）;
+  for (let row = 1; row < data.length; row++) {
+    // 現在の行データを１行読み取る
+    // A列（項目１）
+    const koumoku1 = data[row][0];
+    // 取得した行の「項目１」が空欄かどうか判定
+    if (koumoku1 !== '') {
+      // 現在保存中の「項目１」が空欄かどうか判定
+      if (currentKoumoku1 !== '') {
+        // 取得した行の「項目１」が空欄ではない＆保存中の「項目１」が空欄ではない
+        // ＝最初の行ではない＆項目１が切り替わったタイミング　ということなので、
+        // 蓄積したブロックを処理。
+        const statusValue = checkStatusValues(block, renkeibi)
+        // 結果用配列に項目１とステータス値を格納
+        result.push({ currentKoumoku1, statusValue });
+      }
+      //　切替後の新しい「項目１」をcurrentKoumoku1に代入し、ブロックを初期化する。
+      currentKoumoku1 = koumoku1;
+      block = [];
+    }
+    // ブロックに、現在の行データを追加。
+    block.push([currentKoumoku1, data[row][1], data[row][2], data[row][3]]);
+  }
+  //　ループを抜けたので残っているブロックを処理する
+  const statusValue = checkStatusValues(block, renkeibi);
+  // 結果用配列に項目１とステータス値を格納
+  result.push({ currentKoumoku1, statusValue });
+  //　結果を表示
+  console.log(result)
+}
+/**
+ *  2次元配列から、条件にあてはまる行のステータス最大値を取り出す関数
+ *  block : ２次元配列
+ *  renkeibi　: 連携日
+ *  戻り値：
+ *    ステータスの最大値
+ * 　 条件に当てはまるものがない場合：−1
+ * **/
+function checkStatusValues2(block, renkeibi) {
+  /**** 第１段階：確認日が連携日より後のものを抽出 ****/
+  const block1 = [];
+  for (const row of block) {
+    // 取り出した行の確認日が連携日より後であれば追加。
+    if (row[3] > renkeibi)
+      block1.push(row);
+  }
+  /****　第２段階：block1から、確認日が処理当日に一番近いものを抽出　****/
+  let block2 = [];
+  // 処理日当日
+  const now = new Date();
+  for (const row of block1) {
+    // 最初の１個目は配列に追加するだけ
+    if (block2.length === 0) {
+      block2.push(row);
+      continue;
+    }
+    // 取り出した要素の確認日と当日の差を、現時点の暫定最小値と比較する。
+    // 取り出した要素の確認日と当日の差(=A)
+    const a = now.getTime() - row[3].getTime();
+    // 現在のblock2の確認日と当日の差（暫定最小値）(=B)
+    const b = now.getTime() - block2[0][3].getTime();
+    if (a === b) {
+      // => A=Bであれば、block2に取り出した行を追加する
+      block2.push(row);
+    } else if (a < b) {
+      // => A<Bであれば、取り出した行をblock2とする。
+      block2 = [row];
+    }
+    // => A>Bであれば、何もしない。
+  }
+  /****　第３段階:block2 からステータスが最大のものを抽出　****/
+  let maxStatus = -1;
+  for (const row of block2) {
+    if (maxStatus < status[row[2]]) maxStatus = status[row[2]];
+  }
+  // 結果（ステータス最大値）を返す
+  return maxStatus;
+}
 ```
-０．　「仮MAXの配列」の最初の要素（行）を配列に格納する。これを以下「仮MINの配列」と称する。
-１．　次の行があるか調べ、なければ終了。あれば次の１行を取り出す。
-２．　取り出した要素の確認日と当日の差(=A)を、仮MINの確認日と当日の差(=B)と比較する。
-　　　　=> A=Bであれば、仮MINの配列に現在行を追加する
-　　　　=> A<Bであれば、現在の行を仮MINとする。
-　　　　=> A>Bであれば、何もしない。
-３.　１に戻る
-```
-これにより最終的に残った「仮MINの配列」が求める答えになります。