C#です。if文、HTMLの間違いがわかりません。

Question

プログラミング初心者です。
VS2010でHTMLファイルに出力するC#でプログラミングをかいています。
ですが希望の表にならず、また何が違うのかわからないです。
現在下のコードを実行するとこのようになります。

これを色の～市の縦の行の空欄をなくし、横にはみ出てしまっている記号を同じ市の行に1列に収めたいです。
if文がおかしいのか。HTMLの書き方がおかしいのか、アドバイスお願いします。

コード
 sw.Write("<table border=1>");
                    sw.Write("<tr><td>     </td><td>暴風雪</td><td>大雨</td><td>暴風</td><td>大雪</td><td>波浪</td><td>高潮</td><td>洪水</td><td>雷</td><td>融雪</td>");
                    sw.Write("<td>濃霧</td><td>乾燥</td><td>なだれ</td><td>低温</td><td>霧</td><td>着雪</td><td>着氷</td></tr>");//表のヘッダー

//ここにforeach,seitch文がはいりboufuusetsuやooameに記号が入る処理があります
//下のif文はforeachの中です。

 if (code == oldCode)
                            {
                           
                            sw.Write("<td>"+ boufuusetsu +"</td>");
                            sw.Write("<td>" + ooame + "</td>");
                            sw.Write("<td>" + bouhuu + "</td>");
                            sw.Write("<td>" + ooyuki + "</td>");
                            sw.Write("<td>" + haro + "</td>");
                            sw.Write("<td>" + takashio + "</td>");
                            sw.Write("<td>" + kouzui + "</td>");
                            sw.Write("<td>" + kaminari + "</td>");
                            sw.Write("<td>" + yuusetsu + "</td>");
                            sw.Write("<td>" + noumu + "</td>");
                            sw.Write("<td>" + kanso + "</td>");
                            sw.Write("<td>" + nadare + "</td>");
                            sw.Write("<td>" + teion + "</td>");
                            sw.Write("<td>" + shimo + "</td>");
                            sw.Write("<td>" + chakuhyo + "</td>");
                            sw.Write("<td>" + chakusetsu + "</td>");
                            
                            oldCode = code;

                            }
                        else
                        {
                            sw.Write("<tr><td>" + name +"</td><td>"+ boufuusetsu +"</td>");
                            sw.Write("<td>" + ooame + "</td>");
                            sw.Write("<td>" + bouhuu + "</td>");
                            sw.Write("<td>" + ooyuki + "</td>");
                            sw.Write("<td>" + haro + "</td>");
                            sw.Write("<td>" + takashio + "</td>");
                            sw.Write("<td>" + kouzui + "</td>");
                            sw.Write("<td>" + kaminari + "</td>");
                            sw.Write("<td>" + yuusetsu + "</td>");
                            sw.Write("<td>" + noumu + "</td>");
                            sw.Write("<td>" + kanso + "</td>");
                            sw.Write("<td>" + nadare + "</td>");
                            sw.Write("<td>" + teion + "</td>");
                            sw.Write("<td>" + shimo + "</td>");
                            sw.Write("<td>" + chakuhyo + "</td>");
                            sw.Write("<td>" + chakusetsu + "</td></tr>");
                            //oldCode = code;
                        }
                        oldCode = code;
                    }
                    sw.Write("</table>");   

Answer 1

市が空白の行は上の行の市の行に合わせこんで出したいのでしょうか？
HTMLに吐き出す処理より前に
市ごとに（一行ごとのデータ）を作りこんでから、
HTMLに書き出してはどうでしょうか？

Comments

[pro-poke5]

ありがとうございます

はい、市が空白の行は上の行に1列に表示させたいです。

データを作りこむということは、新しくtablenameなどの変数を作るということでしょうか？

[KIYOSHI]

変数名を書かれても全体の作りがわからないのでtablenameが何なのかわかりませんが。
HTMLを作る際に、一度吐き出した行に書き込むことはできませんので、
一行ごとに、市ごとに複数データ行を取得しているということであれば、
市ごとに各データをまとめるといいのではないのでしょうか。

[pro-poke5]

ありがとうございます
一度出した行に書きこむことができないのですね

市ごとにまとめるというのはif文の中で処理すべきでしょうか？
それともif文の前でやるべきなのでしょうか？

[KIYOSHI]

書き出すデータをForかForeachでループさせて書き出していると思います。
その前に、一度ループさせて一行ごとのデータを作成するといいと思います。

Answer 2

HTMLは、タグが「入れ子」構造になります。

表(２行２列)の場合のHTMLは以下のようになります。

<Table><!-- 表の始まり -->
  <Tr><!-- １行目の始まり -->
    <td> <!-- １項目目の始まり -->
   １行目の１項目の内容
    </td><!-- １項目目の終わり -->
    <td> <!-- ２項目目の始まり -->
   １行目の２項目目の内容
    </td><!-- ２項目目の終わり -->
  </Tr><!-- １行目の終わり -->
  <Tr><!-- ２行目の始まり -->
    <td> <!-- １項目目の始まり -->
   ２行目の１項目の内容
    </td><!-- １項目目の終わり -->
    <td> <!-- ２項目目の始まり -->
   ２行目の２項目目の内容
    </td><!-- ２項目目の終わり -->
  </Tr><!-- ２行目の終わり -->
</Table><!-- 表の終わり -->

これを理解した上で、質問のコードを見てみると以下の問題があることが判ると思います。

１）Tableタグの後ろの">"が抜けている。
２）ifのブロックの中のコードでは、<Tr>,</Tr>のタグが出力されていない。（elseのブロックの中のコードでは、<Tr>,</Tr>が出力されていて、こちらは正しい）
３）ifのブロックの中のコードでは、項目(<Td>から</td>まで）が１６個出力されているが、elseのブロックの中のコードでは項目が１７個出力されている。 ifのブロックの中のコードで「～市」に相当する項目(name)を出力し忘れているのではないか。

１列目（～市）が空欄になっているのは、３）が原因。
横にはみ出すのは、２）が原因。（行になっていないので、改行が起きず、後ろに伸びてゆく）
だと思います。

Comments

[pro-poke5]

ありがとうございます

oldCodeとcodeが同じなら同じ行に表示したかったのでnameは出力しないほうがいいと考えました
またifで<tr></tr>、nameを追加するとelse文と同じ内容･･･ということでしょうか？

[coco_bauer]

「oldCodeとcodeが同じなら同じ行に表示したかった」というのが、複数のデータを重ね合わせたような行を作りたかったという事なのであれば、複数のデータが区別できるようになっていなければなりませんが、質問のコードは、そうなっていません。大雨の情報として、ooameひとつではなく、ooameOld, ooameNewがあって、これらを一つにまとめたものを出力するというのなら、１行にまとめるという事が可能になろうかと思われます。

Answer 3

<table border=1 の閉じ括弧がないのは単純なミスでしょうか？

また、code == oldCodeの場合に<tr>がないように見受けられます。

生成されたHTMLのソースを見てみると結構わかりやすいかもしれません。
http://openlab.ring.gr.jp/k16/htmllint/htmllintl.html
のようなチェッカーもありますので利用してみてはいかがでしょうか～。

Comments

[pro-poke5]

閉じカッコがないのは単純なコピペミスです、すみません

code == old Codeの場合に<tr>をつけると
横にはみ出るのはなくなりますが
縦の市の列に空欄ができ、
------------ヘッダー-----------
A市　　　△　　　　　×
　　　　　　　　　○
　　　　　　　　　　　　　　　　　　　　　○
B市　…

となってしまいまして…
1行にすることはできないのでしょうか？

[s.t.]

code == oldCodeの場合にカラムの数が1個足りていないのではないでしょうか？
ヘッダが17列であるのに対してcode == oldCodeの列は16列しかありません。

[s.t.]

あと、個人的主観ですが、他言語からHTMLソースを生成する際は、
HTMLが見やすくなるソースコードにしたほうがよいかもしれません。
データ部のほうはできているので、ヘッダのほうもあわせてあげると見やすくなりますよ。

例えば、
sw.Write("<tr>");
sw.Write("<td> </td>");
sw.Write("<td>暴風雪</td>");
sw.Write("<td>大雨</td>");
・・・
sw.Write("</tr>");
のような感じです。