Обнуляется глобальная переменная

@Jocker · Регистрация: 22.01.2015

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

Добрый день. У меня обнуляется глобальная переменная, хотя и не должна. Идея в том, что каждой галлерее соответсвует своя папка в каталоге и при переименовании галлереи, должна переименовываться и папка. В первом методе я записываю старый путь папки, а во втором хочу её переименовать.
Проблема в том, что в первом методе переменная правильно инициализируется (проверял дебагом), но на входе во второй имеет значение null.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public async Task<IActionResult> EditGal (int? id)
        {
            if (id!= null)
            {
                Gallery gallery = await db.Galleries.FirstOrDefaultAsync(x=>x.ID == id);
                oldGallery = "~/wwwroot/images/"+gallery.Name;
                return View(gallery);
            }
 
            return NotFound();
        }
 
        [HttpPost]
        public async Task<IActionResult> EditGal (Gallery gallery)
        {
            string path = @"wwwroot/images/";
            if (Directory.Exists(path + gallery.Name))
                Directory.Move(oldGallery, path+gallery.Name);
 
            db.Galleries.Update(gallery);
            await db.SaveChangesAsync();
            return RedirectToAction("gallery");
        }

@bodynar · 12.11.2019, 10:24

Судя по [HttpPost] это у вас web-приложение, где на каждый запрос создается новый экземпляр контроллера / контекста / etc => следовательно ваше значение oldGallery осталось в предыдущем экземпляре контроллера

@HF · 12.11.2019, 10:35

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

В первом методе я записываю старый путь папки, а во втором хочу её переименовать.

Это веб-приложение, у него есть жизненный цикл. Не надо пытаться что-то сохранить в каком-то из запросов.
Вы должны прокидывать все нужные параметры. Это значит что при POST запросе, вы должны ещё и передать id старой галереи. И нет смысла генерировать полный путь и сохранять его в переменной. Во-первых, это вы снова делаете/можете сделать. Во-вторых, раз это каталог галереи, то заведите метод/сервис, который будет по ID возвращать путь.

@Jocker · 12.11.2019, 11:10 **[ТС]**

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

Во-вторых, раз это каталог галереи, то заведите метод/сервис, который будет по ID возвращать путь.

На ум пришло только 2 способа:
1) завести статический класс, в который передавать имя старой галереи и в другом методе считывать. (как по мне самый костыльный и неправильный)
2) в модель добавить поле, в котором будет старое имя. А после переименования галереи, переименовать папку, имеющую старое имя галереи.

Как вы считаете, второй вариант приемлемый? Может у Вас другое видение решения этого вопроса)

@bodynar · 12.11.2019, 11:18

Jocker, а зачем вам именовать папку как галлерею? Почему бы не задавать название как идентификатор?
Тогда и переименновывать не нужно.

А если обязательное условие название галлереи в названии папки, то передавайте старые и обновленные данные.
Но если вы передаете модель с идентификатором - вытащите из бд не обновленные данные - и будет вам oldGalleryName

@Jocker · 12.11.2019, 11:35 **[ТС]**

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

а зачем вам именовать папку как галлерею?

Все изображения, которые добавляются на сайт, загружаются в определённую директорию. Что бы не было и все картинки не были сброшены в одну яму - я решил для каждой галереи сделать папку и рассортировать картинки по папкам (галереям). Если передавать иденификатор, как название, то при большом количестве галерей (сайт дизайнерский и предполагает большое количество галерей), вручную будет трудно найти папку с файлом.

В первом методе у меня заходит необновлённая модель, а втором [HttpPost] - уже обновлённая.

@bodynar · 12.11.2019, 12:04

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

В первом методе у меня заходит необновлённая модель, а втором [HttpPost] - уже обновлённая.

В POST-методе у вас принимается модель Gallery с обновленными свойствами, но идентификатор же не обновлен, верно?
Можно сделать по такому принципу:

Кликните здесь для просмотра всего текста

C#
1
2
3
4
5
6
7
8
9
10
11
12
[HttpPost]
        public async Task<IActionResult> EditGal (Gallery gallery)
        {
            string oldGalleryName = await db.Galleries.FirstOrDefaultAsync(x=>x.ID == id).Name;
            string path = @"wwwroot/images/";
            if (Directory.Exists(path + gallery.Name))
                Directory.Move(oldGalleryName, path+gallery.Name);
 
            db.Galleries.Update(gallery);
            await db.SaveChangesAsync();
            return RedirectToAction("gallery");
        }

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

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

А оперирование файлами на сервере вручную необходимо?

@Jocker · 12.11.2019, 12:13 **[ТС]**

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

В POST-методе у вас принимается модель Gallery с обновленными свойствами, но идентификатор же не обновлен, верно?

Да, у меня была идея сделать так, как Вы говорите. Ваш код не работает( компилятор выдаёт ошибку и не видит свойства Name. Похожим образом я делал:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public async Task<IActionResult> EditGal (Gallery gallery)
        {
            var old = await db.Galleries.FirstOrDefaultAsync(x => x.ID == gallery.ID);            
                
            string path = @"wwwroot/images/";
            if (Directory.Exists(Path.Combine(path + old.Name)))
                Directory.Move(path+old.Name, path+gallery.Name);
            old = null;
 
            db.Galleries.Update(gallery);
            await db.SaveChangesAsync();
            return RedirectToAction("gallery");
        }

Но выдало исключение (на скрине)

@bodynar · 12.11.2019, 12:54

Jocker, На какой именно строчке выпадает исключение?
И корректно ли получается сущность из db.Galleries.FirstOrDefaultAsync?

@Jocker · 12.11.2019, 18:11 **[ТС]**

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

Jocker, На какой именно строчке выпадает исключение?
И корректно ли получается сущность из db.Galleries.FirstOrDefaultAsync?

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

C#
1
 db.Entry(entity: old).State = EntityState.Detached;

Весь метод (немного другой, этот чуть сложнее, но сделан по аналогии):

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public async Task<IActionResult> EditImg(Image image)
        {
            // Changing path and file location
            var old = await db.Images.FirstOrDefaultAsync(x => x.ID == image.ID);
 
            if (old.GalleryName!=image.GalleryName)
            {
                image.Path = old.Path.Replace(old.Path.Split('/')[2], image.GalleryName);
                System.IO.File.Move("wwwroot" + old.Path, "wwwroot" + image.Path);              
            }
            db.Entry(entity: old).State = EntityState.Detached;
 
            // Updating database
            db.Images.Update(image);
            await db.SaveChangesAsync();
            return RedirectToAction("image");
        }

Добавлено через 8 минут
Если описать мои методы создания, редактирования и удаления более детально, то:
1) при создании галереи, создаётся папка в директории "wwwroot/images" с названием галереи.
2) при переименовании галереи, переименовывается папка, менятеся свойство Path и GalleryName у картинки (пришла идея привязать свойство не к имени, а к идентификатору галереи)
3) при удалении галереи - удаляется директория, связанная с ней.

У картинки аналогично.

Новые блоги и статьи Все статьи Все блоги /
Конвертировать закладки 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. На борту пять. . .	Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера 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. Пошагово создадим проект для загрузки изображения. . .

@bodynar 345 / 307 / 135 Регистрация: 14.03.2015 Сообщений: 1,158 Записей в блоге: 1
	12.11.2019, 10:24
	Судя по `[HttpPost]` это у вас web-приложение, где на каждый запрос создается новый экземпляр контроллера / контекста / etc => следовательно ваше значение oldGallery осталось в предыдущем экземпляре контроллера 1

@bodynar 345 / 307 / 135 Регистрация: 14.03.2015 Сообщений: 1,158 Записей в блоге: 1
	12.11.2019, 11:18
	Jocker, а зачем вам именовать папку как галлерею? Почему бы не задавать название как идентификатор? Тогда и переименновывать не нужно. А если обязательное условие название галлереи в названии папки, то передавайте старые и обновленные данные. Но если вы передаете модель с идентификатором - вытащите из бд не обновленные данные - и будет вам `oldGalleryName` 1

@bodynar 345 / 307 / 135 Регистрация: 14.03.2015 Сообщений: 1,158 Записей в блоге: 1
	12.11.2019, 12:54
	Jocker, На какой именно строчке выпадает исключение? И корректно ли получается сущность из `db.Galleries.FirstOrDefaultAsync`? 1