StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

POI can't figure out what's wrong, my logic seems correct
text
Body
copied!<p>I know that what's messing up is my checkIfEdu method. Everything else is fine because I've tried it without that method. However, it's reading all the data into the InvalidStudentEmails file and its because of that method! i know it, everything else if fine because when I take that method out, it does everything correctly. Please help!!!</p> <pre><code>import java.util.Scanner; public class CheckStudentEmail { public static void main (String[] args) throws Exception { java.io.File filein = new java.io.File("StudentEmailData.txt"); java.io.File fileout1 = new java.io.File("ValidStudentEmails.txt"); java.io.File fileout2 = new java.io.File("InvalidStudentEmails.txt"); Scanner input = new Scanner(filein); java.io.PrintWriter validOutput = new java.io.PrintWriter(fileout1); java.io.PrintWriter invalidOutput = new java.io.PrintWriter(fileout2); writeHeadings(validOutput, invalidOutput); StudentEmail student = new StudentEmail(); while (input.hasNext()) { student.readStudentData(input); if (student.checkPeriod() == true && student.checkAtSymbol() == true && student.checkIfEdu() == true) student.writeEmails(validOutput); else student.writeEmails(invalidOutput); } input.close(); validOutput.close(); invalidOutput.close(); } public static void writeHeadings(java.io.PrintWriter validOutput, java.io.PrintWriter invalidOutput) { validOutput.printf("%-20s%20s", "Name", "Email Address (Valid)"); validOutput.println(); validOutput.println(); invalidOutput.printf("%-20s%20s", "Name", "Email Address (Invalid)"); invalidOutput.println(); invalidOutput.println(); } } </code></pre> <p>Here are my methods</p> <pre><code>import java.util.Scanner; public class StudentEmail { private String stuName; private String stuEmail; StudentEmail() {} StudentEmail(String name, String email) { stuName = name; stuEmail = email; } public void readStudentData(Scanner input) { stuName = input.next(); stuEmail = input.next(); } public boolean checkPeriod() { if (stuEmail.indexOf(".") != -1) return true; else return false; } public boolean checkAtSymbol() { int atSymbol; if (stuEmail.indexOf('@') != -1) { atSymbol = stuEmail.indexOf('@'); if (stuEmail.indexOf('@', atSymbol+1) != -1) return false; else return true; } else return false; } public boolean checkIfEdu() { int lengthOfEmail = stuEmail.length(); int position = lengthOfEmail - 3; String checkWord = stuEmail.substring(position); if (checkWord == "edu") return true; else return false; } public void writeEmails(java.io.PrintWriter output) { output.printf("%-20s%20s", stuName, stuEmail); output.println(); } } </code></pre>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload