有一个逻辑问没搞明白，求教

[豆芽炒粉丝]

如果满足则执行上传操作。

public final static String[] exts = { "jpg", "png", "gif" };

for (String e : exts) {
			if (!ext.equals(e)) {
				return false;
			}
		}
		return true;

这个判断句中，我上传的文件后缀是jpg，如果不存在于exts中，则返回false ，最终返回true，这么一说的话，jgp存在于exts中，那么结果返回的是true，那么就能执行上传操作了，但我打印出b的值之后确是false，不知道为什么

if (b) {
					fos = new FileOutputStream(new File(PATH + "/img/" + fname));
					System.out.println(PATH + "/img/" + fname);
					fos.write(is, 0, is.length);

[tony4geek]

debug 调试吧。代码不全猜不出来哪里 的原因。不知的 你的b 是怎么来的。

[豆芽炒粉丝]

自己搞明白了，是因为for-each语句的原因，会依次向下执行，jgp会与jpg，png，img等格式对比，而jpg与png，img不一样，这样就会返回false了

[nicholasbobo]

你这样判断有问题，你是判断数组里的每个元素都必须同传进来的后缀相同才能返回true,其中有一个不同都会返回false，正确的写法应该是

for (String e : exts) {
            if (ext.equals(e)) {
                return true;
            }
        }
return false;

[豆芽炒粉丝]

引用 1 楼 rui888 的回复:

debug 调试吧。代码不全猜不出来哪里 的原因。不知的 你的b 是怎么来的。

b是一个boolean变量 而b是检查文件类型的 也就是for-each那条语句

[豆芽炒粉丝]

引用 3 楼 nicholasbobo 的回复:

你这样判断有问题，你是判断数组里的每个元素都必须同传进来的后缀相同才能返回true,其中有一个不同都会返回false，正确的写法应该是

for (String e : exts) {
            if (ext.equals(e)) {
                return true;
            }
        }
return false;

我自己也发现了，刚开始没想通呢

[逝_去_的_光_阴]

for (String e : exts) {
            if (ext.equals(e)) {
                return true;
            }
        }
        return false;

明显逻辑反了 遍历里判断true 如果没有就返回false

[王子文龙]

引用 2 楼 dingchenxixi 的回复:

自己搞明白了，是因为for-each语句的原因，会依次向下执行，jgp会与jpg，png，img等格式对比，而jpg与png，img不一样，这样就会返回false了

你应该创建一个标志位来表示是否存在，存在返回true，不存在继续遍历直到数组最后一个值，此时如果还不存在才应该返回false，简单点你创建一个List代替数组，判断元素是否存在不容易吗