[CHALLENGE24][GCI95][S.T.E.P] : Use every column queried

src/it/java/org/greencodeinitiative/creedengo/java/integration/tests/GCIRulesIT.java

@@ -548,4 +548,48 @@ void testGCI94() {
         checkIssuesForFile(filePath, ruleId, ruleMsg, startLines, endLines, SEVERITY, TYPE, EFFORT_1MIN);
     }
 
+    @Test
+    void testGCI1044_1() {
+        String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/AttributeQueryNonCompliant.java";
+        String ruleId= "creedengo-java:GCI1044";
+        String ruleMsg = "Avoid querying SQL columns that are not used";
+        int[] startLines = new int[]{41};
+        int[] endLines = new int[]{41};
+    }
+
+    @Test
+    void testGCI1044_2() {
+        String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/LitteralQueryNonCompliant.java";
+        String ruleId= "creedengo-java:GCI1044";
+        String ruleMsg = "Avoid querying SQL columns that are not used";
+        int[] startLines = new int[]{40};
+        int[] endLines = new int[]{40};
+    }
+
+    @Test
+    void testGCI1044_3() {
+        String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/MultipleQueriesNonCompliant.java";
+        String ruleId= "creedengo-java:GCI1044";
+        String ruleMsg = "Avoid querying SQL columns that are not used";
+        int[] startLines = new int[]{45};
+        int[] endLines = new int[]{45};
+    }
+
+    @Test
+    void testGCI1044_4() {
+        String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/UseColumnIdsAndNameAttributesNonCompliant.java";
+        String ruleId= "creedengo-java:GCI1044";
+        String ruleMsg = "Avoid querying SQL columns that are not used";
+        int[] startLines = new int[]{42};
+        int[] endLines = new int[]{42};
+    }
+
+    @Test
+    void testGCI1044_5() {
+        String filePath = "src/main/java/org/greencodeinitiative/creedengo/java/checks/UseMethodNonCompliant.java";
+        String ruleId= "creedengo-java:GCI1044";
+        String ruleMsg = "Avoid querying SQL columns that are not used";
+        int[] startLines = new int[]{40};
+        int[] endLines = new int[]{40};
+    }
 }

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/AttributeQueryNonCompliant.java

@@ -0,0 +1,53 @@
+/*
+ * ecoCode - Java language - Provides rules to reduce the environmental footprint of your Java programs
+ * Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.greencodeinitiative.creedengo.java.checks;
+
+import java.sql.Connection;
+import java.sql.DriverManager;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+
+/**
+ * This is the nominal test case, where the SQL query is an attribute of the class.
+ * One field is not accesed, so an issue is raised
+ */
+public class AttributeQueryNonCompliant {
+
+	private static final String DB_URL = "jdbc:mysql://localhost/TEST";
+	private static final String USER = "guest";
+	private static final String PASS = "guest123";
+	private static final String QUERY = "SELECT id, first, last, age FROM Registration";
+
+	public void callJdbc() {
+
+		try (Connection conn = DriverManager.getConnection(DB_URL, USER, PASS);
+				Statement stmt = conn.createStatement();
+				ResultSet rs = stmt.executeQuery(QUERY);) { // Noncompliant {{Avoid querying SQL columns that are not used}}
+			while (rs.next()) {
+				// Display values
+				System.out.print("ID: " + rs.getInt("id"));
+				System.out.print(", First: " + rs.getString("first"));
+				System.out.println(", Last: " + rs.getString("last"));
+			}
+		} catch (SQLException e) {
+			e.printStackTrace();
+		}
+
+	}
+}

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/LitteralQueryNonCompliant.java

@@ -0,0 +1,54 @@
+/*
+ * ecoCode - Java language - Provides rules to reduce the environmental footprint of your Java programs
+ * Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.greencodeinitiative.creedengo.java.checks;
+
+import java.sql.Connection;
+import java.sql.DriverManager;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+
+/**
+ * In this test case, the query is a litteral string directly inserted in the method
+ * One field is not accesed, so an issue is raised
+ */
+public class LitteralQueryNonCompliant {
+
+	private static final String DB_URL = "jdbc:mysql://localhost/TEST";
+	private static final String USER = "guest";
+	private static final String PASS = "guest123";
+
+	public void callJdbc() {
+
+		try (Connection conn = DriverManager.getConnection(DB_URL, USER, PASS);
+				Statement stmt = conn.createStatement();
+				ResultSet rs = stmt.executeQuery("SELECT id, first, last, age FROM Registration");) { // Noncompliant {{Avoid querying SQL columns that are not used}}
+			while (rs.next()) {
+				// Display values
+				System.out.print("ID: " + rs.getInt("id"));
+				System.out.print(", First: " + rs.getString("first"));
+				System.out.println(", Last: " + rs.getString("last"));
+			}
+		} catch (SQLException e) {
+			e.printStackTrace();
+		}
+
+	}
+
+
+}

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/MultipleQueriesNonCompliant.java

@@ -0,0 +1,68 @@
+/*
+ * ecoCode - Java language - Provides rules to reduce the environmental footprint of your Java programs
+ * Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.greencodeinitiative.creedengo.java.checks;
+
+import java.sql.Connection;
+import java.sql.DriverManager;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+
+/**
+ * In this test case, multiple queries are done using the same Statement Object.
+ * One field is not accesed, so an issue is raised
+ */
+public class MultipleQueriesNonCompliant {
+
+	private static final String DB_URL = "jdbc:mysql://localhost/TEST";
+	private static final String USER = "guest";
+	private static final String PASS = "guest123";
+	private static final String QUERY = "SELECT id, first, last, age FROM Registration"; 
+	private static final String QUERY2 = "SELECT id, first, last, age FROM Registration2";
+
+
+	public void callJdbc() {
+
+		try (Connection conn = DriverManager.getConnection(DB_URL, USER, PASS);
+				Statement stmt = conn.createStatement();
+				) {
+
+			ResultSet rs = stmt.executeQuery(QUERY); // Noncompliant {{Avoid querying SQL columns that are not used}}
+			while (rs.next()) {
+				// Display values
+				System.out.print("ID: " + rs.getInt("id"));
+				System.out.print(", First: " + rs.getString("first"));
+				System.out.println(", Last: " + rs.getString("last"));
+			}
+			rs = stmt.executeQuery(QUERY2);
+
+			while (rs.next()) {
+				// Display values
+				System.out.print("Age: " + rs.getInt("age"));
+				System.out.print("ID: " + rs.getInt("id"));
+				System.out.print(", First: " + rs.getString("first"));
+				System.out.println(", Last: " + rs.getString("last"));
+			}
+
+
+		} catch (SQLException e) {
+			e.printStackTrace();
+		}
+
+	}
+}

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/UseColumnIdsAndNameAttributesNonCompliant.java

@@ -0,0 +1,54 @@
+/*
+ * ecoCode - Java language - Provides rules to reduce the environmental footprint of your Java programs
+ * Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.greencodeinitiative.creedengo.java.checks;
+
+import java.sql.Connection;
+import java.sql.DriverManager;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+
+/**
+ * In this test case, columns are accesed by IDs and names, some of them being in final variables
+ * One field is not accesed, so an issue is raised
+ */
+public class UseColumnIdsAndNameAttributesNonCompliant {
+
+	private static final String DB_URL = "jdbc:mysql://localhost/TEST";
+	private static final String USER = "guest";
+	private static final String PASS = "guest123";
+	private static final String QUERY = "SELECT id, first, last, age FROM Registration";
+	private static final String ID = "id";
+
+	public void callJdbc() {
+
+		try (Connection conn = DriverManager.getConnection(DB_URL, USER, PASS);
+				Statement stmt = conn.createStatement();
+				ResultSet rs = stmt.executeQuery(QUERY);) { // Noncompliant {{Avoid querying SQL columns that are not used}}
+			while (rs.next()) {
+				// Display values
+				System.out.print("ID: " + rs.getInt(ID));
+				System.out.print(", First: " + rs.getString(2));
+				System.out.println(", Last: " + rs.getString("last"));
+			}
+		} catch (SQLException e) {
+			e.printStackTrace();
+		}
+
+	}
+}

src/it/test-projects/creedengo-java-plugin-test-project/src/main/java/org/greencodeinitiative/creedengo/java/checks/UseMethodNonCompliant.java

@@ -0,0 +1,56 @@
+/*
+ * ecoCode - Java language - Provides rules to reduce the environmental footprint of your Java programs
+ * Copyright © 2023 Green Code Initiative (https://www.ecocode.io)
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+package org.greencodeinitiative.creedengo.java.checks;
+
+import java.sql.Connection;
+import java.sql.DriverManager;
+import java.sql.ResultSet;
+import java.sql.SQLException;
+import java.sql.Statement;
+
+/**
+ * In this test case, the ResultSet is passed through a method
+ * One field is not accesed, so an issue is raised
+ */
+public class UseMethodNonCompliant {
+
+	private static final String DB_URL = "jdbc:mysql://localhost/TEST";
+	private static final String USER = "guest";
+	private static final String PASS = "guest123";
+
+	public void callJdbc() {
+
+		try (Connection conn = DriverManager.getConnection(DB_URL, USER, PASS);
+				Statement stmt = conn.createStatement();
+				ResultSet rs = stmt.executeQuery("SELECT id, first, last, age FROM Registration");) { // Noncompliant {{Avoid querying SQL columns that are not used}}
+			extractGet(rs);
+		} catch (SQLException e) {
+			e.printStackTrace();
+		}
+
+	}
+
+	private void extractGet(ResultSet rs) throws SQLException {
+		while (rs.next()) {
+			// Display values
+			System.out.print("ID: " + rs.getInt("id"));
+			System.out.print(", First: " + rs.getString("first"));
+			System.out.println(", Last: " + rs.getString("last"));
+		}
+	}
+}

src/main/java/org/greencodeinitiative/creedengo/java/JavaCheckRegistrar.java

@@ -50,7 +50,8 @@ public class JavaCheckRegistrar implements CheckRegistrar {
             FreeResourcesOfAutoCloseableInterface.class,
             AvoidMultipleIfElseStatement.class,
             UseOptionalOrElseGetVsOrElse.class,
-            MakeNonReassignedVariablesConstants.class
+            MakeNonReassignedVariablesConstants.class,
+            UseEveryColumnQueried.class
     );
 
     /**