if (int index = 1, index < strLength-1, index++)
	{
                if (emailValid[index].equals("@"))
                {
                        return true;
                }
                 else
                 {
                          if (index == strLength-2)
                         {
                               return false;
                         }
                 }
	}

//String email_String = Validator.getString(sc, "Enter Email Address: ");
System.out.println("Enter Email Address: ");
String email = sc.next();
int strLength = email.length();
System.out.println("" + strLength);
Validator.validateEmail(email, strLength);
c.setEmail(email);

 
public boolean validateEmail(String emailValid, int strLength)
	{
		isvalid = false;
 
		while ((emailValid[0].equals('@'))
				|| (emailValid[emailValid - 1].equals('@')))
		{
			System.out.println("ERROR! you cannot enter @ at the beginning or the end of your email address.\n");
		}
 
			if (int index = 1, index < strLength-1, index++)
			{
		                if (emailValid[index].equals("@"))
		                {
		                        return true;
		                }
		                 else
		                 {
		                          if (index == strLength-2)
		                         {
		                               return false;
		                         }
		                 }
	}
 
		return isvalid;
	}

// You do not need to pass in the length, a String object already knows its own length
// Also you appear to be 'stuck' in a static context (in 'main' I expect), you will need to make this method static or create an Object instance in 'main'
public boolean validateEmail(String emailValid, int strLength)
{
   // This is not declared and also never altered, why not just 'return false' at the end
   isvalid = false;
 
   // This should not be a 'while' loop as it is a one off check, use an 'if' instead
   // emailValid - 1 should be emailValid.length() - 1
   while ((emailValid[0].equals('@')) || (emailValid[emailValid - 1].equals('@'))) {
	  System.out.println("ERROR! you cannot enter @ at the beginning or the end of your email address.\n");
   }
 
   // This should be a 'for' loop not an 'if' statement
   // You should really brush up on your control statements. I.E. if, while, for etc ...
   if (int index = 1, index < strLength-1, index++) {
	  if (emailValid[index].equals("@")) {
		 return true;
	  }
	  // Why not just loop until strLength-2
	  else {
		 if (index == strLength-2) {
			return false;
		 }
	  }
   }
 
   // Just return false instead
   return isvalid;
}

public boolean validateEmail (String asAddress)
{
   if (asAddress == null) { return false; }
 
   if ((asAddress[0].equals('@')) || (asAddress[(asAddress.length()-1)].equals('@'))) {
	  System.out.println("ERROR! you cannot enter @ at the beginning or the end of your email address.");
   }
 
   for (int index = 1, index < (asAddress.length()-2), index++) {
	  if (asAddress[index].equals("@")) {
		 return true;
	  }
   }
 
   return false;
}

public boolean validateEmail (String asAddress)
{
   if (asAddress == null) {
	  System.out.println("Error: Email Address: Is null");
	  return false;
   }
 
   if (asAddress.startsWith('@')) {
	  System.out.println("Error: Email Address: Starts with '@'");
	  return false;
   }
 
   if (asAddress.endsWith('@')) {
	  System.out.println("Error: Email Address: Ends with '@'");
	  return false;
   }
 
   if (asAddress.indexOf('@') == -1) {
	  System.out.println("Error: Email Address: Must contain '@'");
	  return false;
   }
 
   return true;
}

 
	   if (asAddress.indexOf("@") == -1) {
		  System.out.println("Error: Email Address: Must contain '@'");
		  return false;
	   }