常連

## テーマ、知りたいこと
このレビュアーのコメントについて、どう思いますか？
きつい言い方ですか？

このようなコードの書き方は問題ありですね。
複雑で可読性も非常に悪いコードです。修正をお願いいたします。

PRを上げる前に改善が必要だと思ったところは、再度確認をしてからPRするようにお願いいたします。



レビュアーのコメントについて

### 前提・実現したいこと
プログラミング初心者です
Pythonでシフト作成を効率化したいと考え(https://naomichi.work/2020/02/09/2020-02-09-010155/)このサイトを参考にし書いておりました。

### 発生している問題・エラーメッセージ

AttributeError: 'str' object has no attribute 'holiday_list'となってしまいました。

### 該当のソースコード

```Python37
days, number_employee = shift.shape[0], shift.shape[1]
holidays = 'play_ground' .holiday_list(days)
```

### 試したこと

このサイトなどで調べたのですが、知識が足らず明確な解決方法が見つけられませんでした。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

AttributeError: 'str' object has no attributeとなってしまった

TypeError: Cannot read property 'getRange' of null（行 175、ファイル「コード」）
が出てしまいます。
どうしてでしょうか、


//項目名称を取得する
function getItemNameList(col){
  var res = [];
  var spread = SpreadsheetApp.getActiveSpreadsheet() ;
  var sheet = spread.getSheetByName('database');
  if (col == -1) {
    var values = sheet.getRange(1, 1, 1, sheet.getLastColumn()).getValues();
  } else {
    **var values = sheet.getRange(1, 1, 1, col).getValues();**
  }
  Logger.log('values getItemNameList');
  Logger.log(values);
  return values;
}

太字の個所です。
どうしたらいいのかわかりません。
初心者です。

コード.gsについて

###質問内容
NullReferenceException: Object reference not set to an instance of an object
を対処するにはどのようにすれば良いのか教えていただきたいです．
以下のサイトの立ち絵の表示の部分で詰まっております．

[リンク内容](https://note.com/frabbit_mbp/n/nee57093c1704)

以下のエラーのでるコードの改変前で正常に動くものを作っているのですが，文字数制限であげることが出来ないので，次のnote記事に載せます．

[質問用コード](https://note.com/akanaimomo/n/n98f336692ab4)

###エラー
```ここに言語を入力
NullReferenceException: Object reference not set to an instance of an object
GameManager.OnClick () (at Assets/Scripts/GameManager.cs:130)
GameManager.Update () (at Assets/Scripts/GameManager.cs:98)

```


###エラーが出るコード
```C#
using UnityEngine;
using UnityEngine.UI;
using System.Collections.Generic;
using System.Collections;

// MonoBehaviourを継承することでオブジェクトにコンポーネントとして
// アタッチすることができるようになる
public class GameManager : MonoBehaviour{
   // SerializeFieldと書くとprivateなパラメーターでも
   // インスペクター上で値を変更できる
   [SerializeField]
   private Text mainText;
   [SerializeField]
   private Text nameText;
   [SerializeField]
   private float captionSpeed = 0.2f;
   [SerializeField]
    private GameObject nextPageIcon;
    [SerializeField]
    private Image backgroundImage;
    [SerializeField]
    private string spritesDirectory = "Sprites/";
    [SerializeField]
    private GameObject characterImages;
    [SerializeField]
    private string prefabsDirectory = "Prefabs/";

    private List<Image> _charaImageList = new List<Image>();

   
   // パラメーターを追加
    private const char SEPARATE_MAIN_START = '「';
    private const char SEPARATE_MAIN_END = '」';
    private Queue<char> _charQueue;
    private const char SEPARATE_PAGE = '&';
    private Queue<string> _pageQueue;
    private const char SEPARATE_COMMAND = '!';
    private const char COMMAND_SEPARATE_PARAM = '=';
    private const string COMMAND_BACKGROUND = "background";
    private const string COMMAND_SPRITE = "_sprite";
    private const string COMMAND_COLOR = "_color";
    private const string COMMAND_CHARACTER_IMAGE = "charaimg";
    private const string COMMAND_SIZE = "_size";
    private const string COMMAND_POSITION = "_pos";
    private const string COMMAND_ROTATION = "_rotate";
    private const string CHARACTER_IMAGE_PREFAB = "CharacterImage";
    

    private string _text = 
    "!background_sprite=\"background_sprite1\"!charaimg_sprite=\"polygon\"=\"background_sprite2\""+
       "!charaimg_size=\"polygon\"=\"500, 500, 1\"&ともり「トーキョーは　薄暗い閃光放って」&ともり 「消耗した MPも回復できぬまま」" +
       "&!background_sprite=\"background_sprite2\"!background_color=\"255,0,255\"!charaimg_pos=\"polygon\"=\"-500, 500, 0\"&ともり「相当な　厄介者の説法聞いて」";


    /**
    * 文を1文字ごとに区切り、キューに格納したものを返す
    */
    private Queue<char> SeparateString(string str){
        // 文字列をchar型の配列にする = 1文字ごとに区切る
        char[] chars = str.ToCharArray();
        Queue<char> charQueue = new Queue<char>();
        // foreach文で配列charsに格納された文字を全て取り出し
        // キューに加える
        foreach (char c in chars) charQueue.Enqueue(c);
        return charQueue;
    }

    private bool OutputChar(){
        if (_charQueue.Count <= 0){
            nextPageIcon.SetActive(true);
            return false;
        }
        mainText.text += _charQueue.Dequeue();
        return true;
    }


    // 文字送りするコルーチン
    private IEnumerator ShowChars(float wait){
        // OutputCharメソッドがfalseを返す(=キューが空になる)までループする
        while (OutputChar())
            // wait秒だけ待機
            yield return new WaitForSeconds(wait);
        // コルーチンを抜け出す
        yield break;
    }

    //全文表示
    private void OutputAllChar(){
       StopCoroutine(ShowChars(captionSpeed));
        while (OutputChar()) ;
        nextPageIcon.SetActive(true);
    }

    // MonoBehaviourを継承しているとき，毎フレーム呼ばれる
    private void Update(){
        // 左(=0)クリックされたらOnClickメソッドを呼び出し
        if (Input.GetMouseButtonDown(0)) OnClick();
    }

    //文字列を指定した区切り文字毎に区切り，キューに格納したものを返す
    private Queue<string> SeparateString(string str, char sep){
        string[] strs = str.Split(sep);
        Queue<string> queue = new Queue<string>();
        foreach (string l in strs) queue.Enqueue(l);
        return queue;
    }

    //初期化
    private void Init(){
        _pageQueue = SeparateString(_text, SEPARATE_PAGE);
        ShowNextPage();
    }

    //次のページの表示
    private bool ShowNextPage(){
        if (_pageQueue.Count <= 0) return false;
        // オブジェクトの表示/非表示を設定する
        nextPageIcon.SetActive(false);
        ReadLine(_pageQueue.Dequeue());
        return true;
    }

    private void Start(){
        Init();
    }

    //クリックしたときの処理
    private void OnClick(){
        if (_charQueue.Count > 0) OutputAllChar();
        else{
            if (!ShowNextPage()){
                // UnityエディタのPlayモードを終了する
                UnityEditor.EditorApplication.isPlaying = false;
            }
        }
    }


    private void ReadLine(string text){
        // 最初が「!」だったら
        if (text[0].Equals(SEPARATE_COMMAND)){
            ReadCommand(text);
            ShowNextPage();
            return;
        }
        string[] ts = text.Split(SEPARATE_MAIN_START);
        string name = ts[0];
        string main = ts[1].Remove(ts[1].LastIndexOf(SEPARATE_MAIN_END));
        nameText.text = name;
        mainText.text = "";
        _charQueue = SeparateString(main);
        StartCoroutine(ShowChars(captionSpeed));
    }

    /* スプライトをファイルから読み出し、インスタンス化する*/
    private Sprite LoadSprite(string name){
        return Instantiate(Resources.Load<Sprite>(spritesDirectory + name));
    }

    /*パラメーターから色を作成する*/
    private Color ParameterToColor(string parameter){
        string[] ps = parameter.Replace(" ", "").Split(',');
        if (ps.Length > 3)
            return new Color32(byte.Parse(ps[0]), byte.Parse(ps[1]), byte.Parse(ps[2]), byte.Parse(ps[3]));
        else
            return new Color32(byte.Parse(ps[0]), byte.Parse(ps[1]), byte.Parse(ps[2]), 255);
    }

    

    private void SetBackgroundImage(string cmd, string parameter){
        cmd = cmd.Replace(COMMAND_BACKGROUND, "");
        SetImage(cmd, parameter, backgroundImage);
    }

        /* 立ち絵の設定*/
    private void SetCharacterImage(string name, string cmd, string parameter){
        cmd = cmd.Replace(COMMAND_CHARACTER_IMAGE, "");
        name = name.Substring(name.IndexOf('"') + 1, name.LastIndexOf('"') - name.IndexOf('"') - 1);
        Image image = _charaImageList.Find(n => n.name == name);
        if (image == null){
            image = Instantiate(Resources.Load<Image>(prefabsDirectory + CHARACTER_IMAGE_PREFAB), characterImages.transform);
            image.name = name;
            _charaImageList.Add(image);
        }
        SetImage(cmd, parameter, image);
    }

/*パラメーターからベクトルを取得する*/
    private Vector3 ParameterToVector3(string parameter){
        string[] ps = parameter.Replace(" ", "").Split(',');
        return new Vector3(float.Parse(ps[0]), float.Parse(ps[1]), float.Parse(ps[2]));
    }

    //コマンドの呼び出し
    private void ReadCommand(string cmdLine){
        cmdLine = cmdLine.Remove(0, 1);
        Queue<string> cmdQueue = SeparateString(cmdLine, SEPARATE_COMMAND);
        foreach (string cmd in cmdQueue){
            string[] cmds = cmd.Split(COMMAND_SEPARATE_PARAM);
            if (cmds[0].Contains(COMMAND_BACKGROUND))
                SetBackgroundImage(cmds[0], cmds[1]);
            if (cmds[0].Contains(COMMAND_CHARACTER_IMAGE))
                SetCharacterImage(cmds[1], cmds[0], cmds[2]);
        }
    }

    //画像の設定
    private void SetImage(string cmd, string parameter, Image image){
        cmd = cmd.Replace(" ", "");
        parameter = parameter.Substring(parameter.IndexOf('"') + 1, parameter.LastIndexOf('"') - parameter.IndexOf('"') - 1);
        switch (cmd){
            case COMMAND_SPRITE:
                image.sprite = LoadSprite(parameter);
                break;
            case COMMAND_COLOR:
                image.color = ParameterToColor(parameter);
                break;
            case COMMAND_SIZE:
                image.GetComponent<RectTransform>().sizeDelta = ParameterToVector3(parameter);
                break;
            case COMMAND_POSITION:
                image.GetComponent<RectTransform>().anchoredPosition = ParameterToVector3(parameter);
                break;
            case COMMAND_ROTATION:
                image.GetComponent<RectTransform>().eulerAngles = ParameterToVector3(parameter);
                break;
        }
    }




}
```

NullReferenceException: Object reference not set to an instance of an objectの解決

### 前提・実現したいこと

Progateの学習コース初級の模写をしています。
ProgateではCopy-containerを使って表現をしていますが、
copy-containerの必要性（記載する意味）を教えてください。
HTMLでCopy-containerを使わない記載方法はありますか？



### 該当のソースコード
<div class="main">
      <div class="copy-container">
        <h1>HELLO WORLD<span>.</span></h1>
        <h2>プログラミングの世界へようこそ</h2>
      </div>

### 試したこと

模写で、copy-container を記載しなかった。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。細な情報を記載してください。

Copy-containerの必要性を教えてください。(模写しています）

### 前提・実現したいこと

スプレッドシートで期日アラートメールを送信するシステムを作っています。
全くの初心者でwebからソースコードをコピーし、必要な箇所を書き換えて使用しており、
以下のエラーメッセージが発生しました。

ただ、コードを手動で実行すると、期日が過ぎた項目が正常にメールで届くのは届いているのです。

GASの本などで調べてみたのですが、基本的な事を分かっていない状態です。
期日が過ぎた時にアラートメールが届き、エラーを解決したいです。
解決方法をご教授願えませんでしょうか。

### 発生している問題・エラーメッセージ
以下が設定したgoogleアカウントに届くエラーメッセージです。

![イメージ説明](0460c45a04b653dc3d14314df25f2a60.png)



### 該当のソースコード

function mailAlerts() {
  //シートを取得
  var bk = SpreadsheetApp.getActiveSpreadsheet();
  var sh = bk.getActiveSheet();
  
  var last_row = sh.getLastRow(); //最後の行を取得(繰り返し処理の回数)
  var begin_row = 2;　// 処理を開始する行 (1行目は項目名なので2から)
  
  //今日の日付を取得し文字列を成型する
  var today　=  new Date(); 　
  var formatDate = Utilities.formatDate(today,"JST","yyyy/MM/dd");

  //繰り返し処理(1行づつ処理し、行の数だけ繰り返す)
  for(var i = begin_row; i <= last_row; i++) {
　　//それぞれのセルの中身を取得していく
    //通知期日(日付は成型する)
    var sell1 = "B"+i;
    var value1 = sh.getRange(sell1).getValue();
    var value1 = Utilities.formatDate(value1,"JST","yyyy/MM/dd");
    //進捗状況
    var sell2 = "C"+i;
    var value2 = sh.getRange(sell2).getValue();
    //施設名
    var sell3 = "D"+i;
    var value3 = sh.getRange(sell3).getValue();
    //定期メンテナンス内容
    var sell5 = "F"+i;
    var value5 = sh.getRange(sell5).getValue();
    //実施日
    var sell6 = "H"+i;
    var value6 = sh.getRange(sell6).getValue();
    
　　//もし、今日の日付と納期が同じで、なおかつ状況が完了でない場合メールを送る
    if(formatDate == value1 && value3 !== "完了"){
      GmailApp.sendEmail('※ここにはメールアドレスを入力しています',
                         '【定期メンテ通知】施設名:' + value2 + '/' + value5,
                         '■施設名:' + value3 + '\n■定期メンテナンス内容:'+ value5 + '\n■実施日:'+ value6);
    }    
  }
  
 }
    

### 試したこと
webで調べたところ、（String,String,String）のエラーは文字が間違っているかと思い、

 var formatDate = Utilities.formatDate(today,"JST","yyyy/MM/dd");

の箇所のtodayの後ろに空白が入っていたので削除しました。
ただ、削除する際に半角か全角かは確認しませんでした・・・

しかし、解決せずエラーになります。

### 補足情報（FW/ツールのバージョンなど）
MacでGASを入力しています。

GASのエラー「Exception: パラメータ（String,String,String）が Utilities.formatDate のメソッドのシグネチャと一致しません」の解決方法を教えて下さい

```# import libraries
import requests
import pandas as pd
from pandas.io.json import json_normalize
import openpyxl

＊
＊　省略
＊

df1_selected.to_excel('a.xlsx', sheet_name='20210728')　-①

```
上記のコードでAPIから取得したデータをエクセルに格納します。
その後以下のコードと①と入れ替えて実行し、先ほど作成したエクセルに追加のシートを追加し、そこにデータを格納しようとしています。

```
with pd.ExcelWriter("a.xlsx") as writer:
 writer.book = openpyxl.load_workbook("a.xlsx")
 df1_selected.to_excel('a.xlsx', sheet_name="202107282",index=False, header=False)
```

しかし少なくとも一つのシートを持つ必要があるというエラーで追加のシートが作成できません。
```raise IndexError("At least one sheet must be visible")
IndexError: At least one sheet must be visible
```

対処法を教えてもらえると嬉しいです。
ちなみに1回目のデータをexcel作成後格納することまではできていて
その後の2回目の既存エクセルにシート追加後、データを格納する部分でつまづいています。
作成されるエクセルファイルは/USER/名前/の位置に置いてあります。

IndexError: At least one sheet must be visible　というエラーの対処法について教えてください。

いくつかの「XXX座標系」という概念がある状況での話です．
例えば，３次元のグラフィクスを扱うような場面では，よく｛ワールド座標系，カメラ座標系｝といった概念が出てきますよね．そういう話です．
（以下，コード例では簡便さのため２次元で書きますが）

例えば

```C++
//
//行列やベクトルの型とか演算は残念なことに C++ 標準で用意されていないから
//自前でこんなのを書いて（あるいは既存の何かを持ってきて）使う
//
class Vec2{ ... };  //2次元のベクトル用の型
class Mat2x2{ ... };  //2x2 の行列の型

//
int main()
{
  //なんかベクトルと行列を用いた演算
  Vec2 V{ 10, 20 };
  Mat2x2 TransMat{ {0,1}, {-1,0} };
  Vec2 TransedVec = TransMat * V;  //（こういう operator の実装もあるのだとして）
}
```

みたいなコードがあるとして，この `main()` 内のコードは
「何らかの座標変換をしているんじゃないかな感」はある（と思う）のですが，
「どの座標系の量をどの座標系の量に変換しているのか」は全くわかりません．
それ以前にベクトル `V` とはどの座標系での量として定義している つもり/ハズ なのか？ といったことも謎です．

で，私くらいの馬〇になると，これ系の演算を実装するときに
その辺の事柄を取り違えたり間違ったりしてしまうことが割とあって，その結果は（当たり前ですが）全く意味不明な演算結果になります．
そういう場合のデバッグは結構厄介です．
（特に「分かり切っている」つもりの箇所でやらかしてたりすると気づきにくい）

---

そこで，↑の `Vec2` とか `Mat2x2` とかいう型を
「ベクトルです」「マトリクスです」という単純な（汎用な）存在とするのではなくて，
「ワールド座標系でのベクトルです」とか「ワールド座標系からカメラ座標系への変換行列です」とかいう専用の型にしてしまえば
間違いを事前抑止できる可能性があるのではなかろうか？　とか夢想したわけです．

扱う座標系の種類の個数が定まっているのであれば，

```C++
//座標系の種類
enum class CoordinateSystem{	World, Camera	};

//特定の座標系におけるベクトル
template< CoordinateSystem CS >
class Vec2
{
public:
	//各種演算は同じ座標系のベクトル同士じゃないとできない
	Vec2 &operator +=( const Vec2 &rhs );
	Vec2 &operator -=( const Vec2 &rhs );
	double Dot( const Vec2 &rhs ) const;
};

//From座標系の量（ベクトル）を To座標系での量に変換するマトリクス
template< CoordinateSystem From, CoordinateSystem To >
class Mat2x2
{
public:
	//乗じるベクトルは From座標系で，乗じた結果は To座標系
	Vec2<To> operator *( const Vec2<From> &V ) const;
};

//型名が長ったらしいので using
using WorldVec = Vec2< CoordinateSystem::World >;
using CameraVec = Vec2< CoordinateSystem::Camera >;
using W2CMat = Mat2x2< CoordinateSystem::World, CoordinateSystem::Camera >;
```

とかしてしまえば，
変数を 宣言/定義 する際には必ず座標系を明確に意識せねばならず，馬鹿みたいな間違いを抑制できるのではないかな，と．
また，

```C++
WorldVec V1{ 1,2 };
CameraVec V2{ 3,4 };
auto V3 = V1 + V2;  //←その演算の意味は何なの？
```

みたいなのがコンパイルを通らなくなるのは大変素敵なのではなかろうか？　と．

……が，他方では
「理由ははっきりしないが，なんだか馬鹿馬鹿しい（仰々しい？）コードと思える」
「そんなの変数名とかで表現すれば十分じゃね？　不足ならコメントでも書けと」
みたいな気もしてきます．

↑のようなコード，**どう思いますか？**
（「意見交換」ならこんな雑な聞き方をしても良いと思っている）


コードレビュー