常連

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

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

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　というエラーの対処法について教えてください。

### 前提・実現したいこと

Reactの練習で自己紹介ページを作っています。
自己紹介編集画面をReactModalを用いて作成しようと試みています。
SAVEボタンを押したときに、データを保存しModalWindowを閉じる処理を同時に行うように実装を行いたかったのですが
うまく行きませんでした。
onCliickの中でコンマを使えないのは知っているのですが、ボタンクリックされたときに2つの処理を実行させたいです。
ちなみにキャンセルボタンは想定通りの動きをし、`this.saveProfile`のみならデータは想定通りに保存されています。
ご教授いただけますと幸いです。

### 該当のソースコード

親コンポーネント
```JavaScript
    openModal = () => {
        this.setState({isModalOpen: true})
    }

    closeModal = () => {
        this.setState({isModalOpen: false})
    }

    render(){
        return (
            <div>
                <h2>Profile</h2>
                <Button
                    label={'Edit Profile'}
                    onClick={this.openModal}
                />
                <EditProfile 
                    openModal = {this.state.isModalOpen}
                    onClose = {this.closeModal}
                />
            </div>
        )
    }
```
子コンポーネント
```JavaScript
render(){
        return(
            <div>
                <ReactModal
                    isOpen = {this.props.openModal}
                >
　　　　　　　　　　　・・・
　　　　　　　　　　　<名前や年齢などのプロフィールをInputで入力します。略>
                    ・・・
　　　　　　　　　　　<Button
                        label={'Save'}
                        onClick = {
                            this.saveProfile,
                            this.props.onClose
                        }
                    />
                    <Button
                        label={'Cancel'}
                        onClick={this.props.onClose}
                    />
                </ReactModal>
            </div>
        )
    }
```

### 試したこと

下記などを参考にonClickで複数の処理の実行ができないか試したのですが、うまく実装することができませんでした。
[https://www.366service.com/jp/qa/b582191adb9bfb525d728951fcbafb7a
](https://www.366service.com/jp/qa/b582191adb9bfb525d728951fcbafb7a)

コードレビュー