文字列に直して空白処理をするのはなぜか

Question

前提・疑問

初めて質問させていただきます
プログラミングを始めて2~3ヵ月ほどの初心者でVB.NETが初めての言語になります。

会社で電卓の課題が出て制作しました。
(win10に付属しているようなログが表示されるタイプの電卓です。）
電卓は問題なく動いたのですが、報告したところ

btnResult_Click の

lblAnser.Text = CDec(keep) + CDec(no)

を

Trim(CDec(CDec(keep) + CDec(no)).ToString)

のように文字列にして空白処理をしたほうがいいとご指摘されました。

先輩に事前に見本をつくって頂いてそれを元に作成したのですが、
確かに何か所かTrimが使用されているのは見かけました。(.ToStringは使用されていませんでした。)

そこで疑問です。
・文字列にする必要があるのか
・btnResult_Clickのみを文字列に直す必要があるのか
・この場面で空白処理をするのはどうしてか
の3点が気になりました。

もしわかる方がおられましたら教えてください。
よろしくお願いいたします。

該当のソースコード

'＋の処理
 Private Sub btnplus_Click(sender As Object, e As EventArgs) Handles btnplus.Click
        If door = False Then
            no = lblAnser.Text
            If no = "0" Then
            ElseIf lblLog.Text = "" Then
                keep = no
                lblLog.Text = no & "+"
                no = "0"
            ElseIf sym = "-" Then
                lblAnser.Text = CDec(keep) - CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "+"
                no = "0"
            ElseIf sym = "*" Then
                lblAnser.Text = CDec(keep) * CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "+"
                no = "0"
            ElseIf sym = "/" Then
                lblAnser.Text = CDec(keep) / CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "+"
                no = "0"
            Else
                lblAnser.Text = CDec(keep) + CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "+"
                no = "0"
            End If
        End If
        If door = True Then
            lblLog.Text = Microsoft.VisualBasic.Left(lblLog.Text, Len(lblLog.Text) - 1) & "+"
        End If
        sym = "+"
        door = True
    End Sub

'=の処理
 Private Sub btnResult_Click(sender As Object, e As EventArgs) Handles btnResult.Click
        If door = False Then
            no = lblAnser.Text
            If no = "0" Then
            ElseIf sym = "+" Then
                lblAnser.Text = CDec(keep) + CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "="
                no = "0"
                door = True
            ElseIf sym = "-" Then
                lblAnser.Text = CDec(keep) - CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "="
                no = "0"
                door = True
            ElseIf sym = "*" Then
                lblAnser.Text = CDec(keep) * CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "="
                no = "0"
                door = True
            ElseIf sym = "/" Then
                lblAnser.Text = CDec(keep) / CDec(no)
                keep = lblAnser.Text
                lblLog.Text = lblLog.Text & no & "="
                no = "0"
                door = True
            ElseIf door = True Then
                sym = "="
            End If
        End If
        If door = True Then
            lblLog.Text = Microsoft.VisualBasic.Left(lblLog.Text, Len(lblLog.Text) - 1) & "="
        End If

    End Sub

Comments

[dodox86]

直した方が良いという以下のコードですが、
> Trim(CDec(CDec(keep) + CDec(no)).ToString)

これは、本当ですか？ 　何か雑な感じかつ意味が無い気がします。
CDec(Trim(keep)) + CDec(Trim(no))
とかではありませんか？

それと、VBと言うのは、VB.NETではありませんか？ VB6（古い、Visual Basic 6)やVBAではないのですよね。
実際に、どのような開発環境を使われていますか。

[退会済みユーザー]

Trim(CDec(CDec(keep) + CDec(no)).ToString)
こちらは本当です

VBはVB.NETのことでした。
大変失礼しました。
修正しました。

Answer 1

それは聞く人を間違ってます。
その先輩か、その指摘をしたひとに理由を聞きましょう

Comments

[dodox86]

「質問への追記・修正」へコメントした後、私も同じように思い直しました。

[退会済みユーザー]

確かにその通りですね。ご指摘ありがとうございます。
ご回答ありがとうございました。

[y_waiwai]

まあ、ここでいくらもっともらしい回答を得たところで、それが本当にあなたの会社にとっての理由なのかはわかりませんよね。

Answer 2

先輩に聞きましょうよ。
ここで聞いて答えが得られ、自分の中で解決しても先輩からの評価は変わりません。
逆に先輩に聞くことで、「いいところに気が付いたね」と思われ悪い評価にはならないと思います。

一応わかる範囲で回答もしておきますが。。。

・文字列にする必要があるのか

代入先がlblAnser.Textと明らかにテキスト属性ですから、明示的に文字列にしてあげることを意味しています。
VBの場合、割とルーズでうまいこと処理してくれるので、型をあまり意識せずともうまいこと変換してくれますが、他の言語ではエラーになる確率が高いです。
また明示的に書くということは、プログラマがきちんと意図して書いていることの証ですので、なるべく端折らずに書きましょう。

・btnResult_Clickのみを文字列に直す必要があるのか

前述通りTextに代入するところは同じ考えで直したほうがいいでしょう。

・この場面で空白処理をするのはどうしてか

場合によってはToStringした末尾に空白が入ることがあるかもしれず、それを懸念しているのでしょう。
実験して、この処理の場合には余計な空白が入らないことが確認できればTrimしなくてよいと思います。
ただ目に見えない「空白」は常に意識したほうがよいです。
ToStringに限らず同じような場面は多数あるので、習慣づけておくといいですよ、という先輩なりの知恵なのかもしれません。

Comments

[退会済みユーザー]

ご指摘ありがとうございます。
私は先輩に質問したら評価が下がりそうな気がしてしまったので、こちらで質問させていただきました。
疑問に対する回答を頂けたのでベストアンサーとさせていただきます。
ご回答ありがとうございました。