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

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

ただいまの
回答率

91.03%

  • C#

    5739questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • CSV

    503questions

    CSV(Comma-Separated Values)はコンマで区切られた明白なテキスト値のリストです。もしくは、そのフォーマットでひとつ以上のリストを含むファイルを指します。

CSVとstring[][]型の変換に向くクラス設計の方法は?

解決済

回答 4

投稿

  • 評価
  • クリップ 1
  • VIEW 255

cancat

score 239

こんにちは。 
Windows10でC#のアプリケーションを開発しています。 
Visual Studio 2017 Proを使っています。 

前提・実現したいこと

\r\nと\tで、行とセルに分かれているテキストを構築し編集し、最終的にはstring[][]型にコンバートしたいです。
そのときの型あるいはクラスを何にすればよいか悩んでいます。

試したこと

string[][]は項目の追加が難しいので、List<List<string>>を使ってみました。

該当のソースコード

List<List<string>> lines = new List<List<string>>();
foreach(List<string> line in lines){
 line.Add("A");
 line.Add("B");
 line.Add("C");
}
string[][] resultlines = lines.ToArray();


みたいなかんじ。

これでlinesには複数行があり、lineはcellsであるわけですが、

発生している問題・エラーメッセージ

List<List<string>> lines = new List<List<string>>();
foreach(List<string> line in lines){
 foreach(string cell in line){
  if (cell == "A") cell = "D";
 }
}


みたいに編集すると、
foreach(string cell in line)のところ、lineは単数なのに、実は複数であるところにネーミングの違和感を感じます。

List<List<string>> lines = new List<List<string>>();
foreach(List<string> cells in lines){
 foreach(string cell in cells){
  if (cell == "A") cell = "D";
 }
}


としても、linesのしたにさらにcellsと複数形が来るのが微妙で。

そもそも
List<List<string>>が複雑ですし。
そのくらいならクラス作ったほうがいいでしょうか?
というのが相談です。

補足情報(言語/FW/ツール等のバージョンなど)

Microsoft Visual Studio Pro 2017
Microsoft .NET Framework
Version 4.6.01586

です。 
よろしくお願いします。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 4

checkベストアンサー

+3

こんな感じではどうでしょう?

List<List<string>> table = new List<List<string>>();
foreach(List<string> row in table){
 foreach(string cell in row){
  if (cell == "A") cell = "D";
 }
}


自分なら質問の趣旨から外れますが
1行に対するクラスを作って、そのインスタンスをリストに
持たせる方法を使います。配列は出来るだけ使わない方向にします。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/10 16:37

    ありがとうございます。
    なるほど。そもそも複数形を使わず、table-row-cellと展開すれば、意味的に階層化できるわけですね。
    なるほど。
    1行のクラスを作ってそれでリストですね。それはありかも。
    List<Line> table = new List<Line>();

    Class Line {
    public List<string> cells = new List<string>();
    }
    って感じ?

    キャンセル

+2

Lineに対する、ちょっと面倒な処理をするなら、ノータイムで、Lineクラスを作る話かと。
データと手続きを一緒のところでやる、がオブジェクト指向の基本的な考え方なので。

もちろん、データと手続きがバラバラでもプログラムは作れますけどね。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/10 16:29

    やっぱクラスですね!

    キャンセル

+1

foreach(string cell in line)のところ、lineは単数なのに、実は複数であるところにネーミングの違和感を感じます。 

あるひとつの入れ物(line)に複数の物(cell)が入っているだけなので何も問題ないと思います。

そもそも
List<List<string>>が複雑ですし。
そのくらいならクラス作ったほうがいいでしょうか?
というのが相談です。

List<List<string>>がそんなに複雑かはさておき、
多くの人がやりたがりそうなものは、大抵すでに誰かが作ってます。
CSVの取扱いもすでにライブラリがあります。

たとえば、csvHelper
これを使うか、どうしても自分で作りたい事情があればこれを参考にすればいいと思います。


ついでに:foreach内で列挙変数に代入できません。

List<List<string>> lines = new List<List<string>>();
foreach(List<string> cells in lines){
 foreach(string cell in cells){
  if (cell == "A") cell = "D"; // 不可
 }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/10 16:34

    ありがとうございます。
    自分で作りたくはないのですが、作るのも好きですけど。

    List<List<string>>、複雑じゃないですか?
    >>のあたりで、ちょっとおしりがくすぐったくなっちゃうんですけど。
    わたしの頭がついていっていないだけです。そのうちなれるかな。

    列挙変数は代入できませんでしたね。そうでした。

    キャンセル

+1

同じように行と列をを持つSystem.Data.DataTableを参考にしてはどうでしょうか?
DataTableの場合、全セルの値を利用する場合は以下のようなコードになると思います。

foreach (DataRow row in dataTable.Rows) {
    foreach (DataColumn column in dataTable.Columns) {
        // row[column] を利用
    }
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/10/10 16:29

    ありがとうございます。
    あー。これも考えたのですが、これだとrowとdataTable.Columnsが直接関係なさげに見えて、悩んじゃいました。

    キャンセル

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

  • ただいまの回答率 91.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

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

  • C#

    5739questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • CSV

    503questions

    CSV(Comma-Separated Values)はコンマで区切られた明白なテキスト値のリストです。もしくは、そのフォーマットでひとつ以上のリストを含むファイルを指します。