From 09a6ce56cd63fb76a9fdb85685c99ac3fb77d47d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E2=80=9CNaeel=E2=80=9D?= <tazet@narod.ru>
Date: Wed, 29 Apr 2026 21:31:33 +0400
Subject: [PATCH] cleanup: trim compute and operation list comments

---
 analysis/cleanup-worklog-2026-04-29.md        | 38 +++++++++++++++++++
 v1/resources/instance_operation_ls.cfc        | 11 ++----
 .../svc_operation_cfs_subparam_compute.cfc    |  5 +--
 3 files changed, 43 insertions(+), 11 deletions(-)

diff --git a/analysis/cleanup-worklog-2026-04-29.md b/analysis/cleanup-worklog-2026-04-29.md
index a97feb4..70096d2 100644
--- a/analysis/cleanup-worklog-2026-04-29.md
+++ b/analysis/cleanup-worklog-2026-04-29.md
@@ -644,3 +644,41 @@
 Результат:
 - синтаксических ошибок в `v1/resources/catalog_service_ls.cfc` не обнаружено
 - синтаксических ошибок в `v1/resources/resource_realm.cfc` не обнаружено
+
+## Обновление: выбран cleanup-проход по compute/list слою
+
+Дата фиксации: 2026-04-29.
+
+### Выбранные файлы
+
+- `v1/resources/svc_operation_cfs_subparam_compute.cfc`
+- `v1/resources/instance_operation_ls.cfc`
+
+### Ограничения этапа
+
+- убирать только закомментированные debug-хвосты и комментарии-рассуждения
+- не менять активную обработку ошибок и не исправлять живые runtime-аномалии
+- не заходить глубоко в `post` и крупные блоки бизнес-логики
+
+### Результат этапа
+
+Выполнен cleanup-проход по compute/list слою:
+- `v1/resources/svc_operation_cfs_subparam_compute.cfc`
+- `v1/resources/instance_operation_ls.cfc`
+
+Что убрано:
+- комментарии-рассуждения рядом с helper-инициализацией
+- закомментированные `cfdump`/`cfabort` хвосты
+- старые закомментированные debug-строки валидации и SQL-вывода
+
+Что сознательно не трогалось:
+- активные `cfcatch` и текущее поведение ошибок
+- крупные блоки `post`
+- спорные бизнес-комментарии вне явного debug-слоя
+
+### Проверка
+
+Измененные файлы проверены на ошибки.
+Результат:
+- синтаксических ошибок в `v1/resources/svc_operation_cfs_subparam_compute.cfc` не обнаружено
+- синтаксических ошибок в `v1/resources/instance_operation_ls.cfc` не обнаружено
diff --git a/v1/resources/instance_operation_ls.cfc b/v1/resources/instance_operation_ls.cfc
index 6738cdb..cc9cbb7 100644
--- a/v1/resources/instance_operation_ls.cfc
+++ b/v1/resources/instance_operation_ls.cfc
@@ -2,7 +2,7 @@
 
 	<cfsilent>
 		<cfimport prefix="m" taglib="../lib"/>
-		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/><!---*** странно, почему мы его видим?---><!---вынести в апп?--->
+		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
 	</cfsilent>
 	
 	<!---спецификация полей, пригодных для фильтрации (и сортировки)--->
@@ -41,19 +41,18 @@
 		<cfargument name="orderBy" type="string" hint="type:string, description:comma-separated list of fields to sort by, example:fld1.ASC,fld2.DESC,fld3.ASC" default=""/>
 
 		<cfset var local={}/>
-		<cftry>	<!--- отладка на время проблемы 404 --->	
+		<cftry>
 						
 		
-			<!---parse and validate request parameters--->
+			<!---разбор и проверка параметров запроса--->
 			<cftry>
 				<cfset this.helper.validateField(arguments, "pageSize", "integer")/>
 				<cfset this.helper.validateField(arguments, "page", "integer")/>
 				
 				<!---мы мирно игнорируем поля, отсутствующие в спецификации, что позволяет не делать исключения для orderBy и т.п.--->
 				<cfset var filter=this.helper.parseFilterParams(this.fieldsSpec)/>
-				<!--- <cfset var order=this.helper.parseOrderBy(this.fieldsSpec, arguments.orderBy)/> --->
 				
-				<cfcatch type="invalidParamValue"><!--- <cfreturn representationOf("Проверка связи 3")> --->
+				<cfcatch type="invalidParamValue">
 					<cfreturn representationOf(this.helper.formatBadRequestError(cfcatch)).withStatus(400)/>
 				</cfcatch>
 				<cfcatch type="any">
@@ -113,7 +112,6 @@
 					offset #arguments.pageSize*(arguments.page-1)#
 				</cfif>
 			</cfquery>
-			<!--- <cfdump var=#local.qRead#/><cfabort/> --->
 			
 			<cfquery name="local.qCnt">
 				select count(*) as cnt
@@ -153,7 +151,6 @@
 			<cfset "out.size"=#arrayLen(resultCollection)#/>		
 			<cfset "out.results"=#resultCollection#/>		
 			<cfset "out.runDurationMs"=getTickCount()-request.startTickCount/>	
-			<!---<cfset "out.sql"=#local.result.sql#/>--->	
 			<cfreturn representationOf(out)/>	
 
 			<cfcatch type="any">
diff --git a/v1/resources/svc_operation_cfs_subparam_compute.cfc b/v1/resources/svc_operation_cfs_subparam_compute.cfc
index 01a5a7e..58dc49a 100644
--- a/v1/resources/svc_operation_cfs_subparam_compute.cfc
+++ b/v1/resources/svc_operation_cfs_subparam_compute.cfc
@@ -21,7 +21,7 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In
 
 	<cfsilent>
 		<cfimport prefix="m" taglib="../lib"/>
-		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/><!---*** странно, почему мы его видим?---><!---вынести в апп?--->
+		<cfset this.helper=CreateObject("component","lib.rest_api_helper")/>
 	</cfsilent>
 	
 
@@ -86,9 +86,6 @@ arguments.requestArguments.specificationId=usrCustomerInfo.specificationId; //In
 		<cfelse>
 			<cfreturn representationOf(this.helper.formatMessage("Expression Not Found")).withStatus(404)/>
 		</cfif>			
-		<!--- <cfdump var=#local.keys#/>
-		<cfdump var=#local.params#/>
-		<cfdump var=#local.calculatedValueList# abort=true/> --->
 		<!--- Тут нам еще придется сформатировать лист в массив. Но обошлось довольно просто --->		
 		<cfset var out=structNew("linked")/>
 		<cfset "out.queryDurationMs"=getTickCount() - request.startTickCount/>