Welcome to the Treehouse Community
Want to collaborate on code errors? Have bugs you need feedback on? Looking for an extra set of eyes on your latest project? Get support with fellow developers, designers, and programmers of all backgrounds and skill levels here with the Treehouse Community! While you're at it, check out some resources Treehouse students have shared here.
Looking to learn something new?
Treehouse offers a seven day free trial for new students. Get access to thousands of hours of content and join thousands of Treehouse students and alumni in the community today.
Start your free trialHarshit Dixit
519 PointsI am not able to use the right method for same. I am understanding the concept but really having issues coding it.
var results: [Int] = []
for n in 1...100 { if (n!=0) && (n%7==0) { results.append(n) } }
var results: [Int] = []
for n in 1...100 {
if (n!=0) && (n%7==0) {
results.append(n)
}
}
1 Answer
Jason Anders
Treehouse Moderator 145,860 PointsYou are on the right track, and the conditional has the correct syntax. You are, however, not checking the conditions the instructions are asking for. You have the second part correct to see if it is a multiple of 7, but the first part is only checking if n
doesn't equal zero... but the instructions are asking you to check to see if n
is an odd number, so there is a vital piece of logic missing. To check if a number is odd, the modulo cannot be zero when divided by 2 (here's where you'd use the not equal to
operator. So, it would look something like:
n % 2 != 0
Once you fix that up, the rest is good to go.
Also, a quick note/tip. Watch your spacing in code. While whitespace doesn't really matter with swift, for readability it is best practice to use spacing between characters in code. So instead of:
n%2!=0 //which is very hard to read
do:
n % 2 != 0 //which is very easy to read
Otherwise, nice work! :)