ripple_naip 2012-08-29 13:01:25

[VBA] 何故分けた?? このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

何でひとつの時と複数の時で分かれているかが謎です。 その後、この変数を使うときにもわざわざ1つor複数で分岐してるとか一体何がしたかたんだろうかと・・・・

If eqNum > 1 Then
  ReDim multiEqCD(eqNum - 1)
  '他multiEq~のReDimがいくつか
    For LoopA = 0 To (eqNum - 1)
      Set targetCellB = eqDbSheet.Cells.Find(eqIDs(LoopA), SearchOrder:=xlWhole)
      multiEqCD(LoopA) = equiDbSheet.Cells(targetCellB.Row, 3)
      '他multiEq~の代入文続く
   Next
Else
  Set targetCellB = eqDbSheet.Cells.Find(eqID, SearchOrder:=xlWhole)
  eqCD = equiDbSheet.Cells(targetCellB.Row, 3)
  '他の代入文続く
End If

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(2)

#1 InternalServerE 2012-08-29 15:59:52  

「謎」じゃなく、ウンコ云々を評価するなら、そのメソッドが何をしたいのか理解するのが先じゃないかねぇ。
良く分からないので糞、とか、「俺の価値観」には相容れないから糞、というニュアンスの投稿多いけど、読むほうもいまひとつ共感しづらかったりするよ、うん。

#2 hardtimes777 2012-08-29 17:16:07  

「複数の場合は配列に」という仕様だっただけでは?(その仕様がウンコかどうかは場合によるかと)

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[Java] is禁止令

このエントリーをはてなブックマークに追加

ウンコードの趣旨とは違い、レビューで指摘...

// Mod yamada Start

// 一般的に考えて真偽値を返すメ...

鑑賞する »

[Java] 連番

このエントリーをはてなブックマークに追加

もはや人間が読むものではない。

...

package com.renban.erq053.czp008;

/**...

鑑賞する »

[C] ×有限ループ ◯無限ループ

このエントリーをはてなブックマークに追加

有限ループのはずが、無限ループに... ...

int i,j;
for(i=0;i<100;i++) {
 for(j=0...

鑑賞する »