Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Week2 fixes #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
IdoElk merged 32 commits into week2 from week2_fixes
Apr 28, 2019
Merged

Week2 fixes #14

IdoElk merged 32 commits into week2 from week2_fixes
Apr 28, 2019

Conversation

Copy link
Contributor

@IdoElk IdoElk commented Apr 16, 2019 •
edited
Loading

  • Language - Wording, typo and grammar
  • Code fixes

IdoElk added 19 commits April 15, 2019 03:21
@IdoElk IdoElk requested a review from yammesicka April 16, 2019 19:01
Copy link
Member

@yammesicka yammesicka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great one, I feel lucky to have someone how know some shift reading the notebook so thoroughly.
Thanks!

IdoElk reacted with hooray emoji
@@ -514,7 +516,7 @@
"metadata": {},
"source": [
"<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
"כפי ש<code>+</code> משרשר בין מחרוזות, הוא יודע לשרשר גם בין רשימות:\n",
"כפי ש-<code>+</code> משרשר בין מחרוזות, הוא יודע לשרשר גם בין רשימות:\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use the Hebrew hyphen instead: ־

@@ -1016,7 +1017,7 @@
"<ol style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>הביטוי הבוליאני בשורה 1 מחזיר <samp>False</samp>, מכיוון שכל אחד מהאיברים ברשימה <var>pupils</var> הוא רשימה, ואף אחד מהם אינו המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 2 מחזיר <samp>True</samp>, מכיוון שהרשימה האחרונה בתוך <var>pupils</var> מחזיר רשימה המכילה את המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון אשאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה ישירות את הרשימה הקטנה (<code>['Moshe', 'Splinter']</code>).</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון שאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה איבר שהוא שווה בדיוק ל- (<code>['Moshe', 'Splinter']</code>).</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

שהוא שווה -> ששווה
Use less verbose form when possible.
Also, change the - to ־

@@ -264,14 +278,14 @@
"source": [
"<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
"עד כה שרשרנו מחרוזות בזהירות בעזרת סימן השרשור, <code>+</code>.<br>\n",
"דרך נחמדה לעשות את זה היא <dfn>fstring</dfn>, או בשמן המלא, <dfn>formatted strings</dfn>.<br>\n",
"הדרך הנכונה לעשות את זה היא בעזרת <dfn>fstring</dfn>, או בשמן המלא, <dfn>formatted strings</dfn>.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wouldn't go with "נכונה". Maybe "דרך אפשרית נוספת"...

Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ToFix

" <li>כל עוד התו קיים בצד ימין של המחרוזת, מחק אותו משם.</li>\n",
" <li>כל עוד התו קיים בצד שמאל של המחרוזת, מחק אותו משם.</li>\n",
" </ol>\n",
" <li>עבור כל תו מצד <b>שמאל</b> של המחרוזת</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I find it more complicated than the original. ^^"

Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

@@ -1219,8 +1260,8 @@
"metadata": {},
"source": [
"<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
"קבלו מהמשתמש מילה מסוימת ומשפט.<br>\n",
"בדקו שהמילה מופיעה לפני החצי של המשפט.\n",
"קבלו מהמשתמש מילה מסוימת ומשפט ובדקו שהמילה מופיעה לפני החצי של המשפט.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I split it into short sentences for a reason :P

@@ -186,7 +186,7 @@
"מה התרחש בקוד?<br>\n",
"התחלנו את הגדרת הרשימה באמצעות התו <code dir=\"ltr\" style=\"direction: ltr;\">[</code>.<br>\n",
"מיד אחרי התו הזה דאגנו להכניס איברים לרשימה לפי הסדר הרצוי, כאשר כל איבר מופרד ממשניהו בתו פסיק (<code>,</code>).<br>\n",
"במקרה שלנו, כל איבר הוא מחרוזת המייצגת ראש ממשלה, והם <mark>מסודרים ברשימה</mark> לפי סדר כהונתם.<br>\n",
"במקרה שלנו, כל איבר הוא מחרוזת המייצגת ראש ממשלה ודאגנו להגדיר את הרשימה כך שתכיל את הערכים (ראשי ממשלה) <mark>לפי סדר</mark> כהונתם.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just more verbose. What did you try to accomplish here? :)

Copy link
Contributor Author

@IdoElk IdoElk Apr 16, 2019 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My thought was to explain that we explicitly declared the list in the right order (meaning the list keeps its order from init).
It might be too explicit, if so I'll change it back :)

Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe
"ודאגנו להכניס את ראשי הממשלה לרשימה <mark>לפי סדר</mark> כהונתם"?

Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

, before ודאגנו

@@ -342,6 +342,7 @@
" <div style=\"width: 90%\">\n",
" <p style=\"text-align: right; direction: rtl; clear: both;\">\n",
" המספור של התא הראשון הוא 0, ולא 1. יש לכך סיבות טובות, אבל פעמים רבות זה ירגיש לא טבעי ועלול ליצור באגים.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ועלול ליצור <dfn>באגים</dfn> – חלקים בקוד שלא מתנהגים כפי שחשבנו שיתנהגו.

]
},
{
"cell_type": "markdown",
"metadata": {},
"source": [
"<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
"טריק שימושי בטירוף שנחמד לדעת, הוא שניתן להשתמש בהשמה על מנת לשנות את האיבר שמופיע בתא מסוים ברשימה:\n",
"לפעמים נרצה לשנות את הערך של האיברים ברשימה.<br>\n",
"נוכל להקביל את זה לכך שאנו נפתח את המגירה במקום ספציפי ונחליף בין הערך הישן לחדש.\n",
Copy link
Member

@yammesicka yammesicka Apr 16, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

נוכל להקביל שינוי ערך של איבר לכך...

@@ -186,7 +186,7 @@
"מה התרחש בקוד?<br>\n",
"התחלנו את הגדרת הרשימה באמצעות התו <code dir=\"ltr\" style=\"direction: ltr;\">[</code>.<br>\n",
"מיד אחרי התו הזה דאגנו להכניס איברים לרשימה לפי הסדר הרצוי, כאשר כל איבר מופרד ממשניהו בתו פסיק (<code>,</code>).<br>\n",
"במקרה שלנו, כל איבר הוא מחרוזת המייצגת ראש ממשלה, והם <mark>מסודרים ברשימה</mark> לפי סדר כהונתם.<br>\n",
"במקרה שלנו, כל איבר הוא מחרוזת המייצגת ראש ממשלה ודאגנו להגדיר את הרשימה כך שתכיל את הערכים (ראשי ממשלה) <mark>לפי סדר</mark> כהונתם.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

, before ודאגנו

@@ -341,7 +341,8 @@
" </div>\n",
" <div style=\"width: 90%\">\n",
" <p style=\"text-align: right; direction: rtl; clear: both;\">\n",
" המספור של התא הראשון הוא 0, ולא 1. יש לכך סיבות טובות, אבל פעמים רבות זה ירגיש לא טבעי ועלול ליצור באגים.<br>\n",
" המספור של התא הראשון הוא 0, ולא 1. יש לכך סיבות טובות, אבל פעמים רבות זה ירגיש לא טבעי ועלול ליצור <dfn>באגים</dfn>.<br>\n",
" <em>* באג הוא קטע קוד המתנהג לא כמצופה.</em><br>\n",
Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge with the line about באגים, so it would sound more natural.

המספור של התא הראשון הוא 0, ולא 1. יש לכך סיבות טובות, אבל פעמים רבות זה ירגיש לא טבעי.

בלבול של המתכנת בעקבות ההתנהגות עלול לגרום ל<dfn>באגים<dfn>, קטעי קוד שמתנהגים אחרת משציפה המתכנת.

@@ -1016,7 +1017,7 @@
"<ol style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>הביטוי הבוליאני בשורה 1 מחזיר <samp>False</samp>, מכיוון שכל אחד מהאיברים ברשימה <var>pupils</var> הוא רשימה, ואף אחד מהם אינו המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 2 מחזיר <samp>True</samp>, מכיוון שהרשימה האחרונה בתוך <var>pupils</var> מחזיר רשימה המכילה את המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון אשאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה ישירות את הרשימה הקטנה (<code>['Moshe', 'Splinter']</code>).</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון שאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה איבר ששווה בדיוק ל־ (<code>['Moshe', 'Splinter']</code>).</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can remove the parenthesis around the list, and stick it all together.

@@ -264,14 +278,14 @@
"source": [
"<p style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
"עד כה שרשרנו מחרוזות בזהירות בעזרת סימן השרשור, <code>+</code>.<br>\n",
"דרך נחמדה לעשות את זה היא <dfn>fstring</dfn>, או בשמן המלא, <dfn>formatted strings</dfn>.<br>\n",
"הדרך הנכונה לעשות את זה היא בעזרת <dfn>fstring</dfn>, או בשמן המלא, <dfn>formatted strings</dfn>.<br>\n",
Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ToFix

" <li>כל עוד התו קיים בצד ימין של המחרוזת, מחק אותו משם.</li>\n",
" <li>כל עוד התו קיים בצד שמאל של המחרוזת, מחק אותו משם.</li>\n",
" </ol>\n",
" <li>עבור כל תו מצד <b>שמאל</b> של המחרוזת</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 17, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

" <li>כל עוד התו קיים בצד ימין של המחרוזת, מחק אותו משם.</li>\n",
" <li>כל עוד התו קיים בצד שמאל של המחרוזת, מחק אותו משם.</li>\n",
" </ol>\n",
" <li>תעבור תו-תו מההתחלה של המחרוזת:</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

עבור על כל תו מתחילת המחרוזת
(מקף־עילי, גם בפסקה השנייה)

" </ol>\n",
" <li>תעבור תו-תו מההתחלה של המחרוזת:</li>\n",
" <ul style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>כל עוד התו הינו אחד מהארגומנטים מחק אותו והמשך</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

כל עוד התו נמצא בארגומנט – מחק אותו.

" <li>תעבור תו-תו מההתחלה של המחרוזת:</li>\n",
" <ul style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>כל עוד התו הינו אחד מהארגומנטים מחק אותו והמשך</li>\n",
" <li>אחרת, סיימת את הפעולה.</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אין צורך בשורה

" <li>כל עוד התו הינו אחד מהארגומנטים מחק אותו והמשך</li>\n",
" <li>אחרת, סיימת את הפעולה.</li>\n",
" </ul>\n",
" <li>תעבור תו-תו מהסוף של המחרוזת:</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

עבור תו־תו מסוף המחרוזת:

" </ul>\n",
" <li>תעבור תו-תו מהסוף של המחרוזת:</li>\n",
" <ul style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>כל עוד התו הינו אחד מהארגומנטים מחק אותו והמשך</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

כל עוד התו נמצא בארגומנט – מחק אותו.

" <li>תעבור תו-תו מהסוף של המחרוזת:</li>\n",
" <ul style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>כל עוד התו הינו אחד מהארגומנטים מחק אותו והמשך</li>\n",
" <li>אחרת, סיימת את הפעולה.</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

אין צורך בשורה

@@ -1016,7 +1017,7 @@
"<ol style=\"text-align: right; direction: rtl; float: right; clear: both;\">\n",
" <li>הביטוי הבוליאני בשורה 1 מחזיר <samp>False</samp>, מכיוון שכל אחד מהאיברים ברשימה <var>pupils</var> הוא רשימה, ואף אחד מהם אינו המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 2 מחזיר <samp>True</samp>, מכיוון שהרשימה האחרונה בתוך <var>pupils</var> מחזיר רשימה המכילה את המחרוזת <em>\"Moshe\"</em>.</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון אשאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה ישירות את הרשימה הקטנה (<code>['Moshe', 'Splinter']</code>).</li>\n",
" <li>הביטוי הבוליאני בשורה 3 מחזיר <samp>False</samp>, מכיוון שאין בתוך <var>pupils</var> רשימה שאלו בדיוק הערכים שלה. יש אמנם רשימה שמכילה את האיברים האלו, אבל השאלה הייתה האם הרשימה הגדולה (<var>pupils</var>) מכילה איבר ששווה בדיוק ל־ <code>['Moshe', 'Splinter']</code>.</li>\n",
Copy link
Member

@yammesicka yammesicka Apr 22, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

להסיר רווח

Copy link
Member

@yammesicka yammesicka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yay!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yammesicka yammesicka yammesicka approved these changes

Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

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