MySQLにテーブル名 members を コマンドプロンプトのコードを組みましたが

Question

すみません。教えていただけないでしょうか？
PC環境は
Windows10 です。
設計段階なので少し慎重に行きます。

上記の画像を元に以下ですが
MySQLにテーブル名 members を コマンドプロンプトのコードを組みましたが
これでいいのか分かりませんでしたので
質問しました。

CREATE TABLE members (id INT NOT NULL PRIMARY KEY AUTO_INCREMENT, name VARCHAR(255) NOT NULL UNIQUE,email VARCHAR(255), password VARCHAR(100) NOT NULL, picture VARCHAR(255),created DATETIME DEFAULT CURRENT_TIMESTAMP
,　modified TIMESTAMP);
コード

　以上ですが、何かありましたら変更していただけないでしょうか？
よろしくお願いいたします。

Comments

[CHERRY]

「良い」・「悪い」の判断基準は何でしょうか。
基準がわからないと第三者では判断できないと思います。

まずは、テスト環境を用意して試してみることをおすすめします。

[international]

ありがとうございます。
了解です。

[m.ts10806]

https://teratail.com/questions/d97olebttz1zsz
前の質問との違いがわかりません。
それに書籍にあるものを作るのを「設計段階」とは？
指定のとおりにSQLを組もうとしてる時点で設計ではなく製造です。
それに「設計のとおりに作る」が製造なので、設計のとおりになっていれば「正解」なのでは？
一体何が問題ですか？
質問は愚か作業依頼にもなっていません。

Answer 1

mysqlの場合、テーブル名やカラム名はバックスラッシュで囲むようにしてください
命題で言うとmodifiedの前に不要な全角スペースが入っておりおかしいです。
idはprimary key指定するとnot nullは自動設定されます。
nameはユニークである必要ありますか？
逆に同じemailで複数アカをとられるのでemailはユニークにした方がよいでしょう

CREATE TABLE `members` (
`id` INT PRIMARY KEY AUTO_INCREMENT,
`name` VARCHAR(255) NOT NULL,
`email` VARCHAR(255) NOT NULL UNIQUE,
`password` VARCHAR(100) NOT NULL,
`picture` VARCHAR(255),
`created` DATETIME DEFAULT CURRENT_TIMESTAMP
,`modified` TIMESTAMP);

Comments

[international]

ご指摘くださいまて、ありがとうございました。
大変助かります^^
開発設計段階中ですが、参考資料には
「`email` VARCHAR(255) NOT NULL UNIQUE,」
あり、「個人情報」は後々トラブの元なりますので省きます。
なので、PHPプログラムも変更することになり、私のリテラシイーで、できるかなーです。
頑張ります。
ご指摘で
　１)テーブル名やカラム名はバックスラッシュで囲む
　２)modifiedの前に不要な全角スペースが入っており
　３)idはprimary key指定するとnot nullは自動設定されます。
　４)nameはユニークである必要ありますか？

　　以上了解です。
　変更します。
　今後とも宜しくお願い致します。