Задачи 22 и 11 #23
Задачи 22 и 11 #23
Conversation
| // Возвращаемое значение: | ||
| // Строка - путь, оформленный по правилам ОС | ||
| // | ||
| Функция НормализоватьПуть(Знач ОтносительныйИлиПолныйПуть) Экспорт |
There was a problem hiding this comment.
А зачем ещё одна функция? Почему не вызвать ПолныйПуть?
There was a problem hiding this comment.
Мне название нравится, понятнее, чем ПолныйПуть. Оставил как предложение сделать такой "алиас". Разумеется, можно убрать
There was a problem hiding this comment.
Мне название нравится, понятнее, чем ПолныйПуть. Оставил как предложение сделать такой "алиас". Разумеется, можно убрать
Мне также нравится название НормализоватьПуть, предлагаю оставить
| @@ -180,7 +180,62 @@ | |||
| Функция ПолныйПуть(Знач ОтносительныйИлиПолныйПуть) Экспорт | |||
| Файл = Новый Файл(ОтносительныйИлиПолныйПуть); | |||
| Возврат Файл.ПолноеИмя; | |||
| КонецФункции // ПолныйПуть(Знач ОтносительныйИлиПолныйПуть) Экспорт | |||
| КонецФункции // ПолныйПуть(Знач ОтносительныйИлиПолныйПуть) | |||
There was a problem hiding this comment.
подобные дубль-комментарии после КонецФункции\Процедуры предлагаю удалять - я про // ПолныйПуть(Знач ОтносительныйИлиПолныйПуть)
| // | ||
| Функция НормализоватьПуть(Знач ОтносительныйИлиПолныйПуть) Экспорт | ||
| Возврат ПолныйПуть(ОтносительныйИлиПолныйПуть); | ||
| КонецФункции // НормализоватьПуть(Знач ОтносительныйИлиПолныйПуть) Экспорт |
| &Параметры("C:/projects/SB/vanessa\apache/uplevel/..", "C:/projects/SB/vanessa/apache") | ||
| #КонецЕсли | ||
| Процедура Тест_ПутиРавны(Первый, Второй) Экспорт | ||
| Ожидаем.Что(лФС.ПутиРавны(Первый, Второй)).ЭтоИстина(); |
There was a problem hiding this comment.
не хватает кейса сравнения относительного пути через . и абсолютного пути к этому же каталогу
например, через УстановитьТекущийКаталог
There was a problem hiding this comment.
Это как?
можно просто
лФС.ПутиРавны("./папка1", ОбъединитьПути(ТекущийКаталог(), "папка1"));
There was a problem hiding this comment.
ну и аналогично с двумя точками в начале пути
лФС.ПутиРавны("../fs", ОбъединитьПути(ТекущийКаталог()));
Нужны для oscript-library/opm#208