Excel VBA 質問スレ Part50 [無断転載禁止]©2ch.net

2017/08/27(日) 12:40:17.57

!extend:checked:vvvvv:1000:512

スレ立ての際は一行目に
!extend:checked:vvvvv:1000:512
と入れてスレ立てして下さい

ExcelのVBAに関する質問スレです
コード書き込みや作成依頼もOK

※関連スレ
VBAなんでも質問スレ Part2
http://mevius.2ch.net/test/read.cgi/tech/1432173164/
Access VBA 質問スレ Part1
http://mevius.2ch.net/test/read.cgi/tech/1328536426/
Excel総合相談所 126
https://mevius.2ch.net/test/read.cgi/bsoft/1496487719/

※前スレ
Excel VBA 質問スレ Part49©2ch.net
http://mevius.2ch.net/test/read.cgi/tech/1498410914/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured

2017/09/07(木) 20:45:36.76

キーボードマクロ

2017/09/07(木) 21:03:02.57

そら、Recordset 消してるだけだしな　毎回Recordset 消してどうしたいねん　っていう
Save せんでええのんか　っていう

2017/09/07(木) 22:52:40.38

select * なんてせずに使いたい行だけselect するってのはダメなんかい

2017/09/07(木) 22:53:51.32

あっ、行じゃなくフィールドね

2017/09/07(木) 22:58:55.63

>>42
選べよw

2017/09/07(木) 23:53:31.42

>>45
Save メソッドですか？

Recordset をファイルまたは Stream オブジェクトに保存します。

とMSDNに書いてあるから、やりたいこととは少し違う気がします
とはいえ、削除をコミットしたいという感覚は間違いありません

>>46
>>42のSQLはあくまでサンプルですが、SQL文そのものはインプットなので、編集はできない状態だと考えていただきたいです
その上で結果のレコードセットを編集して出力することを考えています

2017/09/08(金) 08:13:46.68

削除クエリー走らせるとか

2017/09/08(金) 09:31:09.93

>>43
マウスを動かしてクリック

2017/09/08(金) 09:37:36.00

>>42
こういうことではないのか？
http://www.accessclub.jp/vbakaisetu/37.html

2017/09/08(金) 11:23:59.37

>>52
レコードセットから列を削除したいのです
やりたいことはこんな感じです

'1.クエリの実行
Set rs = cn.Execute("select A,B,C from foo")

'2.レコードセットから列Bを削除する
rs.Fields.Delete("B")

'3.残った列A,CをA1セル起点に貼り付ける
Range("A1").CopyFromRecordset rs

このとき、Fields.Delete メソッドはレコードセットが開いているとエラーになってしまいます
なので、>>42のように前後にClose、Openで挟んだのですが、Open実行時に削除したはずの列が復活してしまいます

2017/09/08(金) 12:48:46.68

>>53
> レコードセットから列を削除したいのです
取得しなきゃいいだけだろ...

'1.クエリの実行
> Set rs = cn.Execute("select A,C from foo")

28 (ｱｳｱｳｶｰ Sa43-G+Vk) · 2017/09/08(金) 12:57:54.99

>>54
>>49でSQLは編集出来ないということらしい。

なんか、物凄く特殊なことしてるね。
レコード数から2次元配列作って1件ずつ入れてくといった泥臭い方法ぐらいしか無いんかね。

2017/09/08(金) 15:26:46.89

>>49
そのインプットとやら
に工夫施す

2017/09/08(金) 18:09:07.35

>>53
3貼り付けがAとCだけにすれば2は要らないのでは？

2017/09/08(金) 18:32:11.80

初めからＢをselectしなければ良いだけだけど、それが無理なら
張りつけてからＢ列消せば良いんじゃね

2017/09/08(金) 20:18:25.34

select A,C,B　のほうがよさげ

2017/09/08(金) 21:03:05.17

rsからrs2を作る

2017/09/08(金) 21:20:35.90

>>55
> SQLは編集出来ないということらしい。
ああ、なるほど見落としてたわ
なら俺も泥臭い方法かSQLをパースして特定の列を抜いたSQLを組み立てるとかのトリッキーな方法しか思い付かんわ

42 (ｻｻｸｯﾃﾛﾚ Sp3b-r1nS) · 2017/09/08(金) 23:59:54.71

みなさんレスありがとうございます

>>55
やっぱりそうするしかないのでしょうかね
いったん配列に落とすとセルに貼り付けた時データ型やレコード数なども気にしなきゃならなくなるので、できればレコードセットのままなんとかしたいのですが…

>>57
そうなんだろうけど、CopyFromRecordsetだと全フィールド張り付いてしまいます
ループで1セルずつ貼り付けて行くと、>>55の言うような「泥臭い方法」と変わらなくなってしまいます

>>58
なるほど、レコードセットでどうにかできなければそれが一番いい方法かもしれません

>>60
それも考えましたが、rs2に複製の際にエラーが出てしまいました
今は手元に環境がないので、エラーの内容についてはまたの機会にします

>>61
SQLは select * になるかもしれないし、予想がつきません

2017/09/09(土) 01:05:27.10

どういうＳＱＬが来るかも分からないのに特定の列を消す必要があるという事だけは分かるのか

2017/09/09(土) 01:14:41.31

文章の中から下みたいな文字列（URL）だけを削除するにはどうしたら良いんだろか。

http://2ちゃん.com/test/20170123456789

数字の部分がうまく処理できない。

2017/09/09(土) 02:28:27.21

>>64
正規表現を使うといいかもです

スマホで手打ちしてるんで細かい部分で間違いがあるかもしれませんが、イメージはこんな感じです
(特にスラッシュのエスケープとか)

Dim strOrg as Sting
Dim strNew as Sting

strOrg = ※文章を代入

With CreateObject("VBScript.RegExp")
.Pattern = "http:////.*$";
.IgnoreCase = True
.Global = True
strNew = .Relpace(strOrg, "")
End With

2017/09/09(土) 09:08:52.36

コンパイルエラー：subまたはfunctionが定義されていませんと出ます。
Callされる関数の引数に問題があるのかと思うのですが、原因が分かりません。

Call ShowInIE(ie, "http://www.vba-ie.net/";, , 0, 0, 700, 200)
で
Sub ShowInIE(ie As InternetExplorer,
url As String,
Optional viewFlg = True,
top As Integer,
left As Integer,
width As Integer,
height As Integer)
を設定して渡すで良いのでしょうか？

ttps://pastebin.com/aJLAUnfc

2017/09/09(土) 09:16:22.62

「vba subまたはfunctionが定義されていません」で検索！

2017/09/09(土) 10:42:29.03

マクロにパスワードかけてたのだが、どうも破られてるらしき兆候があって
ぐぐった結果見つけたこのページの「マクロのパスワードを解除するマクロ」
を試したらほんまに解除できてびっくり
http://neos21.hatenablog.com/entry/2016/09/30/081709

これで破られない対策って誰かご存じないですかね？
自分で根ほり葉ほり動作調べてたら一日かかりそう

2017/09/09(土) 10:44:48.94

てか破り方誰でもわかるセキュリティなんて何もしてねえのと同じだ
目からウ●コが落ちた

2017/09/09(土) 11:26:51.38

標準モジュール

2017/09/09(土) 12:21:56.16

>>66
・1行の文を途中で改行する際の末尾アンダーバー（ _ ）を付け忘れている。
・Optional を付けたらそれ以降の引数もすべてOptional じゃないといけない

2017/09/09(土) 13:21:02.45

>>71
ありがとうございます。出来ました。

2017/09/09(土) 13:32:44.29

>>68
ない
MSですら公式に「間違って変更しないためのパスワードであり、セキュリティ云々ではない」と言ってる
ソースはどっか行った

2017/09/09(土) 13:49:29.47

「ない」ってあんたそんなズバリと（^ ^;

APIのエキスパートなら自分でどうとでもするだろうけど
人に聞いて知ってるような方法はないってことですね
しょーがないか

2017/09/09(土) 14:33:51.65

>>68
パスワードの文字数を多くするとか
記号を使うとか文字の種類を多くする

2017/09/09(土) 14:42:58.59

そういう問題じゃないんで・・
パスワードを解析するんじゃなく、記録してるとこからそれ自体引っ張り出して
解除させちゃうマクロらしいから、パスワードの複雑さ関係がないんで

2017/09/09(土) 15:11:07.11

まず使い方が間違ってる

Excelのパスワードの目的は秘密を守るためじゃなくて、
素人がマクロやシートをうっかり改変して業務に支障を来すのを防ぐのが目的と言われてる

簡単に破れる方法であっても、意図的な操作がなければ破られることはないんで、目的は達成できている

2017/09/09(土) 15:41:57.68

>>65
ありがとう。やってみる。

2017/09/09(土) 16:07:18.45

まあパスワードかけるなら出来合いじゃなくて
プログラマのハシクレなら自分でそんくらい作れってことかね
他人をあてにするもんじゃないっすね

2017/09/09(土) 16:17:06.10

一体どういう目的でパスワードかけるんだろう。実装をどうしても隠したいとかでなけりゃ破られても構わんだろ。

2017/09/09(土) 17:24:58.63

社内で使ってるけど皆が改変して収拾がつかなくなって管理出来なくなるのを防ぐとかだったな。
前に見たパスワード掛けてたとこは。

2017/09/09(土) 17:28:22.68

MSの公式見解だとパスワードはワークグループを分けて誤操作を防ぐ為のものであって、セキュリティ性は無いとされているね。

一応、VBAはバイナリファイルを読み書き出来るから、自分で暗号化技術調べてAES的な暗号化ファイルとか生成出来るぞ。
でも苦労して実装しても運用する奴らが素人だと暗号化した意味が無くなるから作り損だぞ。

2017/09/10(日) 03:50:51.77

ソフト名忘れたけどVectorにあるExcelアドインで>>68の手法が失敗するやつあったな

2017/09/10(日) 05:17:37.27

>>83
しかし、LibreOfficeのCalcからならあっさり見えてしまうんだな。
見えるからコピペし放題

2017/09/10(日) 06:19:08.10

運用ルールで勝手にいじるなって言えばいいじゃん

68 (ﾜｯﾁｮｲ b7e4-NEns) · 2017/09/10(日) 07:28:43.80

>>83
ほおそれは興味深いですねえ
探してみます

2017/09/10(日) 08:56:33.56

>>85
そんなんで解決できるなら誰も苦労せんわ
「何もやってないのに勝手におかしくなった」は聞き飽きた

2017/09/10(日) 09:11:12.77

「何もやってないのに勝手におかしくなった」
これ一番イラッと来るよな

「パソコンが壊れたー（フリーズしてるだけ）」
とか。

2017/09/10(日) 09:12:05.85

「改変したい点がある。自分でやるのでパスワードを教えてください。」と
直球で要望が来たらどうするの？

2017/09/10(日) 09:17:02.60

>>89
修正履歴書いて残せばいい

2017/09/10(日) 09:21:21.87

チーム開発でも修正ルールがあるし
会社のシステムを書き換えるには上司の許可いるだろ

2017/09/10(日) 09:25:28.85

>>89
そいつ用にパスワード外して送ればいいんじゃね
もちろん改変後の質問は受けない

2017/09/10(日) 09:31:38.83

>>89
普通の会社では許可しない
規定のフォーマットで要望書をシステム課に出せ

2017/09/10(日) 09:51:20.19

>>91
vbaには無い。
こういうのは一般事務の女の子向けに男がヘラヘラ書いたもの、ってかvbaが安易に書き換えられる仕様は常識

会社の基幹システムはもちろん無理だけどね
変更許可は絶対に降りない。然るべき手続きが必要

2017/09/10(日) 10:26:56.59

>>94
基幹に近い所でVBAが動いてるうちの会社みたいなところもあるんやで
一部上場でテレビCMもガンガン流してて誰でも知ってる大企業

入力ミスをなくすためにワークシートにロックかけて入力から印刷までフォームなしのVBAで組んである
モッサリでめっちゃ使いにくいシステムで見積書とか作らされてる
下っ端の俺にはどうしようもない

2017/09/10(日) 10:31:20.17

>>95
世間知らずでスマンかった…
ご愁傷様なのだ

2017/09/10(日) 11:32:55.83

売り上げ金額が合わなかったので勝手に書き換えておきましたとかやってんのか？

2017/09/10(日) 11:38:17.51

知らない人がいつの間にか勝手に書き換えてましたじゃ駄目だろ

2017/09/10(日) 11:39:01.10

>>96
アライさんおつｗ

2017/09/10(日) 11:49:00.36

>>97
それ現場、現場の人間はマクロを触る能力なんかない
現場では調整用貯金箱とか作って、金が多かった日は貯金、少なかった日はそこから取り崩しとかやる

2017/09/10(日) 12:47:48.18

後は、客はパスワード解除しないだろう、出来ないだろうという前提でパスワード掛けてたことも有ったな。
客の別の部署の奴がプログラムの仕組みが見れては困るという要望に対して。

そういう案件の場合はExcelでやらずにちゃんとシステム組むべきという俺の指摘は営業的事情により却下されたな。
客がそんなに金使いたく無いんだとよ。

2017/09/10(日) 12:54:29.64

>>94
それやって収拾がつかなくなって、ちゃんと管理しましょうという話になってパスワード掛けることになる。

2017/09/10(日) 13:27:08.19

Range("J5:J12").Value = "AVERAGE (RC[-7]:RC[-2])"

これどういう意味ですか？

2017/09/10(日) 13:28:56.73

>>103
これの右辺が分からないです

2017/09/10(日) 14:00:23.18

エクセルVBAの課題ですが分かりません。
Dim Target As Range
Dim temp As String

For Each Target In Range("C2:C6")
'セルの値をすべて全角に置換して変数tempに代入する。変換する文字列は、Valueプロパティを使って取得する
temp = StrConv(Target.Value, vbUpperCase)　←
'置換後の値の全角スペースを半角スペースに変換し、元のセルに入力する
Range("C2:C6") = temp　　←
Next
End Sub
元の値は半角カタカナで別の名前が各セルの記入されています。
矢印の個所が参考書を元に記述しました。

2017/09/10(日) 14:06:44.63

>>105
の文章がﾜｶﾗﾝｶ

68 (ﾜｯﾁｮｲ b7e4-NEns) · 2017/09/10(日) 14:26:35.95

>>83
ツールあったから結局買いました
情報ありがとう
http://conex-office-password.com/shopdetail/001000000001/
もし駄目でもどうせ1000円だからな

2017/09/10(日) 14:49:33.57

>>107
これな。
仕組みが想像できる。

2017/09/10(日) 15:03:50.97

>>103
こうじゃないの？
Range("J5:J12").FormulaR1C1 = "=AVERAGE(RC[-7]:RC[-2])"
範囲上のセルに数式を入力する

2017/09/10(日) 15:06:41.55

>>105
何が分からんのか具体的に

後多分コードが間違っている
Sub a()
For Each target In Range("C2:C6")
'セルの値をすべて全角に置換して変数tempに代入する。変換する文字列は、Valueプロパティを使って取得する
temp = StrConv(target.Value, vbUpperCase)
'置換後の値の全角スペースを半角スペースに変換し、元のセルに入力する
target.Value = temp
Next
End Sub

2017/09/10(日) 15:15:05.13

>>104
右辺は単なる文字列です
セルにそういう「数式みたいな」文字列が入るだけ

2017/09/10(日) 15:15:26.00

というか、どうしても解除されたくなかったらxllアドインとかにすべきでしょ。

2017/09/10(日) 16:45:46.81

>>100
それ持ち逃げしてもばれないよね

2017/09/10(日) 17:52:32.57

>>107
ああ、それ俺も買ったけど良かったよ。VBEを開こうとすると途端にエラーが出るので
大抵は諦めるね。俺はそれに加えてvbeのフォントを白にして嫁ないようにした。

2017/09/10(日) 22:09:00.85

>>105
何がわからないのか全然わからない
そのソースコードを実行するとどうなるのか、期待値は何なのか書かないと

まあ、ひとまず temp を配列にしないとダメじゃない？

Dim temp() As String
Dim i As Integer

Redim temp(Range("C2:C6").Rows.Count)

i=0
For Each Target In Range("C2:C6")

temp(i) = StrConv(Target.Value, vbUpperCase)
i = i + 1

Next

Range("C2:C6") = temp

2017/09/11(月) 01:14:13.54

>>115
１セルずつループして処理すればtempを配列にする必要なんてない

どうでもいいけど、全角はvbWideじゃないのかね
全角スペースを半角にするのはReplaceあたり使えば良いんじゃね
課題ならその課題出した奴に聞けよと思うが

2017/09/11(月) 07:33:49.76

>>115
速度稼ぐために配列にしてるなら読み出し側もやるべき

2017/09/11(月) 07:48:20.80

>>116-117
うん
送信押して、あっと思った
しかも縦方向だと2次元配列じゃないとだめだったっけ？

>>105
ごめん
多分>>110が正解

課題と質問の意図がよくわからないけど

2017/09/11(月) 08:13:17.53

>>105
矢印の業務が空行の状態が問題文、矢印の行は君が書いた、と解釈した。

矢印1行目は、vbuppercaseじゃなくて、vbwide。uppercaseは大文字だ。
2行目は、target=temp

そういう問題じゃないようだが、高速化するなら、
Rangeを配列に代入
配列内で計算
配列をrangeに書き戻し
する。

2017/09/11(月) 08:14:49.10

>>119
(誤)矢印の業務
(正)矢印の行
スマン

2017/09/11(月) 15:42:49.07

なんで君らは
>置換後の値の全角スペースを半角スペースに変換し
は放置なの？

2017/09/11(月) 20:27:23.15

>>121
質問の仕方が悪いから、だな。

では>>119の2行目を
target=replace(temp, "◻︎", " ")
# ◻︎は全角スペースと読んでくれ
に変更。

2017/09/12(火) 02:00:32.49

target=replace(temp, "全角スペースと読んでくれ", " ")

2017/09/12(火) 23:06:27.96

3万行前後ある列のデータの隣セルに、別シート5000行前後の列から、if条件に該当したものをループで貼り付ける処理をしているけど処理が遅い
3万行の列が外側ループi、5000行の列が内側ループjで回してる
処理時間短縮の為に、最初にiとjのデータを配列で取得して比較を行い
該当するものをセルに出力するように変更した結果、配列使用前より処理時間が短くなった
もっと処理時間を早くするにはどうすればいいかお知恵をください

2017/09/12(火) 23:14:30.57

>>124
5000がユニークデータなら
コレクション

2017/09/12(火) 23:23:18.75

>>124
別シート5000をディクショナリーに放り込んで、引いてくれば良いんでない？その手の話で高速化したいなら二重ループはナンセンス。

2017/09/12(火) 23:27:05.41

>>124
条件と出力形式が分からないからハッキリとは言えないけど、
もし該当したものを都度吐き出しているなら、結果自体を配列にして最後に吐き出す様に直すと早くなるはず。

2017/09/13(水) 00:11:27.19

>>124
ここに載ってるDOUBLE TRUE VLOOKUPを試してみてよ
lookupの時間だけ比較してもDictionaryより速いらしいんだが
http://analystcave.com/excel-vlookup-vs-index-match-vs-sql-performance/

アルゴリズムだけ考えたらバイナリサーチより
ハッシュテーブルのほうが速そうなもんだけどな

2017/09/13(水) 00:31:34.36

>>125-126
ありがとうございます！
なるほど、コレクションかディクショナリーで連想配列を使用すれば早いってことか
イマイチ使い方が分からず使用していなかった
大量にデータを処理する場合は活用すれば、処理が早くなる可能性があるって事なんですね
今後は意識して使わせていただきます

>>127
ありがとうございます！
今のコードは　ifがTrue時にその都度セルに出力するコードです。
if 配列A(i)=配列B(j) then
worksheets(出力先シート)cells(i,出力先列)=配列B(j)
end if

結果自体を配列にするというのこういう処理でいいでしょうか?
結果を取得する配列Cを宣言
if 配列A(i)=配列B(j) then
配列C(i)=配列B(j)
end if
この後の処理は、配列C(i)の値をループでシートに出力する以下の処理でしょうか?
worksheets(出力先シート)cells(i,出力先列)=配列C(i)

>>128
ありがとうございます！
これは普通のキーワードでネットで調べてもヒットした記憶がないワードです
今後に活用させていただきます

2017/09/13(水) 00:32:10.56

ハッシュテーブルはハッシュの算出方法で変わるからな
対象データに対して衝突が少なくて計算の軽いハッシュ関数じゃないと

2017/09/13(水) 00:58:34.20

>>129
配列とセルは相互に直接的なやりとりが出来るんで調べてみて。
いちいちループ回さなくていいから。

簡単に言えば

Dim A As Variant
A = Range(範囲).Cells

とすれば配列Aに範囲の中身が入り、

Range(範囲) = A

とすれば配列Aの中身が範囲に入る。

2017/09/13(水) 07:30:42.98

ネットでエクセルVBAを勉強中なんですがオススメ参考本とかないでか？
特にユーザーフォームを使って作業が多いです。

2017/09/13(水) 07:35:00.26

>>124
条件の内容にもよるけど、ワークシート関数のvlookupは使えないの？

2017/09/13(水) 08:14:57.58

>>133
横からすまん。試してみりゃいいんだろうけど...
vlookup(ワークシート関数)って、マクロから使うメリットがあるほど速いの？

2017/09/13(水) 08:26:49.86

お前VBAスレの自称初心者だろW

2017/09/13(水) 08:46:51.19

>>134
試せよｗ

って言いたけど昔試験したから教えるわ
実はvlookupは参照範囲を広くしなければかなり早い
ただ2003の頃の話だから、今はどうかしらん

**107** (ﾜｯﾁｮｲ 8a2e-NEns) · 2017/09/13(水) 09:25:03.74

たかがソフト買うのに個人情報を洗いざらい要求してくるセキュリティ企業って・・・
なんで俺の住所や電話番号がいるんだよ。なんに使う気だ

しかもダウンロードしようとしたら送ってきたパスワード
そっくりコピペなのにエラー出るじゃねえか
大丈夫かここ

2017/09/13(水) 10:51:59.22

会社紹介を見る限りまともじゃなさそう
ツールの中身も海外フォーラムに載ってるソースコピペしてたりしてな

2017/09/13(水) 12:27:02.45

標準のワークシート関数ってVBAで作ってある上に、色々な目的に使えるように冗長性があるから、
目的に対して最も効率的な処理方法であるという条件下だと処理時間は必ず

ワークシート関数 ≧ VBAマクロ

が成り立つ。
特にVLookupとかはセル毎に走査してるから共有可能な部分も個別に計算してるんで効率面はお察し。

ただしワークシート関数とほぼ同じ機能をVBAマクロで実装するのが時間の無駄って事もあるので、
マクロ内でワークシート関数を利用するという状況を考えるのは難しくない。

2017/09/13(水) 12:34:53.43

>>136
ありがとう。

なんか>>139の言う(汎用|冗長)性による低速処理の懸念があって、あまりワークシート関数を使う事考えないんだよね。

2017/09/13(水) 12:45:19.24

お前らが書いた腐ったvbaコードよりワークシート関数が遅いとかｗ
どんだけ頭がお花畑なんだよｗ

2017/09/13(水) 13:09:42.32

ワークシート関数は条件さえ整えばマルチスレッドで動くぞ
確実にVBAより早いわ

2017/09/13(水) 13:55:24.11

https://msdn.microsoft.com/ja-jp/library/office/bb687899.aspx

公式ページ↑見ると2007からマルチスレッディングになってるらしいけど、
経験上、同じ様な目的の実装で処理時間が

VBAマクロ＞　ワークシート関数

になった事がないんだよな。

十分に最適化されたVBAマクロよりもワークシート関数の方が処理が早くなるって状況の具体例があれば参考になるんだけど、
そもそも目的が違うのもあってワークシート関数が重いからユーザー定義関数を作りましょうみたいなページしか見当たらない。