Jump to content

Need help to understand code piece.


wadej30084

Recommended Posts

9 minutes ago, mconte said:

Yes, that describes how one pre-increment operation works on a variable. That page does not explain how one or more pre- or post-increment operators on the same variable work when in the same expression.

The concept is called a sequence point. Look into it: most everything you'll find will talk about C/C++, but it applies to PHP as well.

Link to comment
Share on other sites

33 minutes ago, ginerjm said:

I personally can not see how I would ever write a block of code that looks like this!! What is that supposed to accomplish?   Never mind - I don't want to know.

Obviously it isn't real code someone would write. Not all of it. But that last line, that's something one could reasonably write in some circumstance.

Link to comment
Share on other sites

Ok, I see you like it, I have another one.
Same situation have code piece:
 

function getAdminName()
{
    global $admin;
    return ($admin['gender'] ? 'Mr' : 'Mrs').' '.$admin['fullname'];
}

and simple question:

"This code isn't ideal. Explain what the problem is and what you could do to fix it"

it's all, please help! Today I have no idea what's wrong except this is creepy realization and I do this in totally another way.

P.S. ginerjm asking what is it, this is a part of practical tasks of technical php test and I trying do my best to solve it, but even if I fail on exam I need to understand what's it about.

All this about: "You need to think globally, we need to see how you think."
And like
Ross Geller's little hand monkey, but in my head, beating in cymbals plates, with deep understanding of this code problem eyes look.

Thanks for help!

Link to comment
Share on other sites

31 minutes ago, maxxd said:

The portion of your last code bit in parenthesis is called a ternary operator. I recommend reading up on it as it's something you'll see often if my experience is at all typical.

Thanks, for reply. I know what is ternary operator, also I know about latest deprecated features about this operator in 7.4 version, but i don't see here any problem with it, any complex use like 1 ? 2 : 3 ? 4 : 5; , the problem is what i don't understand what I'm must looking for in this piece of code. Any advice?

Link to comment
Share on other sites

1 hour ago, wadej30084 said:

but i don't see here any problem with it, any complex use like 1 ? 2 : 3 ? 4 : 5;

But here's the question: what is that expression going to evaluate to? 2 or 4?

1 hour ago, wadej30084 said:

the problem is what i don't understand what I'm must looking for in this piece of code. Any advice?

The problem is the last line. It uses $i++ and $i in one statement, and that's a no-no. Did you see when I mentioned sequence points earlier? Look into it.

Link to comment
Share on other sites

Hi, maybe I'm saying something wrong, but I'm talking about this piece of code, another from first post:

function getAdminName()
{
    global $admin;
    return ($admin['gender'] ? 'Mr' : 'Mrs').' '.$admin['fullname'];
}

 

Help is needed with it, be answering on the question:
"This code isn't ideal. Explain what the problem is and what you could do to fix it"

Link to comment
Share on other sites

59 minutes ago, wadej30084 said:

I'm talking about this piece of code

the convention around here is "New question, new thread". 
That allows for short, direct answer to short, direct questions instead of long, rambling threads where all the "Goodness" gets lost. 

 

1 hour ago, wadej30084 said:

function getAdminName() 
{
 
   global $admin;
   return $admin['gender'] ? 'Mr' : 'Mrs') . ' ' $admin['fullname'] ;
}

Some comments on the above: 

  • the use of "global" breaks encapsulation, requiring the environment "outside" the function to provide the variable. 
    It is better to pass the data as an argument to the function. 
  • What value does admin['gender'] have? 
    Any value passed that resolves to true will cause the ternary operator to return "Mr" and everything else will return "Mrs". 
  • The code makes no attempt to ensure that the array indexes used actually exist; this may or may not be an issue.  
  • What if the individual is female and not married
    They might object to being called "Mrs". 
  • What if the individual is not gender-identifying
    They would object most strongly to be referred to by either of the terms used here. 

Marital status and/or gender are both Personal Data and should be stored in the User's "record" (whatever form that takes) so that it can be managed by/on behalf of the User and changed over time.  

Regards, 
   Phill  W.

 

  • Like 2
Link to comment
Share on other sites

4 minutes ago, Phi11W said:

the convention around here is "New question, new thread". 
That allows for short, direct answer to short, direct questions instead of long, rambling threads where all the "Goodness" gets lost. 

 

Some comments on the above: 

  • the use of "global" breaks encapsulation, requiring the environment "outside" the function to provide the variable. 
    It is better to pass the data as an argument to the function. 
  • What value does admin['gender'] have? 
    Any value passed that resolves to true will cause the ternary operator to return "Mr" and everything else will return "Mrs". 
  • The code makes no attempt to ensure that the array indexes used actually exist; this may or may not be an issue.  
  • What if the individual is female and not married
    They might object to being called "Mrs". 
  • What if the individual is not gender-identifying
    They would object most strongly to be referred to by either of the terms used here. 

Marital status and/or gender are both Personal Data and should be stored in the User's "record" (whatever form that takes) so that it can be managed by/on behalf of the User and changed over time.  

Regards, 
   Phill  W.

 

Wow, this a big explanation, many thanks Phill!
P.S. I keep in mind about new thread.

Link to comment
Share on other sites

Phi11w provided an excellent and thorough analysis.  In particular consider this data for $admin:

 

<?php

$admin = ['fullname' => 'Sally Smith', 'gender' => 'F'];

function getAdminName() 
{ 
   global $admin;
   return ( $admin['gender'] ? 'Mr' : 'Mrs') . ' ' . $admin['fullname'] ;
}

echo getAdminName();

Do you get what you would expect for a gender 'F' person named 'Sally Smith'?  While all of Phi11w's points are valid, I think this is the main thrust of the issue.  Even without knowing the scheme to be used, it is unlikely that gender would be a boolean value.

Another issue being glossed over, is that 'Mrs' has the connotation of marriage, as well as Barand's point about 'Dr' and other sultations, but that makes things more complicated, and I don't think that was factored into the question, because the existing code has an obvious data typing issue.

So an improved function (assuming php7):

 

<?php

$admin1 = ['fullname' => 'Sally Smith', 'gender' => 'F'];
$admin2 = ['fullname' => 'Randy Jones', 'gender' => 'm'];
$admin3 = ['fullname' => 'Pat Samuels'];

function getAdminName($admin) 
{ 
   $admin['fullname'] = trim($admin['fullname']) ?? 'unknown';   
   $admin['gender'] = $admin['gender'] ?? '';
  
   $pre = '';
   switch (strtolower($admin['gender'])) {
       case 'f':
           $pre = 'Mrs ';
           break;
       case 'm':
           $pre = 'Mr ';
           break;
        default:
            $pre = '';
   }
   return $pre . $admin['fullname'];
}

echo getAdminName($admin1) . PHP_EOL;
echo getAdminName($admin2) . PHP_EOL;
echo getAdminName($admin3) . PHP_EOL;
Quote

Output:

Mrs Sally Smith

Mr Randy Jones

Pat Samuels

 

 

 

 

 

 

Link to comment
Share on other sites

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.