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

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

新規登録して質問してみよう
ただいま回答率
85.50%
PHP

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

HTML

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

Q&A

1回答

512閲覧

PHPの簡易掲示板での編集機能

mizuiti0616

総合スコア6

PHP

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

HTML

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

0グッド

1クリップ

投稿2018/07/24 07:43

現在PHPとHTMLを使い簡易掲示板を作成しているのですが2点ほど実装できない問題が発生しているので質問させていただきます。

php

1<?php 2$lines = file('keijiban.txt'); 3 4 5if ($_POST['write'] && $_POST['mode'] != 'editmode') { 6 7 //投稿番号をつくる処理 8 $items = explode("\t", $lines); 9 $no = count($lines) + 1 ; 10 11 //データを変数に格納 12 $name = $_POST['name']; 13 $comment = $_POST['comment']; 14 $pass = $_POST['pass']; 15 $time = date("Y/m/d H:i:s"); 16 17 //データを文字列でつくる処理 18 $data = "$no\t$name\t$comment\t$pass\t$time\n"; 19 array_push($lines, $data); 20} 21 22//削除の処理 23if ($_POST['delete']) { 24 for ($i = 0; $i < count($lines); $i++) { 25 $items = explode("\t", $lines[$i]); 26 if ($items[0] == $_POST['delno'] && $items[3] == $_POST['delkey']) { 27 array_splice($lines, $i, 1); 28 } 29 } 30} 31 32//編集ボタンを押したときの処理 33if ($_POST['edit']) { 34 for ($i = 0; $i < count($lines); $i++) { 35 $items = explode("\t", $lines[$i]); 36 if ($items[0] == $_POST['edino'] && $items[3] == $_POST['edikey']) { 37 $edit_editer = $items[0]; 38 $edit_name = $items[1]; 39 $edit_comment = $items[2]; 40 $edit_pass = $items[3]; 41 } 42 } 43} 44 45//編集内容を変更するときの処理 46if ($_POST['mode'] == 'editmode') { 47 for ($i = 0; $i < count($lines); $i++) { 48 $items = explode("\t", $lines[$i]); 49 if ($items[0] == $_POST['edino']) { 50 $items[1] = $_POST['name']; 51 $items[2] = $_POST['comment']; 52 $items[3] = $_POST['pass']; 53 54 $lines[$i] = "$items[0]\t$items[1]\t$items[2]\t$items[3]\t$items[4]\n"; 55 56 } 57 } 58} 59 60 61//書き込みも編集もしないときの処理 62if ($_POST['write'] || $_POST['delete']) { 63 $fp = fopen('keijiban.txt', 'w'); 64 foreach($lines as $line) fputs($fp, $line); 65 fclose($fp); 66} 67 68?> 69 70 71 72<!DOCTYPE html> 73<html lang="ja"> 74<head> 75<meta charset="utf-8"> 76<title>簡易掲示板</title> 77</head> 78<body> 79<form method="post" action=""> 80お名前:<input type="text" name="name" value="<?php echo $edit_name; ?>"><br> 81コメント:<input type="text" name="comment" value="<?php echo $edit_comment; ?>"><br> 82<?php 83if($_POST['edino']){ 84 $edino = $_POST['edino']; 85echo '<input type="hidden" name="mode" value="editmode">'; 86echo '編集番号<input type="text" name="edino" value = echo $edit_editer> '; 87} 88?> 89 90パスワード:<input type="password" name="pass" value="<?php echo $edit_pass; ?>"><br> 91<input type="submit" name="write" value="送信"> 92</form> 93<hr> 94 95<form method="post" action=""> 96削除指定番号:<input type="text" name="delno"> 97 パスワード: <input type="password" name="delkey"> 98 <input type="submit" name="delete" value="削除"> 99</form> 100 101<form method="post" action=""> 102編集指定番号:<input type="text" name="edino"> 103 パスワード: <input type="password" name="edikey"> 104 <input type="submit" name="edit" value="編集"> 105</form> 106<hr> 107 108<?php 109foreach($lines as $line) { 110 $line = trim($line); 111 $items = explode("\t", $line); 112 print "{$items[0]} "; 113 print " $items[1]"; 114 print " &nbsp; "; 115 print "{$items[2]}"; 116 print " {$items[4]}<hr> \n"; 117 118} 119?> 120 121</body> 122</html> 123

実装したい機能は2点です
・編集番号を指定し、編集を押した際に上の編集指定番号に番号が表示されるようにしたい。
・テキストファイルで掲示板の投稿を保存しているが、そちらの方だけ変数と変数の間に<>という区切り文字を入れて保存したい。例:投稿番号<>名前<>コメント<>日時

この2つです。よろしくお願いします><

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

m.ts10806

2018/07/24 08:08

以前の質問での指摘事項がほとんど反映されていません。
退会済みユーザー

退会済みユーザー

2018/07/24 08:59

というか簡易掲示板(PHP)=荒らしじゃ?
guest

回答1

0

パット見で問題点に気づいた。
$no = count($lines) + 1 ;で投稿番号を発番しているようだけど、
投稿を削除したら行数(データ数)も減ってしまうのだけど
そこの仕様はちゃんと考えてあるのだろうかと。
例えば、
1,2,3,4,5番が使われていました、
3番を削除すると1,2,4,5になるけど、行数は4。
4+1で5番を使ってデータを追記すると、
5番のデータが複数存在する状態になるね。
array_splice($lines, $i, 1);なんてコードで配列要素を削除しているんだけど、
そのときに投稿番号を振り直してはいないようだし、
ミスが発生しやすい状況だと思います。
投稿番号を振り直すやり方でやってみるのも一興、
振り直さずに保存しておいて新たな投稿番号を発番するところを工夫するのも一興。

file()関数での読み込み、
fopen()&fputs()&fclose()での書き込み、
なんら排他制御もかかっていないから、
同じタイミングで似たようなアクションがあった場合
簡単にデータファイルが壊れそうですね。

flock()という汎用のファイルロック関数もあるけど、
それを使ったところで読み込みから書き込みまでの一連の処理中に
他のアクションが発生して割り込まない保証もないので、
そういう意味で「簡易掲示板」であって実用性は低いものだと思ってください。

実務的にはトランザクション処理の概念があるデータベースシステムと組み合わせます。

投稿番号は、
データファイルを読み込んで精査して、
例えば使われている投稿番号の最大値+1を使う、
などとすれば、
今読み込んでいるデータの中での投稿番号のダブリは防げそう。
(但しほぼ同じタイミングで似たようなアクションがあった場合においては保証されない。)

データの区切り文字を「<>」にしたいそうだけど、
$data = "$no\t$name\t$comment\t$pass\t$time\n";

$data = "$no<>$name<>$comment<>$pass<>$time\n";
になるだろうし、
$items = explode("\t", $lines[$i]);

$items = explode("<>", $lines[$i]);
になるだろう。

さて、投稿テキスト内に「<>」があった場合どうなるだろうね。
実務に置いては入力されたデータ一つ一つ検証する必要があるよ。
もしも「<>」が入力されたら例えば「<>」に置き換えるようにしようか、
さらに「<>」が入力されたら例えば「&lt;&gt;」に置き換えるようにしようか、
ってところまで考慮しなくちゃいけない。
投稿テキスト内にタブコード\tを含めるのも、<>を含めるのも、
意地悪しようと思えば比較的簡単なことなので
対策を講じる必要があるよ。

さぁがんばって。

投稿2018/07/24 08:15

編集2018/07/24 08:17
退会済みユーザー

退会済みユーザー

総合スコア0

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問