レッスン-投稿一覧パソコン仕事５倍塾ポータル

投稿/コメントを表示します。

凜さんの投稿

(投稿ID: 5487)

いつもお世話になっております。
動画で教えて頂いた1文字ずつ検索する方法はわかりやすく、ForNext構文が工夫次第で活用の幅が広がる事に驚きと楽しみを感じています。

いつも動画を見る前に自分で挑戦している為、今回は別のアプローチのプログラムができました。
動作として問題はなさそうなのですが、書き方に工夫ができるのではないかと考えております。
「サブプロシージャの分割と呼び出し」を活かせないか考えてみたのですが、変数を含むと上手くいきません。
下記、6行が7回も繰り返されるのですがすっきりと書ける方法がありましたら、ご教示ください。
(問題と転記先が違ったり、連番のプログラムが抜けていたりするのですが、そこはご容赦ください)

            Range("I" & tenki).Value = Range("A" & gyo).Value
            Range("J" & tenki).Value = Range("B" & gyo).Value
            Range("K" & tenki).Value = Range("C" & gyo).Value
            Range("L" & tenki).Value = Range("D" & gyo).Value
            Range("N" & tenki).Value = Range("F" & gyo).Value
            Range("O" & tenki).Value = Range("G" & gyo).Value

----

Sub mondai()
    Dim yakuwari
    Dim tenki
    Dim ten1
    Dim ten2
    Dim ten3
    Dim gyo
    tenki = 2
    
    For gyo = 2 To 7
        yakuwari = Range("E" & gyo).Value
        ten1 = InStr(yakuwari, "、")
        ten2 = InStr(ten1 + 1, yakuwari, "、")
        ten3 = InStr(ten2 + 1, yakuwari, "、")
        
        If ten1 = 0 Then
            Range("M" & tenki).Value = yakuwari
            Range("I" & tenki).Value = Range("A" & gyo).Value
            Range("J" & tenki).Value = Range("B" & gyo).Value
            Range("K" & tenki).Value = Range("C" & gyo).Value
            Range("L" & tenki).Value = Range("D" & gyo).Value
            Range("N" & tenki).Value = Range("F" & gyo).Value
            Range("O" & tenki).Value = Range("G" & gyo).Value
    
            tenki = tenki + 1
        Else
            Range("M" & tenki).Value = Mid(yakuwari, 1, ten1 - 1)
            Range("I" & tenki).Value = Range("A" & gyo).Value
            Range("J" & tenki).Value = Range("B" & gyo).Value
            Range("K" & tenki).Value = Range("C" & gyo).Value
            Range("L" & tenki).Value = Range("D" & gyo).Value
            Range("N" & tenki).Value = Range("F" & gyo).Value
            Range("O" & tenki).Value = Range("G" & gyo).Value
            
            tenki = tenki + 1
                If ten2 = 0 Then
                    Range("M" & tenki).Value = Mid(yakuwari, ten1 + 1)
                    Range("I" & tenki).Value = Range("A" & gyo).Value
                    Range("J" & tenki).Value = Range("B" & gyo).Value
                    Range("K" & tenki).Value = Range("C" & gyo).Value
                    Range("L" & tenki).Value = Range("D" & gyo).Value
                    Range("N" & tenki).Value = Range("F" & gyo).Value
                    Range("O" & tenki).Value = Range("G" & gyo).Value
                    
                    tenki = tenki + 1
                Else
                    Range("M" & tenki).Value = Mid(yakuwari, ten1 + 1, ten2 - ten1 - 1)
                    Range("I" & tenki).Value = Range("A" & gyo).Value
                    Range("J" & tenki).Value = Range("B" & gyo).Value
                    Range("K" & tenki).Value = Range("C" & gyo).Value
                    Range("L" & tenki).Value = Range("D" & gyo).Value
                    Range("N" & tenki).Value = Range("F" & gyo).Value
                    Range("O" & tenki).Value = Range("G" & gyo).Value
                    
                    tenki = tenki + 1
                    If ten3 = 0 Then
                        Range("M" & tenki).Value = Mid(yakuwari, ten2 + 1)
                        Range("I" & tenki).Value = Range("A" & gyo).Value
                        Range("J" & tenki).Value = Range("B" & gyo).Value
                        Range("K" & tenki).Value = Range("C" & gyo).Value
                        Range("L" & tenki).Value = Range("D" & gyo).Value
                        Range("N" & tenki).Value = Range("F" & gyo).Value
                        Range("O" & tenki).Value = Range("G" & gyo).Value
                        
                        tenki = tenki + 1
                    Else
                        Range("M" & tenki).Value = Mid(yakuwari, ten2 + 1, ten3 - ten2 - 1)
                        Range("I" & tenki).Value = Range("A" & gyo).Value
                        Range("J" & tenki).Value = Range("B" & gyo).Value
                        Range("K" & tenki).Value = Range("C" & gyo).Value
                        Range("L" & tenki).Value = Range("D" & gyo).Value
                        Range("N" & tenki).Value = Range("F" & gyo).Value
                        Range("O" & tenki).Value = Range("G" & gyo).Value
                        
                        tenki = tenki + 1
                        Range("M" & tenki).Value = Mid(yakuwari, ten3 + 1)
                        Range("I" & tenki).Value = Range("A" & gyo).Value
                        Range("J" & tenki).Value = Range("B" & gyo).Value
                        Range("K" & tenki).Value = Range("C" & gyo).Value
                        Range("L" & tenki).Value = Range("D" & gyo).Value
                        Range("N" & tenki).Value = Range("F" & gyo).Value
                        Range("O" & tenki).Value = Range("G" & gyo).Value
                        
                        tenki = tenki + 1
                    
                    End If
                End If
        End If
    Next
    

End Sub

2023/07/04 22:03

小川慶一さんのコメント

(コメントID: 8196)

おはようございます。

横への連続移動の方法のひとつとして、 .offset を使うやり方があります。

Sub hoge()
    Dim moji
    moji = "あいうえおかきくけこさしすせそ"
    
    Dim i
    For i = 1 To Len(moji)
        Range("A1").Offset(0, i).Value = Mid(moji, i, 1)
    Next
End Sub

2023/07/05 01:57

凜さんのコメント

(コメントID: 8197)

新しい知識が必要だったんですね。
これまでご指導頂いているように同じことを繰り返し書かず、変数を使ってシンプルにより見やすく書くことで全体の作りもわかりやすく、間違えている部分の発見も容易になりいいこと尽くしだと思い、もっと見やすい書き方はないかと探りすぎました。

今回、教えて頂いたOffsetを使いなんとか形にできましたが・・すごく難しかったです。
練習を繰り返し、活用できるように頑張ります！
ありがとうございました。

Sub mondai()
    Dim yakuwari
    Dim tenki
    Dim ten1
    Dim ten2
    Dim ten3
    Dim moji_moto
    Dim moji_saki
    Dim gyo
    Dim retsu_moto
    tenki = 2
    moji_moto = "ABCDF"
    moji_saki = "IJKLN"
    
    For gyo = 2 To 7
        yakuwari = Range("E" & gyo).Value
        ten1 = InStr(yakuwari, "、")
        ten2 = InStr(ten1 + 1, yakuwari, "、")
        ten3 = InStr(ten2 + 1, yakuwari, "、")
        
        If ten1 = 0 Then
            Range("M" & tenki).Value = yakuwari
            For retsu_moto = 0 To Len(moji_saki) - 1
                Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
            Next
    
            tenki = tenki + 1
        Else
            Range("M" & tenki).Value = Mid(yakuwari, 1, ten1 - 1)
            For retsu_moto = 0 To Len(moji_saki) - 1
                Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
            Next
            
            tenki = tenki + 1
                If ten2 = 0 Then
                    Range("M" & tenki).Value = Mid(yakuwari, ten1 + 1)
                For retsu_moto = 0 To Len(moji_saki) - 1
                    Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
                Next
            
                    tenki = tenki + 1
                Else
                    Range("M" & tenki).Value = Mid(yakuwari, ten1 + 1, ten2 - ten1 - 1)
                For retsu_moto = 0 To Len(moji_saki) - 1
                    Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
                Next
                    
                    tenki = tenki + 1
                    If ten3 = 0 Then
                        Range("M" & tenki).Value = Mid(yakuwari, ten2 + 1)
                        For retsu_moto = 0 To Len(moji_saki) - 1
                            Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
                        Next
                        
                        tenki = tenki + 1
                    Else
                        Range("M" & tenki).Value = Mid(yakuwari, ten2 + 1, ten3 - ten2 - 1)
                        For retsu_moto = 0 To Len(moji_saki) - 1
                            Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
                        Next
                                    
                        tenki = tenki + 1
                        Range("M" & tenki).Value = Mid(yakuwari, ten3 + 1)
                        For retsu_moto = 0 To Len(moji_saki) - 1
                            Range(Mid(moji_saki, retsu_moto + 1, 1) & tenki).Offset(0, 0).Value = Range(Mid(moji_moto, retsu_moto + 1, 1) & gyo).Offset(0, 0).Value
                        Next
                        
                        tenki = tenki + 1
                    
                    End If
                End If
        End If
    Next
    
End Sub

2023/07/06 21:31

小川慶一さんのコメント

(コメントID: 8198)

もともと、基礎編フォローアップアドバンスドは、基礎編で提供された道具だけで腕力で問題解決しましょうというコンセプトの演習シリーズです。
この課題も発展編1, 発展編2で学ぶ手法を使うともっとすっきり解決できます。

これだけのことをやりきれる腕力がついたのでしたら、発展編1に進まれると良いかと思います v(^^*

2023/07/06 22:10

凜さんのコメント

(コメントID: 8199)

ありがとうございます！すごく嬉しいです！！
引き続きよろしくお願い致します。

2023/07/07 12:19