fix: changes the constructor code

AmiyahJo · AmiyahJo · commit 5fa42dc4f7f9 · 2024-11-05T14:01:30.000Z
moves the boolean for hasBusinessOwner to simply if one owner isn't business
diff --git a/lesson_17/bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/BusinessCheckingAccount.java b/lesson_17/bank/bank_app/src/main/java/com/codedifferently/lesson17/bank/BusinessCheckingAccount.java
@@ -15,12 +15,11 @@ public class BusinessCheckingAccount extends CheckingAccount{
      */
     public BusinessCheckingAccount(String accountNumber, Set<Customer> owners, String businessName, double initialBalance) {
         super(accountNumber, owners, initialBalance);
+        this.businessName = businessName;
 
-        boolean hasBusinessOwner = owners.stream().anyMatch(Customer::isBusiness);
-        if (!hasBusinessOwner) {
+        if (owners.stream().noneMatch(owner -> owner.isBusiness())) {
             throw new IllegalArgumentException("At least one owner must be a business.");
         }
-        this.businessName = businessName;
     }   
 
     public String getBusinessName() {

Original file line number	Diff line number	Diff line change
`@@ -15,12 +15,11 @@ public class BusinessCheckingAccount extends CheckingAccount{`
`15`	`15`	`*/`
`16`	`16`	`public BusinessCheckingAccount(String accountNumber, Set<Customer> owners, String businessName, double initialBalance) {`
`17`	`17`	`super(accountNumber, owners, initialBalance);`
	`18`	`+ this.businessName = businessName;`
`18`	`19`
`19`		`- boolean hasBusinessOwner = owners.stream().anyMatch(Customer::isBusiness);`
`20`		`- if (!hasBusinessOwner) {`
	`20`	`+ if (owners.stream().noneMatch(owner -> owner.isBusiness())) {`
`21`	`21`	`throw new IllegalArgumentException("At least one owner must be a business.");`
`22`	`22`	`}`
`23`		`- this.businessName = businessName;`
`24`	`23`	`}`
`25`	`24`
`26`	`25`	`public String getBusinessName() {`