Home » Vehicle registration task [closed]

Vehicle registration task [closed]


First off, variables and functions should be in snake_case, not PascalCase. Only classes should be in PascalCase. If you have a variable with an unchangining value, (constant), it should be in UPPERCASE_SNAKE_CASE.

Secondly, I’m noticing that you’re putting variables in parentheses in a lot of your expressions. For example, Speed = (Distance) / (TimeTaken) can become speed = distance / time_taken, or if (Speed) > (Limit) can become if speed > speed_limit.

You should be using context managers to manage files. When you just use open(), and then close it using .close(), certain memory issues can occur. The proper way to open a file can be seen in the below example:

with open("path/to/myfile", "w") as file_name:
    # Do stuff with file_name

Using this method, there’s no need to call .close() on the file, and the file is closed explicitly, so you don’t need to worry about memory issues.

I notice that you repeat the Speed = (Distance) / (TimeTaken) a lot. Preferably, when you have repetition like this in your code, you can shorten it into a function. In this case, I’d create the following function:

def calculate_speed(distance, time_taken):
    return distance / time_taken

Finally, add some comments to this code. You don’t need comments everywhere, but you should add some in places where your code may be slightly harder to understand just by reading it.

For any style guide tips, see Python’s official style guide, PEP8.

I’m just going to focus on the function NumPlate, because otherwise I’d be writing a novel in this answer.

First: It’s defined in a weird place. Is there any particular reason why you only want it to exist if Speed > Limit?

Second: Why is the only argument self when it’s not an instance method?

Third: I cannot for the life of me tell what this function is supposed to do. Right now, you have NumPlate = str(input("Enter the vehicle's number plate.")), which makes me suspect that the user is supposed to enter in a plate number, but then you have all of that randomization stuff as well.

Fourth: As is, that for loop is effectively useless. The first letter of plateFormat is defined as 'L', so the loop hits the first if block, skips the elif block, hits that break statement, and exits the function (returning None because it never hit a return statement).

Related Solutions

Pin-board effect with CSS [closed]

You can use JavaScript to accomplish this but it can't be done with CSS floats alone. A library like jQuery masonry will do it well. The reason? The specs on floats. In particular #5 which says, "The outer top of a floating box may not be higher than the outer...

Calculate the sum with minimum usage of numbers

Here's a hint: 23 : 11 + 11+ 1 ( 3 magic numbers) 120: 110+ 10 (2 magic numbers) The highest digit in the target number is the answer, since you need exactly k magic numbers (all having 1 in the relevant position) in order for the sum to contain the digit k. So...

Why not drop the “auto” keyword? [duplicate]

Your proposal would be rejected on the basis of backward compatibility alone. But let's say for the sake of argument that the standards committee like your idea. You don't take into account the numerous ways you can initialize a variable widget w; // (a) widget...

Recursive to iterative using a systematic method [closed]

So, to restate the question. We have a function f, in our case fac. def fac(n): if n==0: return 1 else: return n*fac(n-1) It is implemented recursively. We want to implement a function facOpt that does the same thing but iteratively. fac is written almost in...

How can I match values in one file to ranges from another?

if the data file sizes are not huge, there is a simpler way $ join input1 input2 | awk '$5<$4 && $3<$5 {print $2, $5-$3+1}' B100002 32 B100043 15 B123465 3 This Perl code seems to solve your problem It is a common idiom: to load the entire...

Javascript difference between “=” and “===” [duplicate]

You need to use == or === for equality checking. = is the assignment operator. You can read about assignment operators here on MDN. As a quick reference as you are learning JS: = assignment operator == equal to === equal value and equal type != not equal !==...

Compiler complains about misplaced else [closed]

Your compiler complains about an misplaced else because, well, there is an else without a preceding if: // ... for (j=1; j<n-i; j++) { if(a[j]<=a[j+1]) { // ... } // END OF IF } // END OF FOR else { continue; } // ... The else in your code does not follow...

Bootstrap – custom alerts with progress bar

/* !important are just used to overide the bootstrap css in the snippet */ .alertContainer { border-radius: 0 !important; border-width: 0 !important; padding: 0 !important; height: auto !important; position: absolute !important; bottom: 15px !important; left:...

How to Garbage Collect an external Javascript load?

Yes, s.onload = null is useful and will garbage collect! As of 2019, it is not possible to explicitly or programmatically trigger garbage collection in JavaScript. That means it collects when it wants. Although there is cases where setting to null may do a GC...

Math programming with python

At first, what you are looking for is the modulo operator and the function math.floor() Modulo from wikipedia: In computing, the modulo operation finds the remainder after division of one number by another (sometimes called modulus). for example: 12%12=0...

Android slide over letters to create a word [closed]

Here some advice you can use: First for each cell you can create an object that represents the state of that cell: class Cell { char mChar; int row,column; boolean isSelected; } then you can create a 2D array of your cells Cell[][] mTable = ... For views you...

Sum two integers in Java

You reused the x and y variable names (hence the variable x is already defined in method main error), and forgot to assign the ints read from the Scanner to the x and y variables. Besides, there's no need to create two Scanner objects. public static void...

Extend three classes that implements an interface in Java

Using this simplified implementation of the library, using method() instead of M(): interface IFC { void method(); } class A implements IFC { public void method() { System.out.println("method in A"); }; } As akuzminykh mentions in their comment You'd write a...