Skip to main content
Code Review

Return to Answer

replaced http://codereview.stackexchange.com/ with https://codereview.stackexchange.com/
Source Link

You should use (if the SQL version wasn't T-SQL, which it is. Thanks @Millie Smith @Millie Smith and @Vogel612 @Vogel612)

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

You should use (if the SQL version wasn't T-SQL, which it is. Thanks @Millie Smith and @Vogel612)

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

You should use (if the SQL version wasn't T-SQL, which it is. Thanks @Millie Smith and @Vogel612)

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

added version warning
Source Link
Quill
  • 12k
  • 5
  • 41
  • 93

You should use (if the SQL version wasn't T-SQL, which it is. Thanks @Millie Smith and @Vogel612 )

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

You should use

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

You should use (if the SQL version wasn't T-SQL, which it is. Thanks @Millie Smith and @Vogel612 )

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

Source Link
Quill
  • 12k
  • 5
  • 41
  • 93

You should use

lower(u.Location) LIKE '%australia%'

In case of lowercase writing of Australia.

I wouldn't really assign Reputation an alias, because it's actual title is fine.

You literally assign Location, to be Location, just without the u.

That's a tad unnecessary.

I thought perhaps people may use the country abbreviation, au so you could consider that, but other than that, the code is good.

lang-sql

AltStyle によって変換されたページ (->オリジナル) /