8
votes
Accepted
Markup Language Parser in C++
Naming things
The first thing that strikes me about your code is that you have a lot of variable and struct names that are unnecessarily abbreviated. While it saves some typing in the beginning, it ...
2
votes
Accepted
Regex and pandas to read forecast sky condition string
At the start of get_sky_condition(), I don't see why you do a .str.strip() when defining ...
2
votes
Accepted
Text splitter using Regular Expressions in Python
Include spaces after commas. It's a virtually costless habit and it helps
with code readability and editability.
Constants are typically named with uppercase. Your program begins with
several ...
2
votes
Reading 12-bit words from 32-bit stream
Overview:
I can't see anything wrong with this code on a read through it (I did not compile it).
All my comments are minor and just nit picky.
So the below is just a few things I would do differently (...
1
vote
Regex and pandas to read forecast sky condition string
You've landed in trouble with your indices again. I think the shape of your dataframe significantly mischaracterises what your data are actually saying:
Per station,
per station observation time,
per ...
1
vote
Python script for zlib decompression
It works, it's reasonably clear in that you use a library. Mostly it's fine.
Give inflate a docstring which clearly says what it takes in, what it returns. Rename ...
Only top scored, non community-wiki answers of a minimum length are eligible
Related Tags
parsing × 1291python × 308
java × 176
c++ × 157
c# × 153
performance × 130
strings × 124
c × 113
beginner × 112
javascript × 97
python-3.x × 82
regex × 73
xml × 69
json × 56
csv × 51
haskell × 48
php × 45
object-oriented × 41
math-expression-eval × 40
html × 39
file × 39
algorithm × 35
rust × 34
datetime × 32
ruby × 30