Как вы думаете какой код лучше?

Вариант 1
------------------

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    Для Каждого Элемент Из Структура Цикл
        СимволРазделителя = ?(ПустаяСтрока(Результат), "", Разделитель);
        Результат = Результат + СимволРазделителя + Элемент.Ключ;
    КонецЦикла;
    Возврат Результат;
КонецФункции


Вариант 2
-----------------
Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + Элемент.Ключ + СимволРазделителя;
    КонецЦикла;
    Возврат Лев(Результат,СтрДлина(Результат)-1);
КонецФункции

PS Подразумеваем, что Структура всегда имеет минимум 1 значение

экви... ну понимаешь

первый, ибо он по смыслу написан, а не по результату, поэтому его проще понять

автор чекнутый ботан?

(3) Автор тут уже какой год сидит и темы у него разнообразные ))0

хотя, с другой стороны, пофиг

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + ?(ПустаяСтрока(Результат), "", Разделитель) + Элемент.Ключ;
    КонецЦикла;
    Возврат Результат;
КонецФункции

(5) но в каждой итерации делать проверку на условие, это же медленная операция

(6) у тебя миллион итераций? нет? тогда плевать

у 1 читабельность лучше, но в цикле больше операций - плохо на больших объемах

(7) " Но каждый грамм метала должен чувствовать в себе сталь.
    Мы в любой заготовке обязаны увидеть деталь."

ну или - копейка рубль бережет

А вдруг вот жизнь подкинет массив с миллионом

(9) ну, вот, как подкинет - так и оптимизируешь)
Но ты прав, всё начинается с малого - задумайся об этом, читая в очередной раз чужой код или создавая свой новый быстрый.. имхо, лучше по потребностям и без излишней хитрости по мелочам

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + Символы.ПС + Элемент.Ключ;
    КонецЦикла;
    Результат = СтрЗаменить(СокрЛП(Результат), Символы.ПС, Разделитель);
    Возврат Результат;
КонецФункции

первый вариант однозначно хуже всех остальных, т.к. у него цикломатическая сложность самая большая.
Второй вариант плох тем, что там волшебные числа.

(11) СокрЛП() убирает табуляцию?

(0) во 2 варианте ошибка.

(13) да. Но ПС - не табуляция

+13 даже если убирает, все равно не универсально, если в строке таб окажется

(15) ок

(16) откуда он там окажется?

+(14) кстати, да
КлючиСтруктурыВСтроку(Структура, "Вариант 2 - гогно")

прогнал тест на 100 тыс записей

вариант 1 - 263 сек
вариант 2 - 273 сек

(14) Ошибку видел, считай, что там правильное название переменной

(18) в тексте не может быть перевода строки?)
(19) там еще и по синтаксису очепятка)

(21) в функцию во втором параметре могут передать больше одного символа. Там две стрдлины надо, чтобы правильно работало.

(22) в какой версии платформы ключ структуры может содержать табуляцию? Или лучше спросить - в какой параллельной вселенной?

(21) Окей принимается, но вопрос был о другом

это для (23) написал

(23) точно, тут же "Элемент.Ключ", убедил

(24) вот тут развернутый ответ (12)

(20) круто, теперь сделай структуру из 10+ ключей

вариант в (5) -285 сек
вариант 1 - 283 сек

фигня какая-то

по сути можно сказать, что одинаково

Проблема скорее всего упирается в конкатенацию строка Результат. Она становиться очень большой и из-за нее все тормоза

(29) у тебя структура из 100к ключей или 100к вызовов функции?

Как-то так:

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    СимволРазделителя = "";
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + СимволРазделителя + Элемент.Ключ;
    СимволРазделителя = Разделитель;
    КонецЦикла;
    Возврат Результат;
КонецФункции

+ голосовалка

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт

Результат = "";
СимволРазделителя = "";

Для Каждого Элемент Из Структура Цикл
         Результат = Результат + СимволРазделителя + Элемент.Ключ;
         СимволРазделителя = Разделитель;
КонецЦикла;

Возврат Результат;

КонецФункции

(0) Один код не может быть лучше другого, по определению.

в строну внутр и из строки внутре видимо никого в этой теме не учили. какие то дурацкие фиклы по массиву с разделителями.

+(35) программисты фиговы.

(36)Продемонстрируй класс. Выдай свой вариант.

даже пилять в сабже тупой код в обоих вариантах

(37) синтаксис открой неуч.

ЗначениеВСтрокуВнутр
ЗначениеИзСтрокиВнутр

две строчки кода.

А как быть с разделитель. Например ";"?

(31) Опередил
(34) Как это не может? Скажи еще, что гуавнокода не бывает.
А так-то, у каждого додика своя методика. Поэтому

Всех кто в этой теме двоешники, кроме естественно меня, как настоящего пропитанного 1Ска.

(39)"хамишь, парниша"
(43)слифффф

(43) Ничо, что у тебя результат немного другой, чем в сабже?

(44) ну я так и знал что все сольетесь как я прийду и покажу всего две строчки кода.

(43) Знаем, знаем, "Одна я умная, в белом пальто стою красивая"

(45) а ты мозг свой включи. Нафига ему нужна эту функция.
Ответ просто - чтобы потом еще иметь другую функцию которая это считает обратно.

Перенедопил. ИМХО.

Скорее всего автор пытается в строку запихнуть структуру не ведаю про СтрокуВнтур, скорее всего пытается изобрести сохранение настроек.
И так же обратно потом будет мозг делать чтобы эту строку разобрать обратно.

(42) Вот так вот - не может. Одна воля ничуть не лучше и не хуже другой воли.
Твоя эффективность определяется не тем, какой код ты пишешь.
А тем, можешь ли ты здесь и сейчас написать код и сколько времени у тебя это займет.

(50)Гадать можно и на кофейной гуще, а ответить конкретным кодом не получается.

(48) Эта функция ему нужна, чтобы задать вопрос на мисте.
Это же очевидно.

Я не вижу никаких других дурацких задач шде бы из структуры нужно было сделать строку с разделителями строк. Да еще в ракурсе какого то ключа.
Поэтому вы все и двоешники.

И двешники потому что:

КолВсего = Структура.Количество();
Сч = 0;

Результат = "";
    Для Каждого Элемент Из Структура Цикл
Сч = Сч+1;

Если Сч = КолВсего
        Результат = Результат + Элемент.Ключ;
Иначе
Результат = Результат + Элемент.Ключ + СимволРазделителя;
КонецЕсли;
    КонецЦикла;

(54)бггг

(54)"СтрокуВнтур" забыл?

(56) это вообще верхсупермегабыстрый метод 1С. я бы сказал это самый крутой встроенный метод 1С

во все времена

он творит чудаса

неправильно выразился: это самый крутой метод 1С из всей 1С вместе взятой.

извращенцы, блин.

я использую в зависимости от фазу луны вариант 2 и тот, который похож на (5), только используя сред(Результат,2). При этом это не имеет совсем никакого значения.

И пох на разницу в 0,01  мс. Ибо итераций < 100000000.

И в 50% случаев прав (40), ибо с вероятностью 50% применимо (48). Но жизнь все же многообразней, поэтому только 50% за этот вариант.

(40) Не взлетит если структуру заполнить, а она на 99,999% будет заполнена

(0) Сейчас вечером, в пледе, после горячего чая можно решать что быстрее, а за 2 часа до окончания рабочего дня оба метода будут одинаково эффективны.

(0) Только можно поправить

...
Результат = Разделитель + Результат + Элемент.Ключ;
....
Возврат Сред(Результат, 2);
....

(64) Результат = Результат + Разделитель + Элемент.Ключ;

(64)  - (54) не пробовал посмотреть?

могу даже красивее нарисовать из (54) если кому не нравиться!

КолВсего = Структура.Количество();
Сч = 0;

Результат = "";
Для Каждого Элемент Из Структура Цикл
Сч = Сч+1;
Результат = Результат + Элемент.Ключ +  ?(Сч = КолВсего,"",СимволРазделителя);
КонецЦикла;

(67) "Сч = Сч+1;" и "?(Сч = КолВсего,"",СимволРазделителя)" лишние

(60)
Самый крутой это ЗаполнитьЗначенияСвойств()

(31)
Вот оно

(48)
Или передать в другую программу. Но ты, как истинный одинэсник, отрицаешь существование других программ. Ибо ересь

(71) ибо зоопарк

все спасибо, вот такая миста мне нравится

   Eugeneer - получает отедбный приз - плесневелую коробку 1С 7.7

ЗаполнитьЗначенияСвойств - отличный метод, но всегда страшно, как оно там работает

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
Перем Счётчик;
    Результат = ""+Структура[0].Ключ;
    
    Для Счётчик  = 1 по Структура.Количество() Цикл
        Результат = Результат + СимволРазделителя + Структура[Счётчик].Ключ;
    КонецЦикла;
    Возврат Результат;
КонецФункции

(74) Вывалиться с ошибкой без минус 1
(74)+

(0) у меня такой код программа пишет

(74)

Перем Счётчик; // - привет мамонтам )))

(74) и давно к элементам структуры можно стало обращаться по индексу?

мой вариант,
1. пофиг на размерность структуры (отображает корректно пустую)
2. возможен разделитель из нескольких символов
3. быстрый
-----------------
Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    СимволРазделителя = ?(ПустаяСтрока(Результат), "", Разделитель);
    Результат = СимволРазделителя;
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + Элемент.Ключ + СимволРазделителя;
    КонецЦикла;
    Возврат Сред(Результат,СтрДлина(СимволРазделителя)+1);
КонецФункции

(79)+

Функция КлючиСтруктурыВСтроку(Структура, Разделитель = ",") Экспорт
    Результат = "";
    Для Каждого Элемент Из Структура Цикл
        Результат = Результат + Разделитель + Элемент.Ключ;
    КонецЦикла;
    Возврат Сред(Результат,2);
КонецФункции

(0) Из представленных 2 - первый, т.к. корректно отработает при любой длине разделителя. А так - я использую флаг ЭтоПерваяИтерация - с ним быстрее будет, чем с определением, пустая ли строка.