• 0

How do i write this pseudocode?


Question

Hi guys,

I'm currently working on writing a psuedocode which would show a list of total sales between any two dates.

Here is what I came up with thus far:

Use Variables: Date 1, Date 2 of Type Integer

Sales, Total Sales of Type Real

Begin

Display ?Enter Start Date?

Accept Date 1

Display ?Enter End Date?

Accept Date 2

For (Date 1>=Start Date, Date 2<=End Date, +1)

Accept Sales

Total Sales:= Total Sales + Sales

End For

Print Total Sales

End

Am I going about this in the right manner? Was the For loop the correct method to use? I would appreciate any advice or suggestions. :)

Link to comment
https://www.neowin.net/forum/topic/917096-how-do-i-write-this-pseudocode/
Share on other sites

9 answers to this question

Recommended Posts

  • 0

A good approach is to write your pseudo code as potential comments or instructions for writing code, not using code, but plain english to save duplication. You then comment out each line as write the appropriate line of code, self documenting if you will.

for each user in user collection
allow user access to repo
then save

would become...

&lt;?php
#for each user in user collection
foreach($usercollection as $user){
	#allow user access to repo
	$user-&gt;allow('repo', true);
	#then save
	$user-&gt;save();
}
?&gt;

  • 0
  On 05/07/2010 at 11:26, AnthonySterling said:

A good approach is to write your pseudo code as potential comments or instructions for writing code, not using code, but plain english to save duplication. You then comment out each line as write the appropriate line of code, self documenting if you will.

for each user in user collection
allow user access to repo
then save

would become...

&lt;?php
#for each user in user collection
foreach($usercollection as $user){
	#allow user access to repo
	$user-&gt;allow('repo', true);
	#then save
	$user-&gt;save();
}
?&gt;

Writing instructions this way could help you and others know what the potential code will do once it is written. A good advice.

Just try to keep them as simple as possible.

  • 0
  On 05/07/2010 at 11:26, AnthonySterling said:

A good approach is to write your pseudo code as potential comments or instructions for writing code, not using code, but plain english to save duplication. You then comment out each line as write the appropriate line of code, self documenting if you will.

for each user in user collectionallow user access to repothen save

would become...

&lt;?php#for each user in user collectionforeach($usercollection as $user){	#allow user access to repo	$user-&gt;allow('repo', true);	#then save	$user-&gt;save();}?&gt;

This is acceptable practice for when you are first beginning to program. Do not do this when you get beyond introductory stuff. You will end up annoying experienced developers far more than you will be helping them. Wading through twice as many lines of code because someone likes to include commenting gems like the following is annoying, and not nearly as helpful as the likely well intentioned author probably meant them to be.

Bad:

// increment integer i by 1
i = i + 1
// remove Item from a list using remove() method.
list.remove(Item)

Also not good (because it is self-explanatory by glancing at the logic):

&lt;?php
#for each user in user collection
foreach($usercollection as $user){
        #allow user access to repo
        $user-&gt;allow('repo', true);
        #then save
        $user-&gt;save();
}
?&gt;

Basically, you don't want to be wasting your time while coding and someone elses time while maintaining, by commenting very obvious things especially on a line-by-line basis. Documenting code is an art. Doing it by simply replacing line-by-line the pseudo-code/notes/whatever with actual code is a terrible practice to get into. A good comment should succinctly explain the purpose of the a chunk of logic, give a brief overview of what is occurring, document any pre-conditions or post-conditions, and in general be summary of what the code is doing without unnecessary verbosity.

However, there is a time and a place to comment line-by-line. If you have some truly obscure algorithm stuff going on then it is necessary. Similarly, if you have a complicated Regex rule then it is also extremely helpful. These are all edge cases though.

  • Like 2
  • 0
  On 04/07/2010 at 17:13, DaMiEn said:

Hi guys,

I'm currently working on writing a psuedocode which would show a list of total sales between any two dates.

Here is what I came up with thus far:

Use Variables: Date 1, Date 2 of Type Integer

Sales, Total Sales of Type Real

Begin

Display ?Enter Start Date?

Accept Date 1

Display ?Enter End Date?

Accept Date 2

For (Date 1>=Start Date, Date 2<=End Date, +1)

Accept Sales

Total Sales:= Total Sales + Sales

End For

Print Total Sales

End

Am I going about this in the right manner? Was the For loop the correct method to use? I would appreciate any advice or suggestions. :)

Why Date1 AND Date2 controlling the For loop ?

Usually it's just one variable, and it gets incremented until the second condition is no longer true..

i.e. Date 1 <= End Date, instead of Date 2 <= End Date

  • 0
  On 04/07/2010 at 17:13, DaMiEn said:

Hi guys,

I'm currently working on writing a psuedocode which would show a list of total sales between any two dates.

Here is what I came up with thus far:

Use Variables: Date 1, Date 2 of Type Integer

Sales, Total Sales of Type Real

Begin

Display ?Enter Start Date?

Accept Date 1

Display ?Enter End Date?

Accept Date 2

For (Date 1>=Start Date, Date 2<=End Date, +1)

Accept Sales

Total Sales:= Total Sales + Sales

End For

Print Total Sales

End

Am I going about this in the right manner? Was the For loop the correct method to use? I would appreciate any advice or suggestions. :)

This problem requires a set operation, and it wouldn't make sense to use a for loop. If anything, you'd use a while loop provided that all the records are sorted by "sales date".

The pseudo code would look something like this:

Begin

Display ?Enter Start Date?

Accept Date 1

Display ?Enter End Date?

Accept Date 2

Sort SalesRecords by Sales Date Ascending

Found := false

While Not EOF(SalesRecords)

If (Sales Date >= Date 1)

Found:= true

End If

Next SalesRecord

End While

If Found = false

Print 0

Return

End If

Total Sales := 0

While Sales Date <= Date 2 And Not EOF(SalesRecords)

Total Sales := Total Sales + Sales

Next SalesRecord

End While

Print Total Sales

End

BTW, this is the kind of use case that a technology like Microsoft's LINQ turns into a triviality:

var totalsales = dc.Sales.Where(x => x.SalesDate >= Date1 && x.SalesDate < Date2.AddDays(1)).Sum(x => x.SalesAmount);

This topic is now closed to further replies.
  • Recently Browsing   0 members

    • No registered users viewing this page.
  • Posts

    • Can't they just allow people to hide the status bar (battery etc.) when Safari is open and to hide the bottom bar completely when scrolling for a true full-screen experience ? Samsung does that on its Samsung Internet Browser and it's a very good experience.
    • Trend Micro Cleaner One Pro 6.8.0.333 by Razvan Serea Trend Micro Cleaner One Pro is an all-in-one disk cleaning and optimization software. Free up disk space by removing bloatware and junk files, and solve small issues before they become big problems. Cleaner One Pro scans your disks quickly and then displays the garbage files so you can decide which of them you want to be removed. Cleaner One Pro also provides some advanced features including start up manager, application manager and registry cleaner. Keep your PC clean and automatically tune it up for the best performance. Cleaner One Pro key features: Junk File Cleaner Quick scan to remove temporary files Erase leftover hidden files from deleted Apps Clear trash on external drives Large File Scanner Find large files on your disk and free up more storage Filter and manage files by size (from 10MB+), date, name and type Avoid accidental deletions: Lock to protect important large files. No escape for local files synced from Cloud drives Disk Map Analyze your storage in a visual and interactive map. Color-coded breakdown of every folder on your disk, with detailed information at one click. Easier to manually navigate and manage folders/files by size. Duplicate Files Customizable folder search range. Find and filter unused file copies in just a few seconds. Similar Photos Tidy up your albums and remove duplicate photos the easy way. Registry Cleaner Scan Windows Registry Automatically Finds errors & residual items in the registry and then cleans or defrags them Start up Manager Control, configure and review the programs and services that start with Windows Speed up boot time easily. App Manager Uninstall third-party apps and associated files in a breeze. View and manage Apps by name, size or date. Batch remove multiple unused Apps. Note: Cleaner One Pro 6.8.0.333 changelog is not yet available. Download: Cleaner One Pro 6.8.0.333 | 66.1 MB (Freeware, paid upgrade available) View: Home Page | Cleaner One Pro for Mac OS | Windows Store | Screenshot Get alerted to all of our Software updates on Twitter at @NeowinSoftware
    • A friend of mine got this icon being displayed on his iPhone 8 [he's currently in Paris, France - so it might be a local/country icon?!] Do you guys & gals know what this means? I looked on Apple's site, but this icon isn't listed: https://support.apple.com/guide/iphone/learn-the-meaning-of-the-status-icons-iphef7bb57dc/ios
    • Trump is a willing victim.
  • Recent Achievements

    • One Month Later
      serfegyed earned a badge
      One Month Later
    • Dedicated
      firey earned a badge
      Dedicated
    • Dedicated
      fettermanj earned a badge
      Dedicated
    • One Month Later
      SekTheFirst earned a badge
      One Month Later
    • First Post
      zayanhani earned a badge
      First Post
  • Popular Contributors

    1. 1
      +primortal
      638
    2. 2
      ATLien_0
      231
    3. 3
      Michael Scrip
      219
    4. 4
      Xenon
      145
    5. 5
      Steven P.
      141
  • Tell a friend

    Love Neowin? Tell a friend!