ASP.NET MCV Razor - dynamic folder iteration HTML display











up vote
0
down vote

favorite












Variable Model is essentially a folder from the root-url where its child folders gets displayed through razor.



short story long - I got an internship, no past experience with ASP.NET-MCV, a nucance of an assignment (Yet fulfilling), and now I want to optimize my code, giving the content is to be used for our intern job portal.



Ps. I did look up @functions{} for razor, but there was a lot of different opinions for when (if ever) it should be used. So choose a different route, but please share your opinions.



@foreach (var parentFolder in Model)
{
Stack<Folder> folderStack = new Stack<Folder>();
folderStack.Push(parentFolder);
var currentFolder = folderStack.Pop();
int dummyCounter = 1;

//Parent folder
<div class="row">
<div class="col-sm-2">
<a class="btn"
role="button"
data-toggle="collapse"
href="#@currentFolder.Id"
aria-expanded="false"
aria-controls="@currentFolder.Id">
<span class="@GlyphionCategoryIcon"></span>
</a>
</div>
<div class="col-sm-5">@currentFolder.Id</div>
<div class="col-sm-5">@currentFolder.Name</div>
</div>

<div class="collapse" id="@currentFolder.Id">

@if (currentFolder.FoldersContained != 0)
{
do
{
//Prevents a copy of the parent folder
if (dummyCounter != 1)
{
<div class="row">
<div class="col-sm-2">
<a class="btn"
role="button"
data-toggle="collapse"
href="#@currentFolder.Id"
aria-expanded="false"
aria-controls="@currentFolder.Id">
<span class="@GlyphionCategoryIcon"></span>
</a>
</div>
<div class="col-sm-5">@currentFolder.Id</div>
<div class="col-sm-5">@currentFolder.Name</div>
</div>
}

<div class="collapse" id="@currentFolder.Id">
@if (currentFolder.FoldersContained > 0)
{
for (int i = currentFolder.FoldersContained; i > 0; i--)
{
folderStack.Push(currentFolder.Folders[i - 1]);
dummyCounter++;
}
}

@if (currentFolder.FilesContained != 0)
{
foreach (var file in currentFolder.Files)
{
<div class="row">
<div class="col-sm-2">
<a class="btn"
role="button"
href="@webUrl@file.Url"
target="_blank">
<span class="@GlyphionPaperIcon"></span>
</a>
</div>
<div class="col-sm-5">@file.Id</div>
<div class="col-sm-5">@file.Name</div>
</div>
}
}
</div>

//Ends the while loop
if (folderStack.Count == 0)
{
dummyCounter = 0;
}

//Prepares the next nested folder object
if (folderStack.Count != 0)
{
currentFolder = folderStack.Pop();
}


} while (dummyCounter != 0);
}


@if (parentFolder.FilesContained != 0)
{
foreach (var file in parentFolder.Files)
{
<div class="row">
<div class="col-sm-2">
<a class="btn"
role="button"
href="@webUrl@parentFolder.Url"
target="_blank">
<span class="@GlyphionPaperIcon"></span>
</a>
</div>
<div class="col-sm-5">@parentFolder.Id</div>
<div class="col-sm-5">@parentFolder.Name</div>
</div>

}
}
</div>
}


Output: (dummy data, folders expanded)
enter image description here










share|improve this question









New contributor




Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.
























    up vote
    0
    down vote

    favorite












    Variable Model is essentially a folder from the root-url where its child folders gets displayed through razor.



    short story long - I got an internship, no past experience with ASP.NET-MCV, a nucance of an assignment (Yet fulfilling), and now I want to optimize my code, giving the content is to be used for our intern job portal.



    Ps. I did look up @functions{} for razor, but there was a lot of different opinions for when (if ever) it should be used. So choose a different route, but please share your opinions.



    @foreach (var parentFolder in Model)
    {
    Stack<Folder> folderStack = new Stack<Folder>();
    folderStack.Push(parentFolder);
    var currentFolder = folderStack.Pop();
    int dummyCounter = 1;

    //Parent folder
    <div class="row">
    <div class="col-sm-2">
    <a class="btn"
    role="button"
    data-toggle="collapse"
    href="#@currentFolder.Id"
    aria-expanded="false"
    aria-controls="@currentFolder.Id">
    <span class="@GlyphionCategoryIcon"></span>
    </a>
    </div>
    <div class="col-sm-5">@currentFolder.Id</div>
    <div class="col-sm-5">@currentFolder.Name</div>
    </div>

    <div class="collapse" id="@currentFolder.Id">

    @if (currentFolder.FoldersContained != 0)
    {
    do
    {
    //Prevents a copy of the parent folder
    if (dummyCounter != 1)
    {
    <div class="row">
    <div class="col-sm-2">
    <a class="btn"
    role="button"
    data-toggle="collapse"
    href="#@currentFolder.Id"
    aria-expanded="false"
    aria-controls="@currentFolder.Id">
    <span class="@GlyphionCategoryIcon"></span>
    </a>
    </div>
    <div class="col-sm-5">@currentFolder.Id</div>
    <div class="col-sm-5">@currentFolder.Name</div>
    </div>
    }

    <div class="collapse" id="@currentFolder.Id">
    @if (currentFolder.FoldersContained > 0)
    {
    for (int i = currentFolder.FoldersContained; i > 0; i--)
    {
    folderStack.Push(currentFolder.Folders[i - 1]);
    dummyCounter++;
    }
    }

    @if (currentFolder.FilesContained != 0)
    {
    foreach (var file in currentFolder.Files)
    {
    <div class="row">
    <div class="col-sm-2">
    <a class="btn"
    role="button"
    href="@webUrl@file.Url"
    target="_blank">
    <span class="@GlyphionPaperIcon"></span>
    </a>
    </div>
    <div class="col-sm-5">@file.Id</div>
    <div class="col-sm-5">@file.Name</div>
    </div>
    }
    }
    </div>

    //Ends the while loop
    if (folderStack.Count == 0)
    {
    dummyCounter = 0;
    }

    //Prepares the next nested folder object
    if (folderStack.Count != 0)
    {
    currentFolder = folderStack.Pop();
    }


    } while (dummyCounter != 0);
    }


    @if (parentFolder.FilesContained != 0)
    {
    foreach (var file in parentFolder.Files)
    {
    <div class="row">
    <div class="col-sm-2">
    <a class="btn"
    role="button"
    href="@webUrl@parentFolder.Url"
    target="_blank">
    <span class="@GlyphionPaperIcon"></span>
    </a>
    </div>
    <div class="col-sm-5">@parentFolder.Id</div>
    <div class="col-sm-5">@parentFolder.Name</div>
    </div>

    }
    }
    </div>
    }


    Output: (dummy data, folders expanded)
    enter image description here










    share|improve this question









    New contributor




    Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.






















      up vote
      0
      down vote

      favorite









      up vote
      0
      down vote

      favorite











      Variable Model is essentially a folder from the root-url where its child folders gets displayed through razor.



      short story long - I got an internship, no past experience with ASP.NET-MCV, a nucance of an assignment (Yet fulfilling), and now I want to optimize my code, giving the content is to be used for our intern job portal.



      Ps. I did look up @functions{} for razor, but there was a lot of different opinions for when (if ever) it should be used. So choose a different route, but please share your opinions.



      @foreach (var parentFolder in Model)
      {
      Stack<Folder> folderStack = new Stack<Folder>();
      folderStack.Push(parentFolder);
      var currentFolder = folderStack.Pop();
      int dummyCounter = 1;

      //Parent folder
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      data-toggle="collapse"
      href="#@currentFolder.Id"
      aria-expanded="false"
      aria-controls="@currentFolder.Id">
      <span class="@GlyphionCategoryIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@currentFolder.Id</div>
      <div class="col-sm-5">@currentFolder.Name</div>
      </div>

      <div class="collapse" id="@currentFolder.Id">

      @if (currentFolder.FoldersContained != 0)
      {
      do
      {
      //Prevents a copy of the parent folder
      if (dummyCounter != 1)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      data-toggle="collapse"
      href="#@currentFolder.Id"
      aria-expanded="false"
      aria-controls="@currentFolder.Id">
      <span class="@GlyphionCategoryIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@currentFolder.Id</div>
      <div class="col-sm-5">@currentFolder.Name</div>
      </div>
      }

      <div class="collapse" id="@currentFolder.Id">
      @if (currentFolder.FoldersContained > 0)
      {
      for (int i = currentFolder.FoldersContained; i > 0; i--)
      {
      folderStack.Push(currentFolder.Folders[i - 1]);
      dummyCounter++;
      }
      }

      @if (currentFolder.FilesContained != 0)
      {
      foreach (var file in currentFolder.Files)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      href="@webUrl@file.Url"
      target="_blank">
      <span class="@GlyphionPaperIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@file.Id</div>
      <div class="col-sm-5">@file.Name</div>
      </div>
      }
      }
      </div>

      //Ends the while loop
      if (folderStack.Count == 0)
      {
      dummyCounter = 0;
      }

      //Prepares the next nested folder object
      if (folderStack.Count != 0)
      {
      currentFolder = folderStack.Pop();
      }


      } while (dummyCounter != 0);
      }


      @if (parentFolder.FilesContained != 0)
      {
      foreach (var file in parentFolder.Files)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      href="@webUrl@parentFolder.Url"
      target="_blank">
      <span class="@GlyphionPaperIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@parentFolder.Id</div>
      <div class="col-sm-5">@parentFolder.Name</div>
      </div>

      }
      }
      </div>
      }


      Output: (dummy data, folders expanded)
      enter image description here










      share|improve this question









      New contributor




      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      Variable Model is essentially a folder from the root-url where its child folders gets displayed through razor.



      short story long - I got an internship, no past experience with ASP.NET-MCV, a nucance of an assignment (Yet fulfilling), and now I want to optimize my code, giving the content is to be used for our intern job portal.



      Ps. I did look up @functions{} for razor, but there was a lot of different opinions for when (if ever) it should be used. So choose a different route, but please share your opinions.



      @foreach (var parentFolder in Model)
      {
      Stack<Folder> folderStack = new Stack<Folder>();
      folderStack.Push(parentFolder);
      var currentFolder = folderStack.Pop();
      int dummyCounter = 1;

      //Parent folder
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      data-toggle="collapse"
      href="#@currentFolder.Id"
      aria-expanded="false"
      aria-controls="@currentFolder.Id">
      <span class="@GlyphionCategoryIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@currentFolder.Id</div>
      <div class="col-sm-5">@currentFolder.Name</div>
      </div>

      <div class="collapse" id="@currentFolder.Id">

      @if (currentFolder.FoldersContained != 0)
      {
      do
      {
      //Prevents a copy of the parent folder
      if (dummyCounter != 1)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      data-toggle="collapse"
      href="#@currentFolder.Id"
      aria-expanded="false"
      aria-controls="@currentFolder.Id">
      <span class="@GlyphionCategoryIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@currentFolder.Id</div>
      <div class="col-sm-5">@currentFolder.Name</div>
      </div>
      }

      <div class="collapse" id="@currentFolder.Id">
      @if (currentFolder.FoldersContained > 0)
      {
      for (int i = currentFolder.FoldersContained; i > 0; i--)
      {
      folderStack.Push(currentFolder.Folders[i - 1]);
      dummyCounter++;
      }
      }

      @if (currentFolder.FilesContained != 0)
      {
      foreach (var file in currentFolder.Files)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      href="@webUrl@file.Url"
      target="_blank">
      <span class="@GlyphionPaperIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@file.Id</div>
      <div class="col-sm-5">@file.Name</div>
      </div>
      }
      }
      </div>

      //Ends the while loop
      if (folderStack.Count == 0)
      {
      dummyCounter = 0;
      }

      //Prepares the next nested folder object
      if (folderStack.Count != 0)
      {
      currentFolder = folderStack.Pop();
      }


      } while (dummyCounter != 0);
      }


      @if (parentFolder.FilesContained != 0)
      {
      foreach (var file in parentFolder.Files)
      {
      <div class="row">
      <div class="col-sm-2">
      <a class="btn"
      role="button"
      href="@webUrl@parentFolder.Url"
      target="_blank">
      <span class="@GlyphionPaperIcon"></span>
      </a>
      </div>
      <div class="col-sm-5">@parentFolder.Id</div>
      <div class="col-sm-5">@parentFolder.Name</div>
      </div>

      }
      }
      </div>
      }


      Output: (dummy data, folders expanded)
      enter image description here







      c# performance asp.net-mvc-5 razor






      share|improve this question









      New contributor




      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.











      share|improve this question









      New contributor




      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      share|improve this question




      share|improve this question








      edited 23 mins ago





















      New contributor




      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 36 mins ago









      Alexander Lawaetz

      12




      12




      New contributor




      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      Alexander Lawaetz is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.



























          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function () {
          return StackExchange.using("mathjaxEditing", function () {
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix) {
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          });
          });
          }, "mathjax-editing");

          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "196"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });






          Alexander Lawaetz is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f209823%2fasp-net-mcv-razor-dynamic-folder-iteration-html-display%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown






























          active

          oldest

          votes













          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          Alexander Lawaetz is a new contributor. Be nice, and check out our Code of Conduct.










          draft saved

          draft discarded


















          Alexander Lawaetz is a new contributor. Be nice, and check out our Code of Conduct.













          Alexander Lawaetz is a new contributor. Be nice, and check out our Code of Conduct.












          Alexander Lawaetz is a new contributor. Be nice, and check out our Code of Conduct.
















          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.





          Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


          Please pay close attention to the following guidance:


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f209823%2fasp-net-mcv-razor-dynamic-folder-iteration-html-display%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Quarter-circle Tiles

          build a pushdown automaton that recognizes the reverse language of a given pushdown automaton?

          Mont Emei