早押しボタンの回路とプログラム

Question

###実現したいこと
回路中にボタンが二つ(A, B)あり，Aを先に押したなら"player1"，Bならば"player2"と表示させるもの．
片方のボタンが押されたらLEDを点灯させて，入力の受付を5秒間拒絶する．

###困っていること
次に示す二つのコードそれぞれで，ボタンによるLEDのON/OFFとLCDの表示は成功しているのだが，組み合わせた「現段階のコード」にあるものを用いると，ボタンを押してもLEDとLCDが点灯しない．(コードと配線を少し変えたらLCDの表示は成功した．LEDはボタンに関わらずずっと点灯している．これは5番につないでいたところを5Vにつないで常時供給しているからである．)
ボタンを押したときに光って，こちらも5秒経過後に消灯したいです．

コードや配線でおかしなところを教えていただきたいです．

int ledPin = 5;
int buttonApin = 9;
int buttonBpin = 8;

byte leds = 0;

void setup() 
{
  pinMode(ledPin, OUTPUT);
  pinMode(buttonApin, INPUT_PULLUP);  
  pinMode(buttonBpin, INPUT_PULLUP);  
}

void loop() 
{
  if (digitalRead(buttonApin) == LOW)
  {
    digitalWrite(ledPin, HIGH);
  }
  if (digitalRead(buttonBpin) == LOW)
  {
    digitalWrite(ledPin, LOW);
  }
}

#include <Wire.h>
#include <LiquidCrystal_I2C.h>
LiquidCrystal_I2C lcd(0x27,16,2); // set the LCD address to 0x27 for a 16 chars and 2 line display
void setup()
{
lcd.init(); // initialize the lcd
lcd.init();
// Print a message to the LCD.
lcd.backlight();
lcd.setCursor(0,0);
lcd.print("Hello");
lcd.setCursor(0,1);
lcd.print("Hi");
}
void loop()
{
}

###現段階のコード(1回改訂)

#include <Wire.h>
#include <LiquidCrystal_I2C.h>
LiquidCrystal_I2C lcd(0x27,16,2); // set the LCD address to 0x27 for a 16 chars and 2 line display

int ledPin = 5;
int buttonApin = 9;
int buttonBpin = 8;

byte leds = 0;

void setup() {
  // put your setup code here, to run once:
  lcd.init();
  lcd.backlight();
  pinMode(ledPin, OUTPUT);
  pinMode(buttonApin, INPUT_PULLUP);  
  pinMode(buttonBpin, INPUT_PULLUP);
}

void loop() {
  // put your main code here, to run repeatedly:
  if (digitalRead(buttonApin) == LOW)
  {
    digitalWrite(ledPin, LOW);
    // Print a message to the LCD.
    lcd.init();
    lcd.setCursor(0,0);
    lcd.print("answer");
    lcd.setCursor(0,1);
    lcd.print("player1");
    delay(5000);
    lcd.init();
  }
  if (digitalRead(buttonBpin) == LOW)
  {
    digitalWrite(ledPin, LOW);
    // Print a message to the LCD.
    lcd.init();
    lcd.setCursor(0,0);
    lcd.print("answer");
    lcd.setCursor(0,1);
    lcd.print("player2");
    delay(5000);
    lcd.init();
  }
}

###現段階の配線
ご指摘により掲載した写真と異なり，「~5」につないでいたものを「5V」のところにつなぎました．



次の二つの画像で示しているもので，二枚目の赤い線を一枚目の赤い線と抵抗の間に挟んでいるものとなっています．

###組み合わせようとしているもの

###参考にしているサイト
参考1
このLesson5と
参考2
ここのKesson26を組み合わせようとしています．

###追記1
困ったことを変更いたしました．
現段階のコードを変更いたしました．
現段階の配線に説明を付け加えました．

Answer 1

配線について言えば、LCDのVCCが5番ピンに繋がっています。
これはLCDの電源を制御しようとしているのでしょうか？単なる間違い？
いずれにしても、5Vに繋いでください。
ピンの出力能力的に、LEDとLCDの直接制御は止めた方が良いです。

スケッチで言えば、このスケッチでどこまで行うつもりで書いたのでしょうか？
バックライトはスイッチが押された時に点く（押されるまでは消す）仕様なのでしょうか？
普通は

lcd.init();
lcd.backlight();

の2行はsetup関数内に書けばいいです。loop関数でわざわざ初期化まではしません。
※現在のスケッチでは押されたら数十回loop関数が回って初期化する事になります。
バックライトを制御したいのなら
nobacklight()?関数あたりとセットで行う事になります。

現在のスケッチが早押しにはなっていないのは判りますよね？

「5秒間拒絶」は単にdelay(5000);をそれぞれの最期に付ければ実現できるでしょうけど、
あまり良くない仕様だと思います。
出題者がリセットする。或いは回答者にリセットさせる（プレートが立ちあがる仕様など）、が普通だと思います。

もう少し仕様を考えた方が良いです。

Comments

[grape_ll]

配線についてですが，撮影の角度が良くありませんでした．実際は5番に正しく入っています．見づらい撮影で申し訳ございません．
5Vにつなぐと
```
pinMode(ledPin, OUTPUT);
```
などは意味のないものとなってしまうのですが，常時5Vを流してもよいのでしょうか．

スケッチは，現段階ではひとまずボタンを押したらLCDに出力，LEDが点灯してくれればいいなと思って書きました．

まだいじり始めたばかりで，何ができるのかが良くわかっていないため，あまりいい仕様となっていないのは自分でもわかります．ひとまず，
ボタンを押したらLCDとLEDが点く→早押し機能を入れる
という順で1つずつ片づけていこうと思っています．

[nac_tnk]

LCDのVCC？或いは5V？（電源の+ピン）の事です。
上の一番下のイラスト図での赤ケーブルです。

[nac_tnk]

LEDを操作する部分はそのままです。
何故かそこに、LCDの電源ピンが刺さっている、という事です。

[grape_ll]

状態が改善したので質問内容を編集しました．
変えたことは，コードのloop部分と5Vにつなぐというものです．
ご指摘いただいたように修正させていただいたところ，ボタンを押した方のplayerを表示させることは出来ました．しかし，LEDが常時ついた状態です．これを，ボタンを押したとき~5秒後までの間に制限したいのですが，何か案はございますでしょうか．

[nac_tnk]

何か変な事をしたのでは？
元の写真から言えば、LEDの右にある濃灰（又は或いは黒）のケーブルを抜いて、それを5Vに挿し直すだけです。
※常時、LEDがついているのなら多分LED部も5Vに繋いでしまっているのでしょう。
上のスケッチでは
digitalWrite(ledPin, LOW);
ですから光る事はありません。→そこはHIGHでは？

[grape_ll]

LEDの方を5Vにさしてました．
点灯の方もHIGHにしたら光りました．
質問点はすべて解決しました．ありがとうございました．

Answer 2

考えたアルゴリズム
片方のボタンが押されたらLEDを点灯させて，入力の受付を5秒間拒絶する．

それは「アルゴリズム」(処理手順)というものではないでしょう?期待する動作を記述しているだけで、「どうやってそれを実現する」という話を書いて初めてアルゴリズムと呼べると思います。

配線については...
LED/スイッチ/LCDはそれぞれ独立しているのですから、配線は最新版でそれぞれのテスト用のスケッチが動作するかどうか試してみましょう。「動かなくなった」ところがあればそれについて変更したところが間違いだということです。

スケッチについては...言ってしまえば、やりなおし。
いろいろな要素をいっぺんに詰め込まないで、
・Aが押されたら/Bが押されたらそれぞれがLCDに表示される(A/Bの識別)
・Aを押して離す前にBを押しても、逆にBを押して離す前にAを押しても表示が変化しないようにする(早押し検出の実現)
・Aを押して(Bを押して)離してから5秒以内にAまたはBを押しても反応しないようにする
を順次取り込んでプログラムを「育てて」行ったらどうでしょう。

Comments

[grape_ll]

アルゴリズムについて，おっしゅる通りですね．あまりにも大ざっぱすぎました．
これも何をしたいのか，ですね

段階的にプログラムを書くのは基本ですよね．
arduinoでプログラムを書いた経験がほとんどなく，理解に乏しいためこのようになってしまいました．
もう一度悉皆読んでみようと思います．

[grape_ll]

少し改善したので質問を編集させていただきました．

[thkana]

全然別の結線図で、たまたま同じ色だったから、と線を繋ぎ合わせていませんか。
ちゃんと回路の意味を考えて接続してください。