質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.50%

  • PHP

    24014questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • HTML

    11473questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

PHPとHTMLの綺麗な書き方、インデントを意識した書き方というのがいまいち理解できていないです

解決済

回答 5

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,424

kokokooooooooo

score 12

イメージ説明
イメージ説明

コードの綺麗な書き方がわからないです。
あと質問として、スペースとタブキーは、混在したものにするのはよくないのでしょうか

<?php

$user     = 'root';
$password = 'root';

//ユーザ名とパスワードが送信された時
if (isset($_POST['name']) && isset($_POST['password'])){//論理積

try{
  $dbh = new PDO('mysql:host=localhost;dbname=task;charset=utf8',$user,$password);
  $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
  //プリペアドステートメントを作成
  $stmt = $dbh->prepare("SELECT * FROM task WHERE name= :name AND password= :pass");
  //プリペアドステートメントは名前が:nameかつパスワード:passであるユーザを検索している

  //パラメータを割り当て
  $stmt->bindParam(':name', $_POST['name'],     PDO::PARAM_STR);
  $stmt->bindParam(':pass', $_POST['password'], PDO::PARAM_STR);
  $stmt->execute();

   if($stmt->fetch()){//値の取得を行います
   header('Location: index2.php');
   exit();
   }else{
   header('Location: login.php');
   exit();
   }
}catch(PDOException $e){
  die('エラー:' . $e->getMessage());
  }
    }else{//パスワード、名前を送っていない場合は、html表示
?>

<html>
<head>
  <meta http-equiv="Content-Type" content="text/html;charset=UTF-8">
  <title>ログイン画面</title>
</head>
<body>
  <h1>ログインサイト</h1>
  <h2>ログイン</h2>
<form action="login.php" method="post">
  <p>ユーザ名:    <input type="text" name="name" /></p>
  <p>パスワード:<input type="text" name="password" /></p>
  <p><input type="submit" value="ログイン" /></p>
</form>
  <a href="register.php">新規登録</a>
</body>
</html>
<?php } ?>
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 5

+3

コードのキレイな書き方=他の人が見ても読みやすいコード
と考えればいいと思います。

タブとスペースは混在しないほうがいいです。どちらか片方に統一したほうがいいです。

またどの様に整頓すべきかについては、エディタのコードを整頓してくれる機能(例えばAtomエディタの「Atom-Beautify」プラグインなど)を参考すると良いかもしれません。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

+2

一般的に、プログラムのコードは階層構造を持っています。
そして、その階層構造を一目でわかるようにしよう、というのがインデントの思想です。

究極的には、そのプロジェクトにかかわる人間が読みやすければどのような形式でもよいのですが、
一般的に読みやすいとされているパターンがあるので、それを紹介します。

 タブとスペースについて

混在しないほうが良いです。タブ幅はエディタや設定によって異なるので、混在しているとあなたの環境でインデントがそろっていても、別の環境で読んだ際にぐちゃぐちゃになる可能性があります。

 階層構造の考え方

例えば、以下の PHP コードなら

<?php
  echo "Hello, PHP";
?>


PHP として解釈する範囲 <?php ~ ?> の中に、 出力文 echo ~  が存在するので、
中身にあたる echo ~  は外枠の <?php ~ ?> に対して1レベルインデントをつけています。

HTML や XML のインデントは簡単で、タグの階層構造そのままにインデントをつければよいでしょう。
開始タグ <hoge> から終了タグ </hoge> までの間がそのタグの中身になるので、1ランクインデントをつけます。

<html>
  <head>
    <title>ログイン画面</title>
  </head>
  <body>
    <h1>ログインサイト</h1>
    <h2>ログイン</h2>
    <form action="login.php" method="post"></form>
  </body>
</html>

条件分岐など、処理の流れを変化させる制御文などでは、分岐される処理についてそれぞれ1ランクインデントをつけます。
基本的には、プログラムコードの括弧の中身で1レベルインデントをつける、と覚えておけばよいでしょう。

try{
  $dbh = new PDO('mysql:host=localhost;dbname=task;charset=utf8',$user,$password);
  if($stmt->fetch()){
    header('Location: index2.php');
  }else{
    header('Location: login.php');
  }
  exit();
}catch(PDOException $e){
  die('エラー');
}


どの try と catch が対応しているのか、
どの if と elseが対応しているのか、
見ただけですぐわかりますよね?

プログラムのif 文等で、実行するコードが1行のみの場合は { } を省略できますが、
実行するコードが条件分岐の中身ということは変わらないので、改行を入れる場合はやはり1レベルインデントをつけます。

if($stmt->fetch())
    header('Location: index2.php');

質問者様のコードですと、例えば

}else{//パスワード、名前を送っていない場合は、html表示


の行なんかどこのif に対応するものか判りにくいですよね。
適切にインデントがつけられていれば、同じ高さにある直前のif または else if を探すだけなので簡単、ということになります。

 インデントの省略について

インデントによる整形ははぱっと見でコードの階層構造を把握できる優れた技術ですが、インデントが深くなりすぎるとコードが横に伸びて見にくくなるという欠点があります。

そのため、階層構造として判りきった部分のインデントを省略する場合があります。

よくあるのが、<html>~</html> 内の head タグと body タグのインデントです。
決まり切った表現で、しかもいずれも1文書中に通常一回しか出現しないので、インデントをつけなくてもすぐわかるだろう、ということです。

<html>
<head>
  <title>ログイン画面</title>
</head>
<body>
  <h1>ログインサイト</h1>
  <h2>ログイン</h2>
  <form action="login.php" method="post"></form>
</body>
</html>

この辺はセンスの問題なので、慣れない内は省略せずインデントをつけておけばよいと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/09/14 11:07

    とてもわかりやすい説明で、僕でも理解することができました。ありがとうございます

    キャンセル

+1

言語問わず「リーダブルコード」という書籍は必読かと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/09/13 16:41

    https://www.amazon.co.jp/dp/4873115655/
    Amazonのリンクを貼っとこうか。アフィなしで。

    キャンセル

  • 2018/09/13 16:43

    補足ありがとうございます。

    キャンセル

+1

スタイルは規約で定め、ツールで保証するものです

規約は、PHPコーディング規約まとめなどPSRをご参考に。
ツールについては他の方の回答にもありますが、エディタの機能で実現しましょう
参考としてPHP コードの整形はプログラマがやるべきことじゃない

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

簡単に言うと"{"が出てきたらインデントが一つ増えて、"}"で減ります。
HTMLの場合はタグの入れ子で。

提示されたコードの場合、ifの次の行のheader()などはインデントが足りないように見えます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

同じタグがついた質問を見る

  • PHP

    24014questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

  • HTML

    11473questions

    HTMLとは、ウェブ上の文書を記述・作成するためのマークアップ言語のことです。文章の中に記述することで、文書の論理構造などを設定することができます。ハイパーリンクを設定できるハイパーテキストであり、画像・リスト・表などのデータファイルをリンクする情報に結びつけて情報を整理します。現在あるネットワーク上のほとんどのウェブページはHTMLで作成されています。

  • トップ
  • PHPに関する質問
  • PHPとHTMLの綺麗な書き方、インデントを意識した書き方というのがいまいち理解できていないです