上下A列を比較し、同じであればA列に紐づくB列の小計を累計しC列に出力出来るようにしたいです。

Question

###前提・実現したいこと
上下A列を比較し、同じであればA列に紐づくB列の小計を累計しC列に出力出来るようにしたいです。
＊写真参照お願いします。
(例)PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラーメッセージ

###該当のソースコード```
Sub macro1()

Dim color As Integer
Dim flg As Integer
Dim row As Long
Dim i As Long
Dim p_row_value As String
Dim n_row_value As String
Dim s_row As Long
Dim e_row As Long
        
p_row_value = ""
n_row_value = ""
color = 0
flg = 0
        
For row = 1 To Cells(Rows.Count, 1).End(xlUp).row
    n_row_value = Cells(row, 1).Value
            
    If n_row_value = p_row_value Then
        If flg = 0 Then
           color = (color + 1) Mod 2
           s_row = row - 1
        End If
            
    flg = 1
    Cells(row, 3) = Cells(row - 1, 2) + Cells(row, 2)
    Cells(row - 1, 1).Interior.color = f_color(color)
    Cells(row, 1).Interior.color = f_color(color)
            
    Else
        flg = 0
    End If
            
    p_row_value = n_row_value
        
Next row

End Sub

Function f_color(flg As Integer) As Long
If flg = 0 Then
f_color = RGB(255, 0, 0)
Else
f_color = RGB(255, 255, 0)
End If

End Function

###試したこと
試して見るもののうまく行かず。。。

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

Comments

[hatena19]

テンプレートの元から記述してある例文部分は削除してください。コードは、コードタグで囲ってください。コードを選択して、</>ボタンをおせば自動でタグが挿入されます。

Answer 1

処理の流れを１行ずつ追っていくとわかりますが、問題はC列に入力している値が常に（前行のB列の値）+（今回行のB列の値）となっていることです。
今回行いたいことはA列に同じ値が続く限り累計していく計算だと思いますが、これでは直近２行の合計しか計算できません。

またA列が連続していないときにC列が出力されていないという問題もあるようです。

というわけで、改修点は以下になります。
①A列が前行と異なる場合、C列には今回行のB列の値を出力する。
②A列が前行と同じ場合、前行のC列の値（累積値）に今回行のB列の値を加算してC列に出力する。
③A列が前行と同じ場合、前行のC列の値を空にする。

以上です。

コードの改修箇所は、

        If n_row_value = p_row_value Then
            If flg = 0 Then
                color = (color + 1) Mod 2
                s_row = row - 1
            End If
        
            flg = 1
            'Cells(row, 3) = Cells(row - 1, 2) + Cells(row, 2) 
            Cells(row, 3) = Cells(row - 1, 3) + Cells(row, 2)   '前行のC列の値に今回行B列の値を加算する
            Cells(row - 1, 3) = ""                              '前行のC列の値はクリアする
            Cells(row - 1, 1).Interior.color = f_color(color)
            Cells(row, 1).Interior.color = f_color(color)
        
        Else
            flg = 0
            Cells(row, 3) = Cells(row, 2)                       '前行A列の値と異なる場合、今回行B列の値をC列に出力する
        End If

というかんじになります。

処理の流れをひとつずつ整理して追っていけば、思い通りいかない原因が見えてきます。
デバッグ機能などを活用してコードの問題点を読み取れるよう、がんばってください。

Comments

[chanken]

返信おくれました！！ご丁寧にここまでありがとうございます。一歩ずつ精進して行きます。

Answer 2

どうもロジックがよくないですね
どのように処理の流れを組み立てましたか？それを質問に追記してみてください

プログラムは書いた通りにしか動かないので、その元になる設計がうまく動くよう考えられているかどうかは重要です
設計がうまく動くようできているのに結果が出ない場合は、コードに問題があります
設計ができていなければ、そもそも動きません

Comments

[chanken]

返信遅れました。アドバイスありがとうございます。次回からは処理の流れも添えて質問するようにします！