slack での退勤管理「みやもとさん」のカスタマイズ

Question

退勤管理をslackで管理しようと考えています。
そこで「みやもとさん」を導入することになりました。
ですが、このアプリでは、1日に1度の出勤にしか対応しておりません。
デフォルトでは

このように出勤、退勤が管理されています。

このように横に1日何度かの出勤退勤を全て保存できるようにしたいと思っていますが、javascript初心者の私には難しいです。
どなたかご教授をお願い致します。

変更点はこのコードだと考えています...

loadGSTimesheets = function () {
  var GSTimesheets = function(spreadsheet, settings) {
    this.spreadsheet = spreadsheet;
    this.settings = settings;
    this._sheets = {};

    this.scheme = {
      columns: [
        { name: '日付' },
        { name: '出勤' },
        { name: '退勤' },
        { name: 'ノート' },
      ],
      properties: [
        { name: 'DayOff', value: '土,日', comment: '← 月,火,水みたいに入力してください。アカウント停止のためには「全部」と入れてください。'},
      ]
    };
  };

  GSTimesheets.prototype._getSheet = function(username) {
    if(this._sheets[username]) return this._sheets[username];

    var sheet = this.spreadsheet.getSheetByName(username);
    if(!sheet) {
      sheet = this.spreadsheet.insertSheet(username);
      if(!sheet) {
        throw "エラー: "+sheetName+"のシートが作れませんでした";
      }
      else {
        // 中身が無い場合は新規作成
        if(sheet.getLastRow() == 0) {
          // 設定部の書き出し
          var properties = [["Properties count", this.scheme.properties.length, null]];
          this.scheme.properties.forEach(function(s) {
            properties.push([s.name, s.value, s.comment]);
          });
          sheet.getRange("A1:C"+(properties.length)).setValues(properties);

          // ヘッダの書き出し
          var rowNo = properties.length + 2;
          var cols = this.scheme.columns.map(function(c) { return c.name; });
          sheet.getRange("A"+rowNo+":"+String.fromCharCode(65 + cols.length - 1)+rowNo).setValues([cols]);
        }
        //this.on("newUser", username);
      }
    }

    this._sheets[username] = sheet;

    return sheet;
  };

  GSTimesheets.prototype._getRowNo = function(username, date) {
    if(!date) date = DateUtils.now();
    var rowNo = this.scheme.properties.length + 4;
    var startAt = DateUtils.parseDate(this.settings.get("開始日"));
    var s = new Date(startAt[0], startAt[1]-1, startAt[2], 0, 0, 0);
    rowNo += parseInt((date.getTime()-date.getTimezoneOffset()*60*1000)/(1000*24*60*60)) - parseInt((s.getTime()-s.getTimezoneOffset()*60*1000)/(1000*24*60*60));
    return rowNo;
  };

  GSTimesheets.prototype.get = function(username, date) {
    var sheet = this._getSheet(username);
    var rowNo = this._getRowNo(username, date);
    var row = sheet.getRange("A"+rowNo+":"+String.fromCharCode(65 + this.scheme.columns.length - 1)+rowNo).getValues()[0].map(function(v) {
      return v === '' ? undefined : v;
    });

    return({ user: username, date: row[0], signIn: row[1], signOut: row[2], note: row[3] });
  };

  GSTimesheets.prototype.set = function(username, date, params) {
    var row = this.get(username, date);
    _.extend(row, _.pick(params, 'signIn', 'signOut', 'note'));

    var sheet = this._getSheet(username);
    var rowNo = this._getRowNo(username, date);

    var data = [DateUtils.toDate(date), row.signIn, row.signOut, row.note].map(function(v) {
      return v == null ? '' : v;
    });
    sheet.getRange("A"+rowNo+":"+String.fromCharCode(65 + this.scheme.columns.length - 1)+rowNo).setValues([data]);

    return row;
  };

  GSTimesheets.prototype.getUsers = function() {
    return _.compact(_.map(this.spreadsheet.getSheets(), function(s) {
      var name = s.getName();
      return String(name).substr(0, 1) == '_' ? undefined : name;
    }));
  };

  GSTimesheets.prototype.getByDate = function(date) {
    var self = this;
    return _.map(this.getUsers(), function(username) {
      return self.get(username, date);
    });
  };

  // 休みの曜日を数字で返す
  GSTimesheets.prototype.getDayOff = function(username) {
    var sheet = this._getSheet(username);
    return DateUtils.parseWday(sheet.getRange("B2").getValue());
  };

  return GSTimesheets;
};

// 出勤
  Timesheets.prototype.actionSignIn = function(username, message) {
    if(this.datetime) {
      var data = this.storage.get(username, this.datetime);
      if(!data.signIn || data.signIn === '-') {
        this.storage.set(username, this.datetime, {signIn: this.datetime});
        this.responder.template("出勤", username, this.datetimeStr);
      }
      else {
        // 更新の場合は時間を明示する必要がある
        if(!!this.time) {
          this.storage.set(username, this.datetime, {signIn: this.datetime});
          this.responder.template("出勤更新", username, this.datetimeStr);
        }
      }
    }
  };

  // 退勤
  Timesheets.prototype.actionSignOut = function(username, message) {
    if(this.datetime) {
      var data = this.storage.get(username, this.datetime);
      if(!data.signOut || data.signOut === '-') {
        this.storage.set(username, this.datetime, {signOut: this.datetime});
        this.responder.template("退勤", username, this.datetimeStr);
      }
      else {
        // 更新の場合は時間を明示する必要がある
        if(!!this.time) {
          this.storage.set(username, this.datetime, {signOut: this.datetime});
          this.responder.template("退勤更新", username, this.datetimeStr);
        }
      }
    }
  };

Comments

[gouf]

利用したことがないのですが、1日のうちに数度出退勤を記録するとどうなりますか? 上書きされてしまうのですか?(そうでない場合、シートの記録をもとにデータのフォーマットをあとで加工してあげるとよいような気もします)

[yumatakei]

１日のうちに数度出退勤出来ないようになっています。もし、出勤をslackで行った場合でも反映されません、しかし、出勤時間や退勤時間の更新はできるようになっています。

Answer 1

こういう、「こうしたいな」という発想はいいと思うのですが、仕様を考えるとき、その機能を実際に追加したときにどんな変化が起こるかを考えなくてはいけません。
例えば「おはようございます」と書いたら出勤扱いですが、二回目の「おはようございます」を検知したら二度目の出勤扱いとする、といった「明確な仕様」をまず記載すること、そして「二度目の出勤の時刻を修正したい」「三度目の出勤を指定時刻で入力する」「二度目の出勤データがないのに三度目の入力があったときどうするか」「おはようございますが二回連続で入力されたときどうするか」など、今まででは起こり得なかった無数の状況について「どう対処するか」の仕様が必要になります。
「おはようございます、お疲れ様でしたを何度でも検知してデータを増やす」ということを実装するのは比較的簡単ですが、こういった細かい例外を考慮しないかぎり、改造をやるべきではありません。もう少し細かいところまで机上で仕様を煮詰めてからにしましょう。我々としてもいまのふわっとした疑問にそのまま答えても、足りないところが発生するでしょうし、使い勝手も思ったものと違うものになることもあります。

Comments

[yumatakei]

ご回答ありがとうございます。
仕様を考える際に例外を考えることを怠っていました。
次回から質問させて頂く際には、例外などの仕様も考えてから質問しようと思います。
ご指摘ありがとうございました。