StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PO
primarykey
Id
17928665
data
AcceptedAnswerId
0
AnswerCount
0
ClosedDate
CommentCount
0
CommunityOwnedDate
CreationDate
2013-07-29T15:58:29.420
FavoriteCount
0
LastActivityDate
2013-07-29T15:58:29.420
LastEditDate
LastEditorUserId
0
OwnerUserId
66131
ParentId
17927835
PostTypeId
2
Score
4
ViewCount
0
LastEditorDisplayName
text
Body
Firstly, there are three results your function can have: do nothing, call <code>this.close()</code> or call <code>this.open()</code>. So ideally the resulting function will just have one if statement which determines which result is used. The next step is to extract all boolean code into variables. Eg <code>var leftPastCenter = this.content.getBoundingClientRect().left > viewport / 2</code>. Finally, use boolean logic to simplify it step by step. Here is how I did it: Firstly, extract all boolean variables: <pre><code>function () { var duration = +new Date() - start.time, isPastHalf = Number(duration) < 250 && Math.abs(delta.x) > 20 || Math.abs(delta.x) > viewport / 2, direction = delta.x < 0, leftPastCenter = this.content.getBoundingClientRect().left > viewport / 2, positiveDelta = this.isEmpty(delta) || delta.x > 0, isPulled = pulled === true; // I'll assume the test is needed rather than just using pulled. if (!isScrolling) { if (isPastHalf) { if (direction) { this.close(); } else { if (leftPastCenter && isPulled) { this.close(); return; } this.open(); } } else { if (leftPastCenter) { if (positiveDelta) { this.close(); return; } this.open(); return; } this.close(); } } } </code></pre> The easiest part to pull out is realizing if <code>isScrolling</code> is true, nothing ever happens. This immediately gets rid of one level of nesting: <pre><code> // above same if (isScrolling) { return; } if (isPastHalf) { if (direction) { this.close(); } else { if (leftPastCenter && isPulled) { this.close(); return; } this.open(); } } else { if (leftPastCenter) { if (positiveDelta) { this.close(); return; } this.open(); return; } this.close(); } } </code></pre> Now look at the cases <code>this.open()</code> are called. If <code>isPastHalf</code> is true, <code>this.open()</code> is only called when <code>!direction</code> and <code>!(leftPastCenter && isPulled)</code>. If <code>isPastHalf</code> is false, then <code>this.open()</code> is only called when <code>leftPastCenter</code> and <code>!positiveDelta</code>: <pre><code> // above same if (isScrolling) { return; } if (isPastHalf) { if (!direction && !(leftPastCenter && isPulled)) { this.open(); } else { this.close(); } } else { if (leftPastCenter && !positiveDelta) { this.open(); } else { this.close(); } } </code></pre> Flipping the ifs (so <code>this.close()</code> comes first), makes the code a bit neater, and gives my final version: <pre><code> function () { var duration = +new Date() - start.time, isPastHalf = Number(duration) < 250 && Math.abs(delta.x) > 20 || Math.abs(delta.x) > viewport / 2, direction = delta.x < 0, leftPastCenter = this.content.getBoundingClientRect().left > viewport / 2, positiveDelta = this.isEmpty(delta) || delta.x > 0, isPulled = pulled === true; // I'll assume the test is needed rather than just using pulled. if (isScrolling) { return; } if (isPastHalf) { if (direction || (leftPastCenter && isPulled)) { this.close(); } else { this.open(); } } else { if (!leftPastCenter || positiveDelta) { this.close(); } else { this.open(); } } } </code></pre> It is difficult for me to do more, without knowing your codebase. One thing to note is <code>direction</code> and my new variable <code>positiveDelta</code> are nearly identical - you could possible remove <code>positiveDelta</code> and just use <code>direction</code>. Also, <code>direction</code> isn't a good name for a boolean, something like <code>movingLeft</code> would be better. 
Tags
Title
singulars
PostAcceptedAnswerId
1. This table or related slice is empty.
PostParentId
1. POhow could I reduce the cyclomatic complexity?
 singulars
 PostTypePostTypeId
 PTQuestion
PostTypePostTypeId
1. PTAnswer
UserLastEditorUserId
1. This table or related slice is empty.
UserOwnerUserId
1. USDavid Miani
plurals
PostLinksPostIdRelatedPostId
1. This table or related slice is empty.
PostLinksRelatedPostIdPostId
1. This table or related slice is empty.
PostsAcceptedAnswerId
1. This table or related slice is empty.
PostsParentIdCreationDate
1. This table or related slice is empty.
VotesPostIdCreationDate
1. VO
 singulars
 PostPostId
 PO
 UserUserId
 This table or related slice is empty.
 VoteTypeVoteTypeId
 VTUpMod
CommentsPostId
1. This table or related slice is empty.

Querying!

Guidance

A row detail

Detail views are divided into sections. All the information in the data section comes from columns in the selected row. The other sections display data from other, related rows.

Related data can be related in a to-one or a to-many fashion. Captions of data related in a to-many fashion link to a list view showing a filtered view of the table.

Try moving around until you find a non-empty to-many entry and click on the label to get to one. You can move back to the root by clicking on the database name in the header.