getting an error
Write a function called under_age that takes two positive integer scalar arguments: age that represents someone's age, and limit that represents an age limit. The function returns true if the person is younger than the age limit. If the second arg
315 visualizzazioni (ultimi 30 giorni)
Mostra commenti meno recenti
function too_young = under_age(age,limit)
limit = 21
if age <= limit
too_young = true;
elseif age >= limit
too_young = false;
else
fprintf('invalid\n')
end
16 Commenti
DGM
il 21 Feb 2023
Modificato: DGM
il 29 Ago 2023
Sometimes when clearing out tangled dry underbrush, one is overcome with a profound appreciation for the simple elegance of fire. I think I'd better take a break.
If future me (or anyone else) wants to pick up where I left off, here are my notes on a subset of the answers which very closely follow a common form:
378292 the first of the type, has extra return statement, inconsistent indentation
380788 has extraneous ()
427771 randomly indented
428601 indented, but otherwise identical to 427771
423954 all variable names are replaced with letters
442503 has extra isempty() test, logic is flipped but correct
446375 identical to 428601 excepting whitespace (includes console dump)
477187 identical to 428601 excepting whitespace
477322 trying too hard to say "it's totally different"
478066 identical to 428601 excepting whitespace and unused code as comments
1074038 identical to 428601 excepting whitespace and a comment that says "the shortest way"
EDIT: these also constitute a set of trivial variations of a common answer:
Find the ones that look the most like copypasta, see if user posted other answers, find more threads full of samey copypasta, fall down rabbit hole.
Risposta accettata
John D'Errico
il 23 Mar 2019
Modificato: John D'Errico
il 8 Nov 2020
This is not an error. It is a failure of your code to work as it is supposed to work by your goals.
What did I say? What has EVERYONE said so far? You cannot set a default as you did.
Even though you pass in the number 18 as the limit, the first thing you do inside is reset that value to 21.
Instead, you need to think about how to test to see if limit was provided at all.
Edit: (18 months later)
Now that there are dozens of answers, all of which are lengthy, I'll show how I would write it.
function tooyoung = under_age(age,limit)
% under_age: returns true if the person is under the age limit
% usage: tooyoung = under_age(age,limit)
%
% Arguments: (input)
% age - numeric. As written, age can be scalar, or any size array
%
% limit - optional argument, if not provided, the default is 21
%
% arguments: (output)
% tooyoung - a boolean variable, indicating if age was under the limit.
%
% NO tests are done to verify that age or limit, if provided are valid
% ages itself, or even if they is numeric at all. Better code would
% do these things.
% test for limit having been provided. if not, then the default is 21
if nargin < 2, limit = 21; end
% There is no need to use an if statement. The test itself is the desired result.
tooyoung = age < limit;
end
See that most of my code was actually comments. Help is hugely important, since it allows you to quickly use the code next year when you forget how it was written, or when you forget what the arguments mean.
Think of internal comments as reminders to yourself, for a year from now when you look at the code and need to change the code for some reason, or god forbid, you need to debug it. My recommended target is one line of comment per line of code, or at worst, one line of code per significant thing done in the code.
Comments are free! Good code should look positively green due to all of the comments. My solution code was lengthy only because of all of the comments.
What else?
Remember that white space is hugely important. It makes your code easy to read.
Use intelligent, meaningful variable names. Good mnemonic variable names help to make your code self-documenting, and again, easy to read. When you are scanning through the code, you don't want to continuously go back and be forced to remember what does the variable wxxyy do?
As I said, better code would have included tests to verify that both age and limit, if provided, were actually numeric variables. The best code is friendly code. When it fails, you want the code to fial in a friendly way, telling the person what was seen to be wrong. What you don't want to happen is the code does something screwy and unexpected, or returns some random difficult to understand error message. The best code makes it easy to use that code.
The final test in this code is a vectorized test, in that if you called the code like this:
tooyoung = under_age([12 5 29 75],21)
tooyoung =
1 1 0 0
it will work and return a vector of results. Vectorized code is generally good code, since it allows the user to not be forced to use a loop when they want to use the code many times in a row.
Finally, could I have written code that would have required only one line of code? Thus testing to see if limit was provided, and returning a comparison to age in just one line? Probably, but that would have been unnecessarily complicated, difficult to read and debug. There would have been no gain in the quality of the code or how fast it runs. Good code is simple, easy to read, easy to use, easy to debug.
5 Commenti
Rik
il 5 Ago 2020
Have you read the documentation for the nargin function to see what it does? It returns the n[umber of] arg[uments you provide as] in[put]. So if it is smaller than 2, that means the limit was not provided as an input argument.
Più risposte (25)
Saurabh Kumar
il 28 Mar 2019
Write a function called under_age that takes two positive integer scalar arguments:
- age that represents someone's age, and
- limit that represents an age limit.
The function returns true if the person is younger than the age limit. If the second argument, limit, is not provided, it defaults to 21. You do not need to check that the inputs are positive integer scalars.
function x = under_age(age,limit)
if nargin < 2
limit = 21;
if age < limit
x = true;
else
x = false;
end
end
if nargin == 2
if age < limit
x = true;
else
x = false;
end
end
7 Commenti
Walter Roberson
il 20 Giu 2021
MATLAB (and nearly all programming languages... but not all) are Procedural Languages, in which the order of statements is important.
Suppose you were climbing a ladder. For the most part, you take one step upward at a time. Now suppose you program it that way,
"Move foot 50 cm higher and put weight on it"
But what about when you get to the top?
"Move foot 50 cm higher and put weight on it"
"If there was no higher rung, don't take that step"
Too late. You already put your weight in mid-air before checking whether there was something to put your weight onto.
You instead need a check like
"If there is a higher rung, move foot 50 cm higher and put weight on it"
Check first, before relying on it being there.
The way you coded
if age < limit
if nargin < 2
limit = 21
end
end
is similar to not having checked for a higher rung existing before putting your weight where it would be.
===
It is better programming practice to check for exceptions first, check to see whether a calculation is going to be valid before doing the calculation.
However, there are some situations in which it is valid to "patch up" if you encounter an exception. For example, better programming practice for "1/x if x is not 0, 0 if x is 0" would be
y = zeros(size(x));
mask = x ~= 0;
y(mask) = 1./x(mask);
This code never does any division by 0.
But if you are sure you are using IEEE 754, you can count on the fact that IEEE 754 defines that 1/x has some result for 1/0 (rather than crashing the program), and you can instead do a patch-up-later version:
y = 1./x;
y(x == 0) = 0;
The patch-up-later version of the question would be like,
function too_young = under_age(age, limit)
too_young = age < 21;
if nargin > 1
too_young = age < limit;
end
end
Notice that the variable limit is not used until after it has been checked to be sure that it is present.
Astr
il 8 Set 2019
function too_young = under_age(age, limit)
if nargin == 1
limit = 21;
end
too_young = gt(limit, age);
0 Commenti
mayank ghugretkar
il 7 Giu 2019
this can work too...
A bit compact approach
function too_young = under_age(age, limit)
if nargin < 2
limit = 21;
end
if age < limit
too_young=true;
else
too_young=false;
return
end
end
5 Commenti
Nijita Kesavan Namboothiri
il 25 Giu 2019
function too_young= under_age(age, limit)
if (nargin<2)
limit=21;
end
if (age<limit)
too_young=true
else
too_young=false
end
end
6 Commenti
Kilaru Venkata Krishna
il 2 Mag 2020
its taking age >= limit as older
.......and ......age<limit as young
Siddharth Joshi
il 23 Apr 2020
function too_young = under_age(age,limit)
if nargin<2
limit=21;
end
if age<limit
too_young=true;
else
too_young=false;
end
end
too_young = under_age(18,18)
too_young = under_age(20)
too_young =
logical
0
too_young =
logical
1
0 Commenti
sudeep lamichhane
il 27 Apr 2020
function too_young = under_age(age, limit)
if nargin<2
limit=21;
end
if age < limit
too_young= true;
else
too_young= false;
end
Sai Hitesh Gorantla
il 1 Feb 2020
Modificato: Rik
il 17 Giu 2020
function [too_young] = under_age(age,limit)
if nargin == 2
if age<limit
too_young = true;
else
too_young = false;
end
elseif nargin<2
if age<=21
too_young = true;
else
too_young = false;
end
end
0 Commenti
mohammad elyoussef
il 4 Apr 2020
function c = under_age(a,b)
if nargin < 2
b = 21;
end
if b > a
c = true;
else
c = false;
end
0 Commenti
maha khan
il 9 Apr 2020
function [too_young]= under_age(age,limit)
if (nargin < 2) || isempty(limit)
limit = 21;
end
if age>21
too_young=false;
elseif age < limit
too_young=true;
elseif age==age
too_young=false;
elseif age<=21
too_young=true;
elseif age < age
too_young=false;
elseif age<=21
too_young=true;
else
too_young=true;
end
1 Commento
Walter Roberson
il 9 Apr 2020
Suppose the user passes in a limit of 35, such as testing for eligibility to be President of the United States. And suppose the age passed in is 29. Then if age>21 would be if 29>21 and that would be true, so you would declare too_young=false but clearly the answer should be true: 29 < the specified limit.
Mir Mahim
il 7 Mag 2020
function a = under_age(age,limit)
if nargin < 2
limit = 21;
end
a = age < limit;
end
0 Commenti
Aasma Shaikh
il 26 Mag 2020
function too_young= under_age (age, limit)
if nargin<2
limit=21;
if (age<limit)
too_young = true;
else
too_young = false;
end
elseif ((nargin==2) && (age<limit))
too_young = true;
else
too_young = false;
end
end
% Copy, paste and Run
1 Commento
jaya shankar veeramalla
il 29 Mag 2020
function too_young = under_age(age,limit)
if (nargin < 2) || isempty(limit)
limit = 21;
end
if age < limit
too_young = true;
else age >= limit
too_young = false;
end
0 Commenti
AYUSH MISHRA
il 4 Giu 2020
function too_young =under_age(age,limit)
if nargin <2
limit=21;
end
if age<limit
too_young=true;
else
too_young=false;
end
SOLUTION ;
under_age(18,18)
ans =
logical
0
under_age(20)
ans =
logical
1
0 Commenti
Keshav Patel
il 8 Giu 2020
function too_young =under_age(age,limit)
if nargin<2
limit=21;
if age<limit
too_young=true;
else
too_young=false;
end
end
if nargin ==2
if age<limit
too_young=true;
else
too_young=false;
end
end
1 Commento
DGM
il 21 Feb 2023
This is essentially identical to @Saurabh Kumar's answer, except one of the variable names has been changed.
saurav Tiwari
il 17 Giu 2020
Modificato: Rik
il 17 Giu 2020
function [a]=under_age(n,m)
a=n<m;
if a==1;
fprintf('true')
end
if nargin<2;
m=21;
end
end
2 Commenti
AKASH SHELKE
il 9 Ago 2020
Modificato: AKASH SHELKE
il 9 Ago 2020
function too_young = under_age(age,limit)
if nargin<2
limit = 21;
end
if age < limit
too_young = true;
else
too_young = false;
end
0 Commenti
Muhammad Akmal Afibuddin Putra
il 10 Ago 2020
function too_young = under_age(age,limit)
if nargin < 2
limit = 21;
end
if age < limit
too_young = 1 == 1;
else
too_young = 1 ==2;
end
end
3 Commenti
Omkar Gharat
il 11 Ago 2020
function [too_young] = under_age(age,limit);
% age = input('Enter age of applicant : ')
% limit = input('Enter age limit of applicant : ')
if nargin < 2
limit = 21;
end
if age < limit
too_young = true;
else
too_young = false;
end
end
This is my code .And it is 100% correct
Khom Raj Thapa Magar
il 6 Set 2020
Make sure your indentation is correct while coding
function too_young = under_age(age,limit)
if nargin<2
limit = 21;
if age < limit
too_young = true;
else
too_young = false;
end
end
if nargin == 2
if age < limit
too_young = true;
else
too_young = false;
end
end
Calling functions
>> too_young = under_age(18,18)
>> too_young = under_age(20)
Output:
too_young =
logical
0
too_young =
logical
1
0 Commenti
Jessica Trehan
il 21 Set 2020
function too_young=under_age(age,limit)
if nargin<2
limit=21;
if age<limit
too_young=true;
else
too_young=false;
end
end
if nargin==2
if age<limit
too_young=true;
else
too_young=false;
end
end
%THE PERFECT CODE
7 Commenti
Olha Skurikhina
il 11 Gen 2021
Thank you. It is very stupid. I already solved harder problems along the course but this one couldn`t tackle. That was my problem and plus if the age= limit, it is still should return false so '<=' is incorrect.
amjad ali
il 6 Set 2021
function too_young=under_age(age,limit);
switch nargin
case 1
limit=21;
end
if (limit>age);
too_young=true;
else
too_young=false;
end
switch nargin
case 2
end
if limit>age ;
too_young=true;
else
too_young=false;
end
2 Commenti
amjad ali
il 6 Set 2021
thankx for your comment
i have no idea before that the second switch statement is useless,
VIGNESH B S
il 13 Ott 2021
function [too_young] = under_age(age,limit)
if nargin == 2
too_young = compare(age,limit);
elseif nargin == 1
too_young = compare(age,21);
end
end
function [answer] = compare(x,y)
if x<y
answer = (1>0);
else
answer = (0>1);
end
end
3 Commenti
Image Analyst
il 14 Ott 2021
But it doesn't look elegant. An elegant program would do
answer = x < y;
instead of the clunky
if x<y
answer = (1>0);
else
answer = (0>1);
end
Alexandar
il 29 Giu 2022
This worked well for me. Very short but I sort of don't understand why the "nargout" part worked.
function too_young = under_age(age, limit)
if nargin < 2
limit = 21
end
too_young = age<limit;
if nargout == 2
too_young = true(1);
end
4 Commenti
Muhammad
il 23 Lug 2022
%Help required
Error showing=Output argument "x" (and possibly others) not assigned a value in the execution with "under_age" function.
function x = under_age(age,limit)
if nargin < 2
limit=21;
if age < limit
x=true;
end
end
if nargin==2
if age<limit
x=true;
end
end
3 Commenti
Muhammad
il 23 Lug 2022
function too_young = under_age(age,limit)
if nargin < 2
limit=21;
if age < limit
too_young=true;
else
too_young=false;
end
end
if nargin==2
if age<limit
too_young=true;
else
too_young=false;
end
end
Thanks a lot Mr.Walter Roberson
MURSHED SK
il 13 Ott 2022
Modificato: MURSHED SK
il 13 Ott 2022
This might help:
function too_young = under_age(age, limit)
if nargin <2
limit =21;
end
if age<limit
too_young = true;
else
too_young = false;
end
% the shortest way
1 Commento
DGM
il 13 Ott 2022
Again, how is this different from the many existing answers?
Also, this isn't the shortest way. John's answer at the top of the thread is shorter, and is thoroughly explained.
Vedere anche
Categorie
Scopri di più su Programming Utilities in Help Center e File Exchange
Community Treasure Hunt
Find the treasures in MATLAB Central and discover how the community can help you!
Start Hunting!