Finding the amount of values within two numbers in python
Only started coding a few weeks ago but I've been having trouble seeing where I went wrong.
Goal is to write a function that takes a list of numbers as a parameter and returns the number of values that are within the numbers 24.54 & 47.54.
def count_in_range(x):
sum = 0
for i in x:
if x > 24.54 and x < 47.54:
sum = sum + 1
return sum
Currently getting a "unorderable types: list() > float() error
python list numbers
|
show 1 more comment
Only started coding a few weeks ago but I've been having trouble seeing where I went wrong.
Goal is to write a function that takes a list of numbers as a parameter and returns the number of values that are within the numbers 24.54 & 47.54.
def count_in_range(x):
sum = 0
for i in x:
if x > 24.54 and x < 47.54:
sum = sum + 1
return sum
Currently getting a "unorderable types: list() > float() error
python list numbers
1
if x > 24.54 and x < 47.54:
should beif i > 24.54 and i < 47.54:
because you are iterating the list ofx
with elementi
being retrieved.
– MooingRawr
Nov 22 '18 at 20:39
Also, yourreturn sum
should be outside the loop.
– John Anderson
Nov 22 '18 at 20:42
You would never have coded this bug with meaningful variable names. For example,list_of_numbers
instead ofx
andnumber
instead ofi
.
– timgeb
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
Did you read what John has to say about moving thereturn sum
outside of the loop
– MooingRawr
Nov 22 '18 at 20:44
|
show 1 more comment
Only started coding a few weeks ago but I've been having trouble seeing where I went wrong.
Goal is to write a function that takes a list of numbers as a parameter and returns the number of values that are within the numbers 24.54 & 47.54.
def count_in_range(x):
sum = 0
for i in x:
if x > 24.54 and x < 47.54:
sum = sum + 1
return sum
Currently getting a "unorderable types: list() > float() error
python list numbers
Only started coding a few weeks ago but I've been having trouble seeing where I went wrong.
Goal is to write a function that takes a list of numbers as a parameter and returns the number of values that are within the numbers 24.54 & 47.54.
def count_in_range(x):
sum = 0
for i in x:
if x > 24.54 and x < 47.54:
sum = sum + 1
return sum
Currently getting a "unorderable types: list() > float() error
python list numbers
python list numbers
asked Nov 22 '18 at 20:38
lumelume
1
1
1
if x > 24.54 and x < 47.54:
should beif i > 24.54 and i < 47.54:
because you are iterating the list ofx
with elementi
being retrieved.
– MooingRawr
Nov 22 '18 at 20:39
Also, yourreturn sum
should be outside the loop.
– John Anderson
Nov 22 '18 at 20:42
You would never have coded this bug with meaningful variable names. For example,list_of_numbers
instead ofx
andnumber
instead ofi
.
– timgeb
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
Did you read what John has to say about moving thereturn sum
outside of the loop
– MooingRawr
Nov 22 '18 at 20:44
|
show 1 more comment
1
if x > 24.54 and x < 47.54:
should beif i > 24.54 and i < 47.54:
because you are iterating the list ofx
with elementi
being retrieved.
– MooingRawr
Nov 22 '18 at 20:39
Also, yourreturn sum
should be outside the loop.
– John Anderson
Nov 22 '18 at 20:42
You would never have coded this bug with meaningful variable names. For example,list_of_numbers
instead ofx
andnumber
instead ofi
.
– timgeb
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
Did you read what John has to say about moving thereturn sum
outside of the loop
– MooingRawr
Nov 22 '18 at 20:44
1
1
if x > 24.54 and x < 47.54:
should be if i > 24.54 and i < 47.54:
because you are iterating the list of x
with element i
being retrieved.– MooingRawr
Nov 22 '18 at 20:39
if x > 24.54 and x < 47.54:
should be if i > 24.54 and i < 47.54:
because you are iterating the list of x
with element i
being retrieved.– MooingRawr
Nov 22 '18 at 20:39
Also, your
return sum
should be outside the loop.– John Anderson
Nov 22 '18 at 20:42
Also, your
return sum
should be outside the loop.– John Anderson
Nov 22 '18 at 20:42
You would never have coded this bug with meaningful variable names. For example,
list_of_numbers
instead of x
and number
instead of i
.– timgeb
Nov 22 '18 at 20:43
You would never have coded this bug with meaningful variable names. For example,
list_of_numbers
instead of x
and number
instead of i
.– timgeb
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
Did you read what John has to say about moving the
return sum
outside of the loop– MooingRawr
Nov 22 '18 at 20:44
Did you read what John has to say about moving the
return sum
outside of the loop– MooingRawr
Nov 22 '18 at 20:44
|
show 1 more comment
1 Answer
1
active
oldest
votes
When you check the value of each item in the list, you're inadvertently checking the value of the entire list instead.
if x > 24.54 and x < 47.54:
should become
if i > 24.54 and x < 47.54:
because i
is the variable that takes on the value of each item in the list as you iterate through.
Also, you want to move your return statement to outside the loop, otherwise the loop will terminate after 1 iteration.
add a comment |
Your Answer
StackExchange.ifUsing("editor", function () {
StackExchange.using("externalEditor", function () {
StackExchange.using("snippets", function () {
StackExchange.snippets.init();
});
});
}, "code-snippets");
StackExchange.ready(function() {
var channelOptions = {
tags: "".split(" "),
id: "1"
};
initTagRenderer("".split(" "), "".split(" "), channelOptions);
StackExchange.using("externalEditor", function() {
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled) {
StackExchange.using("snippets", function() {
createEditor();
});
}
else {
createEditor();
}
});
function createEditor() {
StackExchange.prepareEditor({
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: true,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
imageUploader: {
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
},
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
});
}
});
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53437746%2ffinding-the-amount-of-values-within-two-numbers-in-python%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
1 Answer
1
active
oldest
votes
1 Answer
1
active
oldest
votes
active
oldest
votes
active
oldest
votes
When you check the value of each item in the list, you're inadvertently checking the value of the entire list instead.
if x > 24.54 and x < 47.54:
should become
if i > 24.54 and x < 47.54:
because i
is the variable that takes on the value of each item in the list as you iterate through.
Also, you want to move your return statement to outside the loop, otherwise the loop will terminate after 1 iteration.
add a comment |
When you check the value of each item in the list, you're inadvertently checking the value of the entire list instead.
if x > 24.54 and x < 47.54:
should become
if i > 24.54 and x < 47.54:
because i
is the variable that takes on the value of each item in the list as you iterate through.
Also, you want to move your return statement to outside the loop, otherwise the loop will terminate after 1 iteration.
add a comment |
When you check the value of each item in the list, you're inadvertently checking the value of the entire list instead.
if x > 24.54 and x < 47.54:
should become
if i > 24.54 and x < 47.54:
because i
is the variable that takes on the value of each item in the list as you iterate through.
Also, you want to move your return statement to outside the loop, otherwise the loop will terminate after 1 iteration.
When you check the value of each item in the list, you're inadvertently checking the value of the entire list instead.
if x > 24.54 and x < 47.54:
should become
if i > 24.54 and x < 47.54:
because i
is the variable that takes on the value of each item in the list as you iterate through.
Also, you want to move your return statement to outside the loop, otherwise the loop will terminate after 1 iteration.
answered Nov 22 '18 at 20:44
theasianpianisttheasianpianist
8819
8819
add a comment |
add a comment |
Thanks for contributing an answer to Stack Overflow!
- Please be sure to answer the question. Provide details and share your research!
But avoid …
- Asking for help, clarification, or responding to other answers.
- Making statements based on opinion; back them up with references or personal experience.
To learn more, see our tips on writing great answers.
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
StackExchange.ready(
function () {
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53437746%2ffinding-the-amount-of-values-within-two-numbers-in-python%23new-answer', 'question_page');
}
);
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Sign up or log in
StackExchange.ready(function () {
StackExchange.helpers.onClickDraftSave('#login-link');
});
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Sign up using Google
Sign up using Facebook
Sign up using Email and Password
Post as a guest
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
Required, but never shown
1
if x > 24.54 and x < 47.54:
should beif i > 24.54 and i < 47.54:
because you are iterating the list ofx
with elementi
being retrieved.– MooingRawr
Nov 22 '18 at 20:39
Also, your
return sum
should be outside the loop.– John Anderson
Nov 22 '18 at 20:42
You would never have coded this bug with meaningful variable names. For example,
list_of_numbers
instead ofx
andnumber
instead ofi
.– timgeb
Nov 22 '18 at 20:43
hmm that fixed the unorderable types error but now it's just returning an incorrect output
– lume
Nov 22 '18 at 20:43
Did you read what John has to say about moving the
return sum
outside of the loop– MooingRawr
Nov 22 '18 at 20:44