Thread: Not Sure Whats Wrong

import java.util.*;
 
public class Main
{
	public static boolean isPart;
	public static String x[] = {"One", "Two", "Three", "Four", "Five"};
	public static void main(String[] args)
	{
 
		isPartOfArray();
 
	}
	public static boolean isPartOfArray()
	{
		Scanner scan = new Scanner(System.in);
		System.out.println("What is your name?");
		String input = scan.nextLine();
		System.out.println(input);
		int i;
		for(i = 0; i< 4; i++)
		{
 
		}
		if(input.equals(x[i]))
		{
			System.out.println("Name recognized");
			isPart = true;
		}
		else
		{
			System.out.println("Name not recognized");
			isPart = false;
		}
		return isPart;
	}
}

For some reason it's. always returning false even if an item from the array x is entered.

Why is the for() loop empty?

So why not let the OP investigate and answer?

Originally Posted by Time4Java

You may want to look over the for loop you are using to access the array, see if you find something there
Also, you should post your code within code tags, that makes it easier to see.

I have tweaked it And pondered on it for hours and cannot figure it out.

Edit: Lmao nvm

--- Update ---

import java.util.*;
 
public class Main
{
	public static boolean isPart;
	public static String j;
	public static String x[] = {"One", "Two", "Three", "Four", "Five"};
	public static void main(String[] args)
	{
 
		isPartOfArray();
 
	}
	public static boolean isPartOfArray()
	{
		Scanner scan = new Scanner(System.in);
		System.out.println("What is your name?");
		String input = scan.nextLine();
		System.out.println(input);
		int i;
		for(i = 0; i< 4; i++)
		{
			String j = x[i];
		}
		if(input.equals(j))
		{
			System.out.println("Name recognized");
			isPart = true;
		}
		else
		{
			System.out.println("Name not recognized");
			isPart = false;
		}
		return isPart;
	}
}

Still not working.

The curly brackets { and } have a meaning in java. The way you use them is the source of your problem. You have misplaced the brackets for your loop and thus your program does not do what you would think it does.

I'm not seeing any place where the curley brackets are used incorrectly. I'm guessing you are talking about a scope issue?

Yes exactly.
You have 5 items in your array. You have your loop run from 0 to 3 which is 4 indices. But then you only check for the fourth item in your if-condition. This happens because the if-then-else is not within the for-loop. They are both seperated and dont interact with each other at all. This is why Greg pointed out that your loop is actually empty and doesnt do anything.

I actually had that initially but I removed it from inside the for loop because I was getting an exception and I tried to see if it would help removing it from there.

--- Update ---

I actually had that initially but I removed it from inside the for loop because I was getting an exception and I tried to see if it would help removing it from there.

Can someone help me with my bank account program? I finished the code, but I want to make sure everything is correct.

My apologies. How do I do that?