Ошибка при добавлении комментария к ячейке

@iskan13 · Регистрация: 11.08.2012

Студворк — интернет-сервис помощи студентам

День добрый.

Есть такой кусочек кода:

Visual Basic
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
For Count_2 = 1 To 16
    Set SelectedCell = ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2)
    If Not ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2).Comment Is Nothing Then
        CommentText = SelectedCell.Comment.Text
        SelectedCell.Comment.Delete
        Else
        CommentText = Empty
    End If
        
    Select Case Count_2
        Case 2
            SelectedCell.Comment.Text SelectedCell.Comment.Text & vbCr & "Íîìåð ñîïðîâîäèòåëüíîãî ïèñüìà " & SelectedCell.Text
        Case 3
            SelectedCell.Comment.Text SelectedCell.Comment.Text & vbCr & "Äàòà ðåãèñòðàöèè ïèñüìà " & SelectedCell.Text
        Case 11
            GoTo 1
        Case 12
            GoTo 1
        Case 13
            GoTo 1
        Case 14
            GoTo 1
        Case 15
            SelectedCell.Comment.Text SelectedCell.Comment.Text & vbCr & "Ïîñëåäíèé ñòàòóñ àêòà " & SelectedCell.Text
        Case 16
            SelectedCell.Comment.Text SelectedCell.Comment.Text & vbCr & "Äàòà ïîñëåäíåãî èçìåíåíèÿ " & SelectedCell.Text
    End Select
1:
Next

Вкратце: есть две таблицы. Из одной переносим данные в другую. Данные - информация по документации, один документ - одна строка. Если данные повторяются (совпадает номер документа), нужно переписать строку, при этом старые данные некоторых ячеек перенести в примечания к этой ячейке.

Все работает как надо, кроме добавления комментария, о чем и есть код выше. Не работают строки после каждого Case'а, который начинается с SelectedCell. Ошибка на скриншоте. Помогите разобраться, пожалуйста, что я делаю не так.

Спасибо.

@Vlad999 · 09.11.2015, 10:57

Visual Basic
1
2
3
Case 2
      SelectedCell.AddComment
      SelectedCell.Comment.Text SelectedCell.Comment.Text.....

@iskan13 · 09.11.2015, 10:58 **[ТС]**

Vlad999, вон оно че, михалыч) Спасибо, помогло!

@Апострофф · 09.11.2015, 11:02

Как-то так, может быть?

Visual Basic
1
SelectedCell.AddComment.Text CommentText & vbCr & "???®?¬???° ?±?®???°?®???®?¤???????«???*?®???® ?????±???¬?* " & SelectedCell.Text

Перед тем, как обращаться к Comment`у, его надо создать. Вы же удаляете его своим предыдущим кодом.

@iskan13 · 09.11.2015, 11:03 **[ТС]**

Апострофф, да, эту фигню я уже пофиксил, спасибо) Я, ввиду своей криворукости и неграмотности, сначала делаю как-нибудь, лишь бы заработало, а потом уже беру напильник и смотрю, как можно улучшить) в конце концов, делаю, можно сказать, для себя, так что... =)

@iskan13 · 09.11.2015, 11:12 **[ТС]**

А вот еще вопрос родился. Решил подсократить вот таким образом:

Visual Basic
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
For Count_2 = 1 To 16
    Select Case Count_2
        Case 2 Or 3 Or 15 Or 16
            Set SelectedCell = ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2)
            If Not ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2).Comment Is Nothing Then
                CommentText = SelectedCell.Comment.Text
                Else
                CommentText = Empty
            End If
            SelectedCell.Comment.Delete
            SelectedCell.AddComment
        Case 2
            SelectedCell.Comment.Text CommentText & vbNewLine & "Íîìåð ñîïðîâîäèòåëüíîãî ïèñüìà " & SelectedCell.Text
        Case 3
            SelectedCell.Comment.Text CommentText & vbNewLine & "Äàòà ðåãèñòðàöèè ïèñüìà " & SelectedCell.Text
        Case 11 Or 12 Or 13 Or 14
            GoTo 1
        Case 15
            SelectedCell.Comment.Text CommentText & vbNewLine & "Ïîñëåäíèé ñòàòóñ àêòà " & SelectedCell.Text
        Case 16
            SelectedCell.Comment.Text CommentText & vbNewLine & "Äàòà ïîñëåäíåãî èçìåíåíèÿ " & SelectedCell.Text
    End Select
1:
Next

Но не прокатывает с ошибкой (приложение). Подскажите, в чем дело? и как вообще это грамотно оформить?

@Апострофф · 09.11.2015, 11:20

iskan13, при копировании кода включайте русскую раскладку.

Добавлено через 4 минуты

Сообщение от iskan13

'Case 2 Or 3 Or 15 Or 16
Case 2, 3,15, 16'наверно это имелось ввиду?
'но учтите, что

Visual Basic
1
2
3
4
case 2
case 3
case 15
case 16

в этом SELECT`E уже не выполнятся ни при каком раскладе.

@iskan13 · 09.11.2015, 11:21 **[ТС]**

Апострофф, вас понял)

Кэйс в кэйсе не выполняется, значит? Апосля его нада пихать?

@Vlad999 · 09.11.2015, 11:23

Code
1
2
3
 Case 11 Or 12 Or 13 Or 14
 записывается так 
Case 11 To 14

@Казанский · 09.11.2015, 11:26

Сообщение от iskan13

Case 2 Or 3 Or 15 Or 16

Это пишется так

Visual Basic
1
Case 2, 3, 15, 16

И 17 строка не нужна.

@iskan13 · 09.11.2015, 17:05 **[ТС]**

Vlad999, спасибо, от этого я уже избавился за ненадобностью =)
Казанский, спасибо, уже поправил!

Добавлено через 4 часа 59 минут
Товарищи, а вот смотрите... Путем долгих и упорных мыслительных процессов моя недопрограммка (точнее, ее кусок) пришли вот к такому виду:

Visual Basic
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
For Count_2 = 1 To 16
    Select Case Count_2
        Case 2, 3, 15
            Set SelectedCell = ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2)
            If Not ActiveWorkbook.Worksheets(1).Cells(ReRowCount, Count_2).Comment Is Nothing Then
                CommentText = SelectedCell.Comment.Text
                SelectedCell.Comment.Delete
                SelectedCell.AddComment
                SelectedCell.Comment.Shape.Height = SelectedCell.Comment.Shape.Height + 100
                SelectedCell.Comment.Shape.Width = 200
                Else
                CommentText = Empty
                SelectedCell.AddComment
                SelectedCell.Comment.Shape.Height = 100
                SelectedCell.Comment.Shape.Width = 200
            End If
    End Select
    Select Case Count_2
        Case 2, 3, 15
            SelectedCell.Comment.Text CommentText & vbNewLine & vbNewLine & "Предыдущее значение: " & vbNewLine & SelectedCell.Text & vbNewLine & " от " & ActiveWorkbook.Worksheets(1).Cells(ReRowCount, 16)
    End Select
Next

Все прелестно работает, прям как и хотелось. Вопрос только один. Я все это делаю в цикле, как вы видите. Но в итоге я манипуляции произвожу только с 3 столбцами, а не с 16, как планировалось изначально. Так вот, как бы мне избавиться от этого бессмысленного перебора от 1 до 16 и заставить работать? Ну и вообще, как можно вот этот кусочек кода оптимизировать? Еще раз большое спасибо всем, кто мне помогает.

@Казанский · 09.11.2015, 17:41

iskan13, измените цикл так и уберите Select Case вообще

Visual Basic
1
For Each Count_2 In Array(2, 3, 15)

Переменная Count_2 должна быть типа Variant (если Вы не декларируете переменные, это получится автоматически).

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@iskan13 15 / 4 / 0 Регистрация: 11.08.2012 Сообщений: 157
	09.11.2015, 10:58 [ТС]
	Vlad999, вон оно че, михалыч) Спасибо, помогло! 0

@iskan13 15 / 4 / 0 Регистрация: 11.08.2012 Сообщений: 157
	09.11.2015, 11:03 [ТС]
	Апострофф, да, эту фигню я уже пофиксил, спасибо) Я, ввиду своей криворукости и неграмотности, сначала делаю как-нибудь, лишь бы заработало, а потом уже беру напильник и смотрю, как можно улучшить) в конце концов, делаю, можно сказать, для себя, так что... =) 0

@iskan13 15 / 4 / 0 Регистрация: 11.08.2012 Сообщений: 157
	09.11.2015, 11:21 [ТС]
	Апострофф, вас понял) Кэйс в кэйсе не выполняется, значит? Апосля его нада пихать? 0