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

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

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

VBScript(Visual Basic Scripting Edition)はMicrosftが開発したスクリプト言語であり、Visual Basicのサブセットです。

Q&A

解決済

2回答

1401閲覧

CSVファイルの繰り返し処理で、処理済の行のデータを削除したい。

SOHR

総合スコア2

VBScript

VBScript(Visual Basic Scripting Edition)はMicrosftが開発したスクリプト言語であり、Visual Basicのサブセットです。

0グッド

0クリップ

投稿2020/06/16 01:46

VBS初心者です。
csv処理を作っているのですが、調べても実現できない状況のため、解決策やアドバイスをご教示いただきたく、質問致します。

実現したいこと

次のようなCSVファイルがあります。

日付,場所,天気
5/1,東京,晴れ
6/2,千葉,くもり
7/7,神奈川,雨

1.1行目を読み飛ばす。
2.2行目以降を1行ずつ読込、配列にして必要な部分だけを置換処理する。
晴れ→1 くもり→2 雨→3
3.別のcsvファイルに書き込む。

期待する結果

5/1,東京,1
6/2,千葉,2
7/7,神奈川,3

実行して得られる結果

5/1,東京,1
5/1,6/2,東京,千葉,1,2
5/1,6/2,7/7,東京,千葉,神奈川,1,2,3

DO LOOPで繰り返していく中で、処理済の行の配列?まで次の行で書き込まれるようになってしまいます。

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

エラーメッセージは出ません。

該当のソースコード

VBScript

1Option Explicit 2 3Dim objFS 4Dim objRFile 5Dim objWFile 6Dim strReadFile 7Dim strWriteFile 8Dim strLine 9Dim AryStrings 10Dim i 11Dim str(2) 12Dim objRegExp 13Dim strRep(2) 14 15strReadFile = "C:\Users\Desktop\read.csv" '元データファイルを開く 16strWriteFile = "C:\Users\Desktop\write.csv" '生成データファイル 17 18Set objFS = Wscript.CreateObject("Scripting.FileSystemObject") 'ファイルシステムオブジェクトをつくる 19Set objRFile = objFS.OpenTextFile(strReadFile) '元データファイルを開く 20Set objWFile = objFS.OpenTextFile(strWriteFile,8,true) '生成データファイルの末尾に追加書き込みをする 21Set objRegExp = New RegExp '正規表現による検索や置換を行うためのオブジェクト 22 23strLine = objRFile.SkipLine '1行目を読み飛ばす 24 25Do until objRFile.AtEndOfStream '最終行を読み込んだら繰り返し終了 26 strLine = objRFile.ReadLine '次の行を読み込む 27 AryStrings = Split(strLine,",") 'カンマ区切りの文字列を配列にする 28 For i = 0 to 2 29 '日付をそのまま入力 30 If i = 0 Then 31 str(i) = str(i) & AryStrings(i) & "," 32 End If 33 '場所をそのまま入力 34 If i = 1 Then 35 str(i) = str(i) & AryStrings(i) & "," 36 End If 37 '天気を番号に変換する 38 If i = 2 Then 39 objRegExp = "晴れ" 40 If AryStrings(i) = objRegExp Then 41 strRep(i) = Replace(AryStrings(i),"晴れ","1") 42 End If 43 objRegExp = "くもり" 44 If AryStrings(i) = objRegExp Then 45 strRep(i) = Replace(AryStrings(i),"くもり","2") 46 End If 47 objRegExp = "雨" 48 If AryStrings(i) = objRegExp Then 49 strRep(i) = Replace(AryStrings(i),"雨","3") 50 End If 51 str(i) = str(i) & strRep(i) & "," 52 End If 53 'ファイルに書き込み 54 objWFile.Write(str(i)) 55 Next 56Loop 57 58objRFile.Close 59objWFile.Close 60 61Set objFS = Nothing 62Set strReadFile = Nothing 63Set strWriteFile = Nothing 64Set objRFile = Nothing 65Set objWFile = Nothing 66Set strLine = Nothing 67Set AryStrings = Nothing 68Set i = Nothing 69Set str(2) = Nothing 70Set objRegExp = Nothing 71Set strRep(2) = Nothing

何卒、よろしくお願い致します。

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

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

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

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

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

guest

回答2

0

Do Untilのループ内で、配列strRepが初期化されないので、延々と文字列追加が続いてしまうからですね。
そもそもstrRepに一旦格納する必要ないと思います。
CSVの3カラム目のみが変換対象ですし、折角作成した正規表現オブジェクトが活用できてないのでこれを利用して、置換してしまえば良いでしょう。

VBScript

1Option Explicit 2 3Dim objFS 4Dim objRFile 5Dim objWFile 6Dim objRegex1 7Dim objRegex2 8Dim objRegex3 9Dim strLine 10 11Set objRegex1 = New RegExp 12Set objRegex2 = New RegExp 13Set objRegex3 = New RegExp 14objRegex1.Pattern = "晴れ$" 15objRegex2.Pattern = "くもり$" 16objRegex3.Pattern = "雨$" 17 18Set objFS = Wscript.CreateObject("Scripting.FileSystemObject") 19Set objRFile = objFS.OpenTextFile("hogehoge.csv") 20Set objWFile = objFS.OpenTextFile("fugafuga.csv", 8, true) 21 22strLine = objRFile.SkipLine 23 24Do until objRFile.AtEndOfStream 25 strLine = objRFile.ReadLine 26 strLine = objRegex1.Replace(strLine, "1") 27 strLine = objRegex2.Replace(strLine, "2") 28 strLine = objRegex3.Replace(strLine, "3") 29 objWFile.WriteLine(strLine) 30Loop 31

オブジェクトの後始末とか端折ってますが、、、

入力ファイル(hogehoge.csv)

日付,場所,天気 5/1,東京,晴れ 6/2,千葉,くもり 7/7,神奈川,雨

出力ファイル(fugafuga.csv)

5/1,東京,1 6/2,千葉,2 7/7,神奈川,3

投稿2020/06/16 02:35

DreamTheater

総合スコア1095

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

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

SOHR

2020/06/16 06:02

ご回答ありがとうございます。 strRepの初期化がポイントだったのですね。その前のAryStringsの初期化を試してみてうまくいかず、今回質問した次第でした。 変換するカラムが固定なので、ご教示いただいたコードでうまくいきました。 改めて御礼申し上げます。
guest

0

ベストアンサー

VBA

1str(i) = str(i) & AryStrings(i) & ","

と継ぎ足してしまっているのが問題ですね。
あと折角配列にしているので、出力したデータをだけ集めて最後にJoinすると楽です。
あと正規表現で検索する必要性もないですね。
あとループで回す必要もないです(この例に限りますが)
というのを踏まえてDo~Loop内を書き換えてみました。
動作確認はしていませんのであしからず。

VBA

1Do until objRFile.AtEndOfStream '最終行を読み込んだら繰り返し終了 2 strLine = objRFile.ReadLine '次の行を読み込む 3 AryStrings = Split(strLine,",") 'カンマ区切りの文字列を配列にする 4 5 '日付をそのまま入力 6 str(0) = AryStrings(0) 7 8 '場所をそのまま入力 9 str(1) = AryStrings(1) 10 11 '天気を番号に変換する 12 Select Case AryStrings(2) 13 Case "晴れ" 14 str(2) = "1" 15 Case "くもり" 16 str(2) = "2" 17 Case "雨" 18 str(2) = "3" 19 Case Else 20 str(2) = "" 21 End Select 22 23 'ファイルに書き込み 24 objWFile.Write(Join(str, ",")) 25Loop 26

投稿2020/06/16 02:31

ttyp03

総合スコア16998

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

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

SOHR

2020/06/16 06:05

ご回答ありがとうございます。 str(i)の継ぎ足しが反映されていることに気づきませんでした。 こちらをstr(i) = AryStrings(i) & "," にするだけでも解消しました。 (末尾の & "," がないと、カンマ区切りの書き込みにならなかったので付け足しています。) また、Select Case を使っての置換は知りませんでしたので、大変勉強になりました。 今後の参考にさせていただきます。
ttyp03

2020/06/16 06:12

Select Case のは置換ではないですよ(文字列の一部を置き換えているわけではない)。 あくまでも出力するデータを「晴れ」なら「1」のようにしているだけです。 「今日は晴れ」みたいな文字列の「晴れ」を「1」に置き換える場合は置換処理が必要になりますが、今回はそのような必要はなかったというだけです。
SOHR

2020/06/17 07:51

丁寧なご教示ありがとうございます。 まだまだ勉強不足ですが、学んだことを以降のケースで活用していけるよう精進します。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.47%

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

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

質問する

関連した質問