Как сказать «ничего не делаем» используя условные операторы?

Question

[My1Name]

Есть некоторый цикл в котором нужно проверить имена файлов и, в случае совпадения, указать "ничего не делаем":

String imgName;
for(byte i=0; i<arrayImages.size();i++) {
imgName = arrayImages.get(i).getOriginalFilename().toLowerCase();
if (imgName.indexOf("s-1.")!=-1 || imgName.indexOf("s-2.")!=-1 || 
    imgName.indexOf("s-3.")!=-1 || imgName.indexOf("s-4.")!=-1) {
	// ничего не делаем 
}
else if (imgName.isBlank())
   removeSlide(i);
else {
	imgName = setCorrectName(imgName, i);
	if (imgName!=null) {
     // что-то делаем
        }
}

Как это написать правильно?

Answer 1

[Сергей Соколов]

несколько проверок с indexOf() можно заменить на одну imgName.matches(/s-[1-4]\./),

условие isBlank() лучше ставить первым

пустой блок если нужен для брейкпоинта, можно там поставить строчку assert true;

Answer 2

[nApoBo3]

Continue

Но вообще считается "дурным" тоном использовать не "основное" поведение внутри if.

Answer 3

[Akina]

в случае совпадения, указать "ничего не делаем"

Реверсируйте условие и ничего не делайте в случае НЕсовпадения, т.е. в ELSE. Которое в этом случае можно вообще не писать.

Answer 4

[Сергей Водаков]

String imgName;

for(byte i=0; i<arrayImages.size();i++) {
	
	imgName = arrayImages.get(i).getOriginalFilename().toLowerCase();
	
	if (!(imgName.contains("s-1.") || imgName.contains("s-2.") || 
		imgName.contains("s-3.") || imgName.contains("s-4."))) {
			if (imgName.isBlank())
				removeSlide(i);
			else {
				imgName = setCorrectName(imgName, i);
				if (imgName!=null) {
					// что-то делаем
				}
			}
	}
	// ничего не делаем 
}

и если бы сделать так, что setCorrectName не будет возвращать null (как вообще функция с таким именем может это сделать?) то еще оду проверку можно выкинуть.

Comments

[My1Name]

как вообще функция с таким именем может это сделать

- Неправильный формат файла.

[Сергей Водаков]

My1Name, я имею ввиду, что в качественном коде, имя функции должно соответствовать тому, что она делает. Подробнее об этом в книге "Чистый код: создание, анализ и рефакторинг." Роберта Мартина.

[Сергей Водаков]

Кстати, вообще всю проверку

!(imgName.contains("s-1.") || imgName.contains("s-2.") || 
		imgName.contains("s-3.") || imgName.contains("s-4."))

стоило бы вывести в отдельную функцию: isIncorrectFileName

[My1Name]

Сергей Водаков,

вывести в отдельную функцию: isIncorrectFileName

В том-то и дело: Это как раз correctFileName

[Сергей Водаков]

My1Name, у вас надо что-то делать, если имя файла некорректное, следовательно, нужно написать функцию, которая будет возвращать true если имя файла некорректное. Подходящее имя для такой функции будет
isIncorrectFileName тогда код будет примерно таким:

String imgName;

for(byte i=0; i<arrayImages.size();i++) {
	
	imgName = arrayImages.get(i).getOriginalFilename().toLowerCase();
	
	if (isIncorrectFileName(imgName)) {
		if (imgName.isBlank())
			removeSlide(i);
		else {
			imgName = setCorrectName(imgName, i);
			if (imgName!=null) {
				// что-то делаем
			}
		}
	}
	// ничего не делаем 
}

Answer 5

[Dmitry Roo]

1. Можно вообще ничего не писать.
2. Можно написать continue; но в этом нет необходимости
3. Лучше всего вообще не использовать циклы:

arrayImages.stream()
                .map(arrayImage -> arrayImage.getOriginalFilename().toLowerCase())
                .filter(imgName -> !imgName.matches(".*s-1\\..*|.*s-2\\..*|.*s-3\\..*|.*s-4\\..*")
                .forEach(imgName -> {
                    // Что-то делаем
                    if (imgName.isBlank())
                       // removeSlide(i);
                    else {
                       // imgName = setCorrectName(imgName, i);
                       // if (imgName!=null) {
                            // что-то делаем
                       // }
                    }
                });

Comments

[nApoBo3]

От куда у вас в stream "i" взялся?

[My1Name]

nApoBo3, можно завести переменную перед входом в stream. Но где её потом инкрементировать - я тоже не знаю...

[Dmitry Roo]

nApoBo3, резонный вопрос. Но я и не знаю остального кода, что там принимает removeSlide. Надеюсь, его можно адаптировать под имя файла. Так же как и setCorrectName(imgName, i);
В люьом случае, код указан как пример.

[My1Name]

Dmitry Roo, на сколько правильно написано ваше регулярное выражение?

Пока не один из ответов к вопросу не работает в полной мере правильно.

[My1Name]

Только с continue; в том коде, который сопровождает вопрос, всё работает правильно.

Хотел использовать регулярное выражение, но не одно из них не работает правильно. Dmitry Roo, ваше уже не проверял... Отчаялся xd

[Dmitry Roo]

My1Name, правильно для тех случаев, которые я сам придумал. Вопрос не об этом был, это так, факультативно. Регулярки можно потестировать на https://regex101.com/

[My1Name]

Dmitry Roo, хорошая ссылка. Спасибо. Добавил Pattern и с таким вот выражением ^s-[1-4].$ matcher работает. Но, происходит что-то странное... Похоже на "проскальзывание" основного потока программы до завершения проверки условия... Это конечно уже другой вопрос, так что в любом случае - спасибо.

Answer 6

[Rsa97]

String imgName;
for (byte i = 0; i < arrayImages.size(); i++) {
  imgName = arrayImages.get(i).getOriginalFilename().toLowerCase();
  if (imgName.isBlank())
    removeSlide(i);
    continue;
  }
  if (imgName.matches(".*s-[1-4]\\..*")) {
    continue;
  }
  imgName = setCorrectName(imgName, i);
  if (imgName == null) {
    continue;
  }
  // что-то делаем
}

Comments

[My1Name]

if (imgName!= null) {
      // что-то делаем
  }

Мне нравится решение с регулярным выражением. Можно ли заменить continue на assert true; ?

[Rsa97]

My1Name,
1. Конкретный вариант зависит от того, что именно в "что-то делаем". Если там больше нет вложенности, то можно и внутрь if код поместить. Если есть, то чем меньше вложенность, тем проще читать программу.
2. Заменить нельзя, это принципиально разные команды.

Answer 7

[My1Name]

Только так:

String imgName;
for(byte i=0; i<arrayImages.size();i++) {
imgName = arrayImages.get(i).getOriginalFilename().toLowerCase();
if (imgName.isBlank())
    removeSlide(i);
else if (imgName.indexOf("s-1.")!=-1 || imgName.indexOf("s-2.")!=-1 ||
         imgName.indexOf("s-3.")!=-1 || imgName.indexOf("s-4.")!=-1) {
	 assert true; // file exists. Nothing to do.
	}
else {
 	imgName = setCorrectName(imgName, i);
	if (imgName!=null) {
       // some code...
        }
}

Пробовал разные варианты из ответов... Хотел заменить indexOf() на регулярное выражение matches но оно ни в какую не хочет корректно работать.