PHPについて教えてください。

質問

$RSS_data = simplexml_load_file($RSS_url);

foreach($RSS_data->Items->Item as $value){
$img[] = "<img src=\"{$value->Image->URL}\" width=" 200 " height=" 200 ">";
}

for ($i = 0, $n = count($img); $i < $n; $i++) {
if (($i / 6) == 1) {
echo $img[$i] . "<br />";
} else {
echo $img[$i];
}
}

上記の文でパースエラーがでました。エラーのラインは
$img[] = "<img src=\"{$value->SmallImage->URL}\" width=" 200 " height=" 200 ">";
です。
width=" 200 " height=" 200 "　
これを取るとエラーがでずに通ります。画像のサイズを指定したいのですが、このやり方駄目ならどうすればいいのでしょうか？

あと、やりたいことは画像を７つ表示したら改行して、また下に画像を７つ並べることを繰り返す。
というようにしたいのですが、このやり方だと１列目は７つでちゃんと改行されるのですが、２列めからは改行されずに画像が並んでしまいます。

どうすればいいでしょうか？どちらかだけでもいいのでわかる方教えてください。
お願いします。

回答No.1

noname#163110 · ベストアンサー

違っていたらごめんなさい。
if (($i / 6) == 1)→これって割り算でなくて余りで判断ではなかったでしょうか？
あとwidth='200'にしてはどうでしょうか？

回答No.5

bm_hiro

>width=" 200 " height=" 200 "　
>これを取るとエラーがでずに通ります。

良い事を教えましょう。
HTMLタグで 要素をくくるクォートは はしょっても 大概のブラウザは気にせずレンダリングしてくれます。ダメなケースもありますが。
なので ↓ で問題ありません。W3Cには怒られますが。

width=200 height=200

更に言うと HTML5では 公式に はしょってOKになったとかならないとか。

> あと、やりたいことは画像を７つ表示したら改行して、また下に画像を７つ並べることを繰り返す。

<DIV>で囲って CSSで制御すれば良いんじゃないかと思いました。

回答No.4

okbakasine

>width=" 200 " height=" 200 "　
>これを取るとエラーがでずに通ります。
#3の人がすでに書いているけどエスケープ処理って知っている?

回答No.3

ibara994

"～" 文字列リテラルの中で " 文字を記述したい場合は \" と書きます。
$img[] = "<img src=\"{$value->SmallImage->URL}\" width=\"200\" height=\"200\">";

回答No.2

edomin7777

取り敢えず、

if (($i / 6) == 1) {

を

if (($i % 7) == 6) {

に変えてみる？

$iが
0の時0←画像を出力
1の時1←画像を出力
2の時2←画像を出力
3の時3←画像を出力
4の時4←画像を出力
5の時5←画像を出力
6の時6←画像を出力して改行
7の時0←画像を出力
8の時1←画像を出力
…

とか…。