ドットインストールで「PHPでTwitterログインを実装しよう (全23回) プレミアム」と「PHPでFacebookログインを実装しよう (全24回) プレミアム」

Question

ドットインストールで「PHPでTwitterログインを実装しよう (全23回) プレミアム」と「PHPでFacebookログインを実装しよう (全24回) プレミアム」
を同じページに作ると

Notice: Constant DSN already defined in /home/vagrant/config.php on line 13

Notice: Constant DB_USERNAME already defined in /home/vagrant/config.php on line 14

Notice: Constant DB_PASSWORD already defined in /home/vagrant/config.php on line 15

Notice: Constant CALLBACK_URL already defined in /home/vagrant/config.php on line 17

Notice: A session had already been started - ignoring session_start() in /home/vagrant/config.php on line 19

Fatal error: Cannot redeclare h() (previously declared in /home/vagrant/functions2.php:3) in /home/vagrant/functions.php on line 5

となります。

おそらくインデックスにrequire_once(DIR . '/config2.php');
require_once(DIR . '/config.php');

この二つがあるからだと思うのですがどうすれば

どうすれば同じページにツイッターログインボタンとフェイスブックログインボタンを作ることができますか？

Comments

[izkn]

こちらの質問が他のユーザから「質問の範囲が広すぎる」という評価を受けています
わからない点を明確にし、調査したこと・試したことと共に記入していただくと、回答が得られやすくなります。

[退会済みユーザー]

エラーメッセージの解決方法は分かっているのですか？
最後の一文のせいで、何を解決したいのかがぼやけてしまっています。
どうすればできるかは実装次第です。
まず、config.phpを二つrequireしているのをやめてみたらどうでしょうか。

[KosukeYamamoto]

configを二つ作らなければ、DSNを　defineできなくはないでしょうか？

[退会済みユーザー]

どちらにしてもredeclareで重複定義されているというエラーが出ているので、config.phpを整理して、一個にまとめるべきでは？それか、どちらかのファイルのh()という関数を消すとか。

[KosukeYamamoto]

なるほど！わかりました。ありがとうございます。

[KosukeYamamoto]

一つにまとめたとき、DSNはどうしましょうか？

[KosukeYamamoto]

ちなみにh（）はconfigに出てきていません＞＜

[退会済みユーザー]

こちらの質問が他のユーザから「やってほしいことだけを記載した丸投げの質問」という指摘を受けました
「質問を編集する」ボタンから編集を行い、調査したこと・試したことを記入していただくと、回答が得られやすくなります。

Answer 1

エラーメッセージを読みましょう。
まずこれができないと、エラー一個ずつ質問しないといけなくなりますよ。

Fatal error: Cannot redeclare h() (previously declared in /home/vagrant/functions2.php:3) in /home/vagrant/functions.php on line 5
ここにファイル名が出ていますよね。config.phpから呼び出していませんか？
自分の書いたコードが、どのように動いているか、追いかけていきましょう。

Comments

[KosukeYamamoto]

h（）は二回使えないということですか？
二回使うにはどうすればいいでしょうか？

[KosukeYamamoto]

h(）は解決しました。
一つにまとめたとき、DSNはどうしましょうか？

[退会済みユーザー]

DSNの設定は2つ要らないですよね。
既にconfigで設定してて、config2でも設定しているために重複の警告が出ているのですから、不要な方のdefineを消せば良いです。
そういうことを続けていくと、結局config2は不要で、configに必要な設定を編集・追記すればいいと気が付くはずです。

[KosukeYamamoto]

いえDSNは二つ必要です。ツイッター用とフェイスブック用です。
なので、困っているのです。

Answer 2

Constant DSN already defined

ここで言っているDSNというのは只の定数名なので、意味はないのです。HOGEでもHAGEでもいいのです。

だから、只動けばいいのであれば、
片方をFACEBOOK_DSN、もう片方をTWITTER_DSNなんかにしてしまえばいいです。
他の定数も同様、それを使う場所も書き換えればいいです。

本来は、こういうのを定数にしちゃうのはおかしいのですが、
ドットインストールは勉強サイトなので、まぁ、勉強用と割りきりましょう。

Comments

[退会済みユーザー]

>A session had already been started

これは、session_start()が二回呼ばれてるってことでしょう。
これは、後に実行されてる方を消すだけでいいですが、そもそもの書く場所を変えるほうがいいと思います。

[KosukeYamamoto]

ちなみに
'FACEBOOK_DB_USERNAME',
こういった形にしても問題ないでしょうか？
Notice: A session had already been started - ignoring session_start() in /home/vagrant/config.php on line 19

あと上記の部分はどうしましょうか？？

[退会済みユーザー]

あんまり良く見ないで書いたのですが、新規質問のほうをみてなんとなく意味がわかりました。

>'FACEBOOK_DB_USERNAME',
>こういった形にしても問題ないでしょうか？

このDBというのは、あなたのデータベースのことですよね？
であれば、

DB_USERNAME
DB_PASSWORD
CALLBACK_URL
については、どちらか片方を削除するだけでいいです。あるいは、例えばcommon_config.phpなんかをつくって、そこで１回定義すればいいです。
session_start()も、そこで１回実行すればいいかなと。（configファイルでsession_start()するって変な気もしますが）

[KosukeYamamoto]

ありがとうございます。無事解決しました。
もしよければ、https://teratail.com/questions/24856

こちらもお願い致しますmm